Skip to content

Conversation

deodorhunter
Copy link
Contributor

Implementa il componente accordion, #40

PR Checklist

  • My branch is up-to-date with the Upstream main branch.
  • The unit tests pass locally with my changes (if applicable).
  • I have added tests that prove my fix is effective or that my feature works (if applicable).
  • I have added necessary documentation (if appropriate).

Short description of what this resolves:

Implementato il componente accordion

Changes proposed in this Pull Request:

Implementato il componente accordion

Copy link

changeset-bot bot commented Sep 16, 2025

🦋 Changeset detected

Latest 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

@deodorhunter deodorhunter self-assigned this Sep 16, 2025
Copy link

vercel bot commented Sep 16, 2025

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Preview Updated (UTC)
design-web-components Ready Ready Preview Oct 13, 2025 1:00pm

@deodorhunter deodorhunter changed the title Accordion test Aggiunto componente accordion Sep 16, 2025
@giuliaghisini
Copy link
Contributor

@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?
l'accordion-button mi sembra abbia il font troppo bold e il family non settato

@deodorhunter
Copy link
Contributor Author

@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.

l'accordion-button mi sembra abbia il font troppo bold e il family non settato

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

@pnicolli
Copy link
Member

Solo una cosa rimasta ad una prima review:

  • le frecce sono invertite

@pnicolli pnicolli linked an issue Sep 19, 2025 that may be closed by this pull request
18 tasks
@zetareticoli zetareticoli changed the title Aggiunto componente accordion feat: componente accordion Sep 22, 2025
@Fupete
Copy link
Contributor

Fupete commented Sep 22, 2025

Come detto facciamo un approfondimento su temi di:

  • Responsabilità della libreria per la mitigazione dei problemi di Layout Shift e FOUC che abbiamo definito nel documento tecnico
  • Tema che alcuni strumenti di a11y non percepiscono gli heading seppur riconosciuti correttamente nei lettori di schermo principali

@deodorhunter deodorhunter changed the title feat: componente accordion Aggiunto componente accordion Sep 26, 2025
@pnicolli pnicolli linked an issue Oct 9, 2025 that may be closed by this pull request
18 tasks
@pnicolli
Copy link
Member

pnicolli commented Oct 9, 2025

@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.

@zetareticoli zetareticoli changed the title Aggiunto componente accordion feat: componente accordion Oct 9, 2025
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.

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>
Copy link
Member

Choose a reason for hiding this comment

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

Suggested change
<h2>Header attivi</h2>
<h2>Stato attivo</h2>


<!-- Hover degli Header -->
<section>
<h2>Hover degli Header</h2>
Copy link
Member

Choose a reason for hiding this comment

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

Suggested change
<h2>Hover degli Header</h2>
<h2>Stato hover</h2>

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

Successfully merging this pull request may close these issues.

feat: componente accordion feat: componente it-video

5 participants