Skip to content

Improve Header Menu Styling#9105

Merged
dartajax merged 1 commit intoilios:masterfrom
jrjohnson:locale-menu
Feb 3, 2026
Merged

Improve Header Menu Styling#9105
dartajax merged 1 commit intoilios:masterfrom
jrjohnson:locale-menu

Conversation

@jrjohnson
Copy link
Copy Markdown
Member

Put a few fixes together:

  • No longer jumps when expanded as the icon changes
  • Extra space above logout gone when hovering
  • Dropdown aligns with button and doesn't overlap it anywhere
  • More consistent spacing between menus
  • At larger screen sizes the items are centered

Put a few fixes together:
  - No longer jumps when expanded as the icon changes
  - Extra space above logout gone when hovering
  - Dropdown aligns with button and doesn't overlap it anywhere
  - More consistent spacing between menus
  - At larger screen sizes the items are centered
@netlify
Copy link
Copy Markdown

netlify Bot commented Jan 30, 2026

Deploy Preview for ilios-frontend ready!

Name Link
🔨 Latest commit c7255ae
🔍 Latest deploy log https://app.netlify.com/projects/ilios-frontend/deploys/697d21ae9a3f8100088bdffc
😎 Deploy Preview https://deploy-preview-9105--ilios-frontend.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify project configuration.

@github-actions
Copy link
Copy Markdown
Contributor

🔍 Visual Diff Results

🔍 Comparing screenshots...

Baseline: /home/runner/work/frontend/frontend/baseline-frontend
Candidate: /home/runner/work/frontend/frontend/candidate-frontend
Output: /home/runner/work/frontend/frontend/visual-diff-9105

📊 Results:
Total images: 752
With differences: 0
Identical: 752
Removed: 0
Added: 0

✅ All checks passed!

📄 Report generated: /home/runner/work/frontend/frontend/visual-diff-9105/index.html

Download the results.

@jrjohnson jrjohnson marked this pull request as ready for review January 30, 2026 21:39
Copy link
Copy Markdown
Member

@stopfstedt stopfstedt left a comment

Choose a reason for hiding this comment

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

nice. LGTM.

@dartajax dartajax merged commit 6d76077 into ilios:master Feb 3, 2026
59 of 60 checks passed
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