-
Notifications
You must be signed in to change notification settings - Fork 11
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
592 assistant add keyboard navigation control #622
Conversation
c7c01da
to
5efabb8
Compare
a5e740d
to
9f5dee1
Compare
7da75a5
to
8309fe7
Compare
8309fe7
to
a1bb8ab
Compare
b8c03b5
to
ff0b848
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Top comme modifs, ça en faisait pas mal. Je suis pas fan des méthodes qui retournent pleins d'objets différents. tu as dû faire des extract d'IDE pour plus de lisibilité, mais les méthodes sont pas oufs au niveau des paramètres passés et des retours, lol si on veut clarifier, il faudra adapter l'algo. On le fera plus tard.
packages/docs/docs/03-recommendations/02-improving-accessibility.md
Outdated
Show resolved
Hide resolved
En effet, là je suis allé au plus simple en faisant un extract avec l'ide mais je pense que certains champs retournés ne sont même pas utiles, je vais vois comment refacto simplement ca |
ff0b848
to
0f3a3ab
Compare
No description provided.