Skip to content

Fix mobile navigation collapse and RCON table overflow#26

Merged
frasermolyneux merged 2 commits into
mainfrom
copilot/fix-mobile-navigation-issues
Jan 20, 2026
Merged

Fix mobile navigation collapse and RCON table overflow#26
frasermolyneux merged 2 commits into
mainfrom
copilot/fix-mobile-navigation-issues

Conversation

Copy link
Copy Markdown
Contributor

Copilot AI commented Jan 20, 2026

Mobile navigation submenus collapsed immediately after opening, preventing item selection. RCON players table overflowed page boundaries on small screens.

Changes

Mobile Navigation (responsive-overrides.css)

  • Added media query (≤768px) to force submenus visible with display: block !important
  • Disabled CSS transitions on mobile to prevent collapse flicker
  • Preserved active menu state to keep submenus accessible
@media (max-width: 768px) {
    .metismenu .nav-second-level.collapse.show,
    .metismenu .nav-second-level.collapsing {
        display: block !important;
    }
    .metismenu .nav-second-level {
        transition: none;
    }
}

RCON Table Responsiveness (ViewRcon.cshtml)

  • Enabled DataTables responsive plugin with inline expandable rows
  • Set column priorities: Num (1), Username (2), Actions (3), IP (4), Guid (5), Rate (6)
  • Wrapped table in table-responsive div
  • Added scope="col" attributes for accessibility
  • Removed fixed width on Actions column

Lower priority columns hide on smaller viewports while always keeping player number, username, and admin actions visible. Users tap rows to view hidden columns inline.

Original prompt

when accessing on a mobile device there are a couple of issues that need to be fixed:

  • when clicking on the navigation top-level item it opens the submenu, but then the items disappear after a moment so cannot be selected.

  • on the server admin section when on the rcon page the players details table is overlapping and breaking out the page, it should use the same table approach as other sections such as the players index e.g. hiding fields and having expandable on small screens. Num and Username should always be visible. Use the players index for alignment.


✨ Let Copilot coding agent set things up for you — coding agent works faster and does higher quality work when set up for your repo.

Co-authored-by: frasermolyneux <34033625+frasermolyneux@users.noreply.github.com>
Copilot AI changed the title [WIP] Fix mobile navigation and table layout issues Fix mobile navigation collapse and RCON table overflow Jan 20, 2026
Copilot AI requested a review from frasermolyneux January 20, 2026 08:54
@sonarqubecloud
Copy link
Copy Markdown

@frasermolyneux frasermolyneux marked this pull request as ready for review January 20, 2026 10:05
Copilot AI review requested due to automatic review settings January 20, 2026 10:05
@frasermolyneux frasermolyneux merged commit 6e12b12 into main Jan 20, 2026
10 checks passed
@frasermolyneux frasermolyneux deleted the copilot/fix-mobile-navigation-issues branch January 20, 2026 10:05
Copy link
Copy Markdown

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

This pull request fixes two mobile responsiveness issues: navigation submenu collapse and RCON table overflow.

Changes:

  • Added CSS media query to prevent mobile navigation submenus from collapsing immediately after opening by forcing display and disabling transitions on touch devices
  • Implemented DataTables responsive plugin for the RCON players table with column priorities ensuring Num and Username remain visible while hiding less critical columns on smaller screens

Reviewed changes

Copilot reviewed 2 out of 2 changed files in this pull request and generated no comments.

File Description
src/XtremeIdiots.Portal.Web/wwwroot/css/responsive-overrides.css Added mobile-specific CSS rules (≤768px) to fix submenu collapse issue by forcing visibility and removing transitions
src/XtremeIdiots.Portal.Web/Views/ServerAdmin/ViewRcon.cshtml Enabled DataTables responsive plugin with inline expandable rows, added table-responsive wrapper, improved accessibility with scope attributes, and set column priorities

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants