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

Aggiunta una pagina che permette la scelta e la ricerca delle icone #1129

Merged
merged 6 commits into from
Oct 2, 2024

Conversation

Lollo176ITA
Copy link
Contributor

@Lollo176ITA Lollo176ITA commented Jun 14, 2024

Descrizione

Ho creato una pagina html che ti permette di cercare le icone che sono sul vostro sito, sono molto comode ma se qualcuno non ha installato il vostro tema oppure ha modificato il percorso file si trova in difficoltà (parlo di chi è alla prime armi o sperimenta), con questa pagina si può tranquillamente fare una ricerca, ed è tutto implementato in un solo file (so che sicuramente non rispetta nessuno standard ma almeno come idea è utile ed efficiente).
Sono abbastanza sicuro che con qualche modifica si può ben integrare.

Checklist

Copy link

vercel bot commented Jun 14, 2024

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Updated (UTC)
bootstrap-italia ✅ Ready (Inspect) Visit Preview Oct 2, 2024 2:53pm

@Lollo176ITA
Copy link
Contributor Author

Per qualsiasi dubbio chiedere pure

@astagi astagi added docs Issue e PR che riguardano la documentazione enhancement labels Jun 14, 2024
@astagi
Copy link
Member

astagi commented Jun 27, 2024

Grazie mille @Lollo176ITA ci sono alcuni test che falliscono hai indagato come mai?

@Lollo176ITA
Copy link
Contributor Author

Ciao @astagi, ho guardato gli errori e il primo riguarda il fatto che nel commit non sto usando le convenzioni (fix, chore, etc), il secondo che nel codice alla riga 66 chiudo per errore 2 volte il tag style e il terzo riguarda l'accessibilità che è penso 0

@Lollo176ITA
Copy link
Contributor Author

Lollo176ITA commented Jul 4, 2024

Per i primi 2 penso di non avere problemi a risolverli ma per il terzo non ho assolutamente idea di come fare, la mia esperienza di programmazione non si è mai trovata faccia a faccia con i requisiti di accessibilità (ovviamente ne conosco le basi ma non so che livello devo soddisfare)

@astagi
Copy link
Member

astagi commented Jul 18, 2024

Ciao @astagi, ho guardato gli errori e il primo riguarda il fatto che nel commit non sto usando le convenzioni (fix, chore, etc), il secondo che nel codice alla riga 66 chiudo per errore 2 volte il tag style e il terzo riguarda l'accessibilità che è penso 0

ciao @Lollo176ITA si dagli errori è possibile fare le correzioni magari puoi provare e ricommittare su questa PR.

Grazie mille!

@Virtute90
Copy link
Contributor

Per i primi 2 penso di non avere problemi a risolverli ma per il terzo non ho assolutamente idea di come fare, la mia esperienza di programmazione non si è mai trovata faccia a faccia con i requisiti di accessibilità (ovviamente ne conosco le basi ma non so che livello devo soddisfare)

Ti consigliere di predisporre una pagina come quella che trova negli esempi per la validazione della form:

https://github.com/italia/bootstrap-italia/blob/b7992f96b576d83668eaee9b51b44dbfb6761166/docs/esempi/form/index.html

Per non avere problemi di accessibilità dovresti usare i form di BTI.

@astagi
Copy link
Member

astagi commented Oct 1, 2024

Per i primi 2 penso di non avere problemi a risolverli ma per il terzo non ho assolutamente idea di come fare, la mia esperienza di programmazione non si è mai trovata faccia a faccia con i requisiti di accessibilità (ovviamente ne conosco le basi ma non so che livello devo soddisfare)

Ti consigliere di predisporre una pagina come quella che trova negli esempi per la validazione della form:

https://github.com/italia/bootstrap-italia/blob/b7992f96b576d83668eaee9b51b44dbfb6761166/docs/esempi/form/index.html

Per non avere problemi di accessibilità dovresti usare i form di BTI.

@Virtute90 grazie per la risposta! vista la mancata risposta dell'autore procederò io ad aggiornare il codice..

@astagi astagi merged commit db7132a into italia:main Oct 2, 2024
6 of 7 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
docs Issue e PR che riguardano la documentazione enhancement
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants