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.

Sign up for GitHub

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

Jump to bottom

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

Closed
1 task done
zetareticoli opened this issue Oct 18, 2024 · 2 comments · Fixed by #1300
Closed
1 task done

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

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

Comments

@zetareticoli
Copy link
Member

zetareticoli commented Oct 18, 2024 •
edited
Loading

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
@zetareticoli zetareticoli mentioned this issue Oct 18, 2024
Merged
5 tasks
@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 mentioned this issue Dec 10, 2024
Merged
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
@github-project-automation github-project-automation bot moved this from 👀 In review to ✅ Done in Design system .italia 🇮‍🇹 Jan 14, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@zetareticoli zetareticoli

Labels
bug Issue e PR su problemi di funzionamento della libreria
Projects
Design system .italia 🇮‍🇹
Status: ✅ Done
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

fix: link list active state italia/bootstrap-italia
2 participants
@zetareticoli @Fupete