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

[H2 headers in all Gatsby websites on mobile] AutolinkHeader-module--anchor adds "empty row" in a multitude of occasions #1256

Open
DragosRistici opened this issue May 2, 2022 · 1 comment
Labels
role: dev 🤖 severity: 3 Affects minor functionality, has a workaround type: bug 🐛

Comments

@DragosRistici
Copy link

The anchor that preceds the H2 headings on desktop goes AFTER the heading on mobile with an opacity:0, which means its still there. This creates a lot of random spacing problems on a lot of H2 headings on mobile, on every Gatsby website in the Carbon environment.
Screenshot 2022-05-02 at 12 36 25

carbon.heading.issue.mov
second.proof.mov

Screenshot 2022-04-30 at 10 30 17

If you just add a display:none for the mobile, that would be great. You don't need it on mobile anyway.

Screenshot 2022-05-02 at 12 29 52

@DragosRistici
Copy link
Author

@alisonjoseph @mattrosno

Hey Alison & Matt,

I looked into this requested by @mjabbink who caught it this past Friday. Thx!

@abbeyhrt abbeyhrt transferred this issue from carbon-design-system/carbon-website Jul 25, 2022
@sstrubberg sstrubberg added severity: 3 Affects minor functionality, has a workaround role: dev 🤖 and removed status: needs triage 🕵️‍♀️ labels Jul 11, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
role: dev 🤖 severity: 3 Affects minor functionality, has a workaround type: bug 🐛
Projects
Status: ⏱ Backlog
Development

No branches or pull requests

3 participants