-
Notifications
You must be signed in to change notification settings - Fork 0
feat: componente accordion #111
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
base: main
Are you sure you want to change the base?
Conversation
…n and focus styles, gained full a11y
🦋 Changeset detectedLatest commit: 4c0fa2c The changes in this PR will be included in the next version bump. Not sure what this means? Click here to learn what changesets are. Click here if you're a maintainer who wants to add another changeset to this PR |
The latest updates on your projects. Learn more about Vercel for GitHub.
|
@deodorhunter https://design-web-components-git-ac-a23c68-dip-trasformazione-digitale.vercel.app/?path=/docs/componenti-accordion--documentazione#accordion-item questa story è normale che non abbia stili quell'accordion item? |
Voluto, non è dentro un'accordion, non può avere i suoi stili (viene specificato che è da usare solo dentro it-accordion). Viene mostrato solo a fini di documentazione, possiamo anche rimuoverlo se crea confusione.
Ci sto guardando ora, temo devo riscrivere gli stili di bsi anche qui perchè i selettori sono incompatibili. Dovrebbe essere solo il font (arial vs titillium), size e weight sono analoghi a bsi da inspector |
Solo una cosa rimasta ad una prima review:
|
Come detto facciamo un approfondimento su temi di:
|
@zetareticoli @Fupete @astagi ho pushato tutto ciò che vi ho mostrato ieri in call e tutta la documentazione ed esempi aggiornati, questa PR è pronta per un altro giro di review. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Oltre ai due suggerimenti di intestazioni, non ho ben capito questa proprieta:
as: Il livello di heading (h2-h6) da utilizzare per l'header (opzionale, default: h2)
Nell'accordion item non la vedo applicata, mi sono perso qualcosa?
|
||
<!-- Header attivi --> | ||
<section> | ||
<h2>Header attivi</h2> |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
<h2>Header attivi</h2> | |
<h2>Stato attivo</h2> |
|
||
<!-- Hover degli Header --> | ||
<section> | ||
<h2>Hover degli Header</h2> |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
<h2>Hover degli Header</h2> | |
<h2>Stato hover</h2> |
Implementa il componente accordion, #40
PR Checklist
main
branch.Short description of what this resolves:
Implementato il componente accordion
Changes proposed in this Pull Request:
Implementato il componente accordion