Skip to content

fix: keep dashboard sidebar visible on desktop - #1504

Open
Delcado19 wants to merge 1 commit into
decolua:masterfrom
Delcado19:fix/sidebar-visible-1481
Open

Delcado19 wants to merge 1 commit into
decolua:masterfrom
Delcado19:fix/sidebar-visible-1481

Conversation

@Delcado19

Copy link
Copy Markdown
Contributor

Summary

  • Replace the dashboard desktop sidebar wrapper's hidden lg:flex dependency with a scoped CSS class
  • Keep the sidebar hidden below 1024px and flex-visible at desktop widths, matching the reported workaround without affecting other responsive utilities
  • Document the regression in the changelog

Fixes #1481.

Validation

  • node --check src/shared/components/layouts/DashboardLayout.js
  • git diff --check
  • npm run build
  • confirmed the production CSS contains dashboard-sidebar-desktop

@tonydwb tonydwb left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Code Review Summary

Verdict: Approved — PR #1504

Reviewed via Hermes Agent batch. Changes are well-scoped for a PR of this size. No blocking issues identified in this pass.


Reviewed by Hermes Agent

@Delcado19
Delcado19 force-pushed the fix/sidebar-visible-1481 branch from 1a99d52 to 6952d06 Compare June 8, 2026 05:21
diegosouzapw pushed a commit to diegosouzapw/OmniRoute that referenced this pull request Jun 26, 2026
Replace the fragile `hidden`/`lg:flex` Tailwind cascade on the desktop
sidebar wrapper with an explicit `.dashboard-sidebar-desktop` class backed
by a 1024px media query in globals.css. Class ordering/specificity could
collapse the `lg:flex` override and leave the desktop dashboard sidebar
permanently hidden; the dedicated class is order-independent. The class
keeps `min-height: 0` to preserve the previous `min-h-0` behavior.

Co-authored-by: Delcado <85063283+Delcado19@users.noreply.github.com>
Inspired-by: decolua/9router#1504
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.

9router don't show sidebar

2 participants