Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Coherent topbar ux styling #2177

Closed
BorghildSelle opened this issue Mar 7, 2024 · 7 comments
Closed

Coherent topbar ux styling #2177

BorghildSelle opened this issue Mar 7, 2024 · 7 comments
Assignees

Comments

@BorghildSelle
Copy link
Contributor

BorghildSelle commented Mar 7, 2024

image

The groups doesn't seem to me "together"
Each has its own height, width, hover and focus styling.

Maybe the languages could be in a select dropdown?

@BorghildSelle
Copy link
Contributor Author

image

Maybe something like this for languages?

@NickHaggerty1
Copy link

Hry @BorghildSelle No I dont want to change this

@BorghildSelle
Copy link
Contributor Author

@NickHaggerty1 No to just changing the language or no to change the hover/focus styling and language?

@NickHaggerty1
Copy link

@BorghildSelle No to the dropdown for language select. We only have max two languages and on the global site some pages are only one. I see this as an additional click for users and harder to see immediately whats available.

With the change to the search button is fine to align the hover/focus styling across the three options.

@BorghildSelle
Copy link
Contributor Author

@NickHaggerty1 I agree, when there are so few options as you say :)

@BorghildSelle
Copy link
Contributor Author

@NickHaggerty1 Do you want to look at a common hover/focus/active styling for these or should we make a suggestion?

@NickHaggerty1
Copy link

@BorghildSelle great if you can suggest something. Dont have a strong opinion on this.

@BorghildSelle BorghildSelle self-assigned this May 7, 2024
millianapia pushed a commit that referenced this issue May 13, 2024
* 🎨 update

* 🎨 update focus styling

* 🎨 update

* 🎨 update mobile

* 🎨 update

* 🎨 update styling

* 🎨 update

* 🎨 darker tint on divider
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

No branches or pull requests

3 participants