Skip to content

LIBSCH1-76 Bug fix: menu width correction#1208

Merged
haitzlm merged 2 commits intodevelopfrom
libsch1-76-fix-menu-width
Apr 20, 2026
Merged

LIBSCH1-76 Bug fix: menu width correction#1208
haitzlm merged 2 commits intodevelopfrom
libsch1-76-fix-menu-width

Conversation

@Janell-Huyck
Copy link
Copy Markdown
Contributor

@Janell-Huyck Janell-Huyck commented Apr 17, 2026

Fix sidebar facet menu width when body uses flex layout

This PR fixes an issue where the sidebar facet menu items appeared only one character wide after the body was set to display: flex. The solution is to explicitly set display: block on the facet menu container and its children, ensuring correct width and layout.

Changes:

  • Add display: block to .facet-values, .facet-values > *, .facet-label, and .facet-count in hyrax/dashboard.scss.
  • Add width: 100% to the same to allow labels to display in one line

No other changes are included. This resolves the menu width regression while preserving the site-wide flexbox layout.

Before this change:
Screenshot 2026-04-17 at 9 28 30 AM

After this change:
Screenshot 2026-04-17 at 11 50 27 AM
Screenshot 2026-04-17 at 11 50 21 AM

@Janell-Huyck Janell-Huyck changed the title make menu use display:block LIBSCH1-76 Bug fix: menu width correction Apr 17, 2026
@Janell-Huyck Janell-Huyck changed the title LIBSCH1-76 Bug fix: menu width correction WIP. LIBSCH1-76 Bug fix: menu width correction Apr 17, 2026
@Janell-Huyck Janell-Huyck changed the title WIP. LIBSCH1-76 Bug fix: menu width correction LIBSCH1-76 Bug fix: menu width correction Apr 17, 2026
Copy link
Copy Markdown
Contributor

@haitzlm haitzlm left a comment

Choose a reason for hiding this comment

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

The CSS is correct. Though I dont have this running locally, at least i can confirm the CSS.

@haitzlm haitzlm merged commit 9508232 into develop Apr 20, 2026
4 checks passed
@haitzlm haitzlm deleted the libsch1-76-fix-menu-width branch April 20, 2026 01:37
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.

2 participants