fix: use data to set focus instead of class #1397
Conversation
|
The latest updates on your projects. Learn more about Vercel for Git ↗︎
|
|
@astagi prossimi giorni faccio un po' di test su diversi device e browser 🤞 |
There was a problem hiding this comment.
@astagi Dai miei test sembra tutto apposto in questa semplificazione verso il solo approccio "data-" invece di "class-" per escludere il mouse.
Ho testato su:
- Windows 11 Home (con UTM Virtual Machine): Browser Edge 132
- OSX 15: Browsers Firefox 135 + Safari 18 (tutti latest)
Il focus mi sembra si comporti esattamente allo stesso modo che nell'attuale release che usa il metodo class 'focus--mouse'. Quindi dovrebbe essere corretto questo approccio di rimuovere quest'ultimo a favore del solo approccio con "data-", che può permetterci più scalabilità sui framework funzionali React e c.
Dovrebbe restare per il futuro "solo" quanto evidenziato nella #1352 più, auspicabile, documentare in un fondamento "Focus" l'approccio da aggiungere sia alla documentazione di .italia che a quella specifica di Bootstrap Italia. Lo farei una volta che saremo riusciti ad aggiornare il focus anche in UI Kit Italia.
|
@astagi ovviamente questa è una bella "Breaking" da documentare perché se qualcuno avesse usato quella classe ".focus--mouse" per qualche controllo o implementazione ulteriore di accessibilità in questo modo si romperà. Alternativa lasciare entrambi... ma anche secondo me, come dicevamo, se siamo a semplificare in ottica futuro salto a Web Components facciamolo. |
Descrizione
Fixes italia/design-react-kit#1150
Checklist