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

Rimuovere lo stato active per i links di tipo lista semplice #1255

Open
1 task done
zetareticoli opened this issue Oct 18, 2024 · 2 comments · May be fixed by #1300
Open
1 task done

Rimuovere lo stato active per i links di tipo lista semplice #1255

zetareticoli opened this issue Oct 18, 2024 · 2 comments · May be fixed by #1300
Assignees
Labels
bug Issue e PR su problemi di funzionamento della libreria

Comments

@zetareticoli
Copy link
Member

zetareticoli commented Oct 18, 2024

Esiste già una discussione sul tema che ti interessa, o su un tema simile?

  • Ho verificato e non esiste

Versione della libreria

v2.10.0

Cosa

I links presenti nelle liste semplici prevedono in documentazione uno stato active che però non dovrebbe esistere in quanto appunto semplici liste che potrebbero far parte del body di una pagina.

Perché

Lo stato active è fuorviante perché appartiene solo alle liste di link presenti in menu di primo o secondo livello.

Contesto

No response

Altro

Verficare interferenze con il componente List group di Bootstrap.

@zetareticoli zetareticoli added the bug Issue e PR su problemi di funzionamento della libreria label Oct 18, 2024
@zetareticoli zetareticoli self-assigned this Oct 18, 2024
@Fupete
Copy link
Contributor

Fupete commented Oct 19, 2024

Secondo me essendo Bootstrap Italia in origine un tema Bootstrap, le ns liste corrispondono ai "loro" List group più che alle liste nei contenuti di testo. Da questo l'esistenza di tutti gli stati.

Quindi, 1) bisogna stare attenti a non far sì che rimuovendo stile sass poi le classi Bootstrap prendano il sopravvento, e lo stato .active alla fine esista comunque ma non stilato come dovrebbe e 2) in caso documentare meglio la pagina liste questo sì.

@zetareticoli
Copy link
Member Author

Il componente List group segue un markup completamente diverso dalle nostre liste

Bootstrap:

<ul class="list-group">
  <li class="list-group-item">An item</li>
  <li class="list-group-item">A second item</li>
  <li class="list-group-item">A third item</li>
  <li class="list-group-item">A fourth item</li>
  <li class="list-group-item">And a fifth one</li>
</ul>

Bootstrap Italia:

<div class="it-list-wrapper">
  <ul class="it-list">
    <li>
      <div class="list-item">
        <div class="it-right-zone"><span class="text">Testo</span>
        </div>
      </div>
    </li>
    <li>
      <a href="#" class="list-item">
        <div class="it-right-zone"><span class="text">Link</span>
        </div>
      </a>
    </li>
    <li>
      <a class="list-item active" href="#">
        <div class="it-right-zone"><span class="text">Link attivo</span>
        </div>
      </a>
    </li>
  </ul>
</div>

Possiamo lasciare lo stile dello stato active ma specificare in documentazione che non serve per i link nei contenuti.

@zetareticoli zetareticoli moved this from 🆕 New to 📋 Backlog in Design system .italia 🇮‍🇹 Oct 21, 2024
@zetareticoli zetareticoli linked a pull request Dec 10, 2024 that will close this issue
5 tasks
@zetareticoli zetareticoli moved this from 📋 Backlog to 🔖 Ready in Design system .italia 🇮‍🇹 Dec 10, 2024
@zetareticoli zetareticoli moved this from 🔖 Ready to 👀 In review in Design system .italia 🇮‍🇹 Dec 10, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Issue e PR su problemi di funzionamento della libreria
Projects
Status: 👀 In review
Development

Successfully merging a pull request may close this issue.

2 participants