italia / design-comuni-pagine-statiche

Le pagine statiche che compongono il modello di sito web per i comuni
https://italia.github.io/design-comuni-pagine-statiche
BSD 3-Clause "New" or "Revised" License
20 stars 25 forks source link

Componente header - ente di appartenenza #34

Closed AlessandroVecchi closed 1 year ago

AlessandroVecchi commented 2 years ago

Ciao, nella header c'è questo pezzo di codice

<a class="d-lg-block navbar-brand" target="_blank" href="#" aria-label="Vai al portale {Nome della Regione} - link esterno - apertura nuova scheda" title="Vai al portale {Nome della Regione}">Nome della Regione</a>

a mio parere, in questo caso, è superfluo l'utilizzo del aria-label e sarebbe meglio strutturare il codice in questo modo

<a class="d-lg-block navbar-brand" target="_blank" href="#" rel="noopener">Nome della Regione <span class="visually-hidden">link esterno - apertura nuova scheda</span></a>

come del resto è già usato, ad esempio, per la lingua o per i social. il title non lo vedo così indispensabile, secondo me si può anche omettere. al limite si può aggiungere vai al portale prima del nome della regione.

Da notare l'aggiunta di rel="noopener" che per il link esterni è consigliabile.

stale[bot] commented 1 year ago

Questo problema è stato automaticamente contrassegnato come non aggiornato perché non ha avuto attività recentemente. Sarà chiuso se non si verificano ulteriori attività. Considerate inoltre l'aggiornamento alle ultime versioni del tema. Grazie.