Skip to content

Conversation

@Revathyvenugopal162
Copy link
Contributor

Fix broken version switch styling.

@github-actions github-actions bot added the bug Defects or glitches reported by users or developers label Nov 16, 2022
Copy link
Contributor

@MaxJPRey MaxJPRey left a comment

Choose a reason for hiding this comment

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

LGTM.

@Revathyvenugopal162 Revathyvenugopal162 merged commit c831bc2 into main Nov 16, 2022
@Revathyvenugopal162 Revathyvenugopal162 deleted the fix/version-switcher-styling branch November 16, 2022 16:14
@Revathyvenugopal162 Revathyvenugopal162 self-assigned this Nov 16, 2022
background-color: var(--pst-color-background);
border-color: var(--pst-color-border);
color: var(--pst-color-text-base)
background-color: #D09735!important;
Copy link
Member

Choose a reason for hiding this comment

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

Do we really want to use this color, see https://www.color-hex.com/color/d09735.

I am not sure if this is in conflict with our style guidelines. I think it works fine for the dark theme, but for the light one it may not be the best.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

No, it is not conflicting IMO, since the navigation bar is always black (in dark and in light) and it is only at the time of hovering, if you are putting the dropdown. But i am open to change.

Revathyvenugopal162 added a commit that referenced this pull request Nov 28, 2022
* Fix broken version switch styling

* Add style with specific color on active
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug Defects or glitches reported by users or developers

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants