Skip to content

Distinguish navbar active state from hover state - #1079

Open
Mahnoor-Zaffar wants to merge 1 commit into
napari:mainfrom
Mahnoor-Zaffar:fix/navbar-active-hover-styles
Open

Distinguish navbar active state from hover state#1079
Mahnoor-Zaffar wants to merge 1 commit into
napari:mainfrom
Mahnoor-Zaffar:fix/navbar-active-hover-styles

Conversation

@Mahnoor-Zaffar

Copy link
Copy Markdown
# References and relevant issues

Closes #(issue-number)

# Description

The hover and active states in the top navigation bar shared the same styling — both showed a `3px` solid bottom border in the text color. This made it confusing to tell which page was currently active when hovering over other nav items.

The fix increases the active state's bottom border from **3px to 4px**, keeping the same color and style, so the active page is visually distinguishable from a hovered item at a glance.

| State   | Before | After  | Font weight |
|---------|--------|--------|-------------|
| Hover   | 3px    | 3px    | 500         |
| Active  | 3px    | **4px** | **700**     |

Fill in the issue number (e.g. Closes #1080) once confirmed.

@github-actions github-actions Bot added the documentation Improvements or additions to documentation label Jul 22, 2026
@TimMonko

Copy link
Copy Markdown
Member

Hi @Mahnoor-Zaffar, thank you for working on this PR! We are grateful for your interest in our project.
I do see the difference, but I think there could be something more, like a color change in the font or something. That said, this change should be upstreamed to https://github.com/napari/napari-sphinx-theme
Please let me know if you need any assistance with this.
It will also be helpful if you could provide a screenshot and accurately fill out the PR description and be sure to follow our AI contribution policy

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

Labels

documentation Improvements or additions to documentation

Projects

Status: No status

Development

Successfully merging this pull request may close these issues.

2 participants