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

fix: content h2 margin top #1443

Draft
wants to merge 2 commits into
base: main
Choose a base branch
from
Draft

fix: content h2 margin top #1443

wants to merge 2 commits into from

Conversation

Fupete
Copy link
Collaborator

@Fupete Fupete commented Oct 23, 2024

No description provided.

@Fupete Fupete self-assigned this Oct 23, 2024
@Fupete Fupete marked this pull request as draft October 23, 2024 07:44
Copy link
Contributor

github-actions bot commented Oct 23, 2024

⚡ Lighthouse report for the home page 🏠

Category Score
🔴 Performance 49
🟢 Accessibility 100
🟢 Best practices 100
🟢 SEO 100

See the full report...

Other pages

Copy link
Member

@zetareticoli zetareticoli left a comment

Choose a reason for hiding this comment

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

@Fupete non sarebbe meglio gestirlo con questa issue di BI?

@Fupete
Copy link
Collaborator Author

Fupete commented Oct 23, 2024

Questa però è molto site-specific "locale" intanto, serve a uniformare uso h2 nei campi markdown ## titolo h2 a uso componente textImageCTA sotto con il suo h2 e il suo spacing.

Vedi per esempio news lancio design system.

Quello da fare è controllare se facesse saltare altro in qualche contenuto, non ho ancora avuto tempo di guardare le preview.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
Status: 🛠 Doing
Development

Successfully merging this pull request may close these issues.

2 participants