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

Link to site homepage is not underlined on hover in the local navigation bar #627

Open
ndiego opened this issue Jun 17, 2024 · 3 comments
Open
Labels

Comments

@ndiego
Copy link
Member

ndiego commented Jun 17, 2024

I noticed this today when testing the Theme Directory refresh, but this impacts all newly redesigned sites with the local navigation bar.

Unlike the links on the right, the main link back to the site homepage is not underlined on hover, which makes it hard to tell that it's actually a link. Here's an example in Developer Resources.

local-nav-link.mp4

cc @ryelle @adamwoodnz

@ryelle
Copy link
Contributor

ryelle commented Jun 17, 2024

I think this has to do with GB's restructuring of CSS specificity for styles generated from theme.json. The generated CSS rule for hover :root :where(.wp-block-site-title a:where(:not(.wp-element-button)):hover) is no longer overriding the default general a:hover provided by the parent theme.

Maybe we can move this bit of CSS to a rule in theme.json to ensure the styles cascade correctly?

This issue should probably be moved to the parent theme repo, the underlying issue might impact other places we invert link text-decoration.

@ndiego
Copy link
Member Author

ndiego commented Jun 17, 2024

Something like this should work.

"elements": {
    "link": {
        "typography": {
            "textDecoration": "none"
        },
        ":hover": {
            "typography": {
                "textDecoration": "underline"
            }
        }
    }
}
@ndiego
Copy link
Member Author

ndiego commented Jun 17, 2024

This issue should probably be moved to the parent theme repo, the underlying issue might impact other places we invert link text-decoration.

I'll move it over.

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