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

Navbar links CSS fault for mobile view (light mode) #1262

Open
jollydodo opened this issue Jul 29, 2024 · 1 comment
Open

Navbar links CSS fault for mobile view (light mode) #1262

jollydodo opened this issue Jul 29, 2024 · 1 comment

Comments

@jollydodo
Copy link

jollydodo commented Jul 29, 2024

Description

Minor issue that I just noticed (it's very subtle). When in light mode and mobile view (collapsed navbar), the search and expand icon don't get the correct color assigned. This has to do with the btn-link class which is overwritten by the offcanvas class in expanded mode, but not in mobile mode which results in added transparency that looks quite odd (in dark mode it does work as expected).

Wrong color (getdoks website):
Scherm­afbeelding 2024-07-29 om 20 02 47

My own website has an extra icon where it is more noticeable:
Scherm­afbeelding 2024-07-29 om 20 07 15

Expected (full view):
Scherm­afbeelding 2024-07-29 om 20 03 45

Mobile view light mode wrong color:
Scherm­afbeelding 2024-07-29 om 20 03 09

Correct color in normal widescreen view:
Scherm­afbeelding 2024-07-29 om 20 03 41

I've now fixed it with some simple custom css, but it would off course be nicer for everyone if this was fixed in the Doks repo.

Steps to reproduce

Simply resize your browser from normal view to mobile view in light mode and look for the subtle discrepancy.

Expected result

The same class as in darkmode/normal view should be assigned so that the buttons simply get the $primary color assigned.

Actual result

Wrong color + transparency is added

Environment

Tested it in my own environment, but it is also visible / reproducible on the getdoks website.

@h-enk
Copy link
Member

h-enk commented Jul 31, 2024

Good catch!

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

2 participants