-
Notifications
You must be signed in to change notification settings - Fork 24
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
Retirer les boutons appliquer dans les filtres #3898
Conversation
Je me demande vraiment si on ne devrait pas tout simplement dégager ces boutons. Le site a besoin de JS pour le menu, qui permet de se logger et d’accéder à ces vues... |
JavaScript is required in order to login, stop pretending pages can be accessed without JavaScript.
e1c8f6d
to
6be3537
Compare
Oui, c'est l'étape d'après 😅 Après, garder ce bouton ne coûte pas très cher non plus 🤷♂️ |
Je viens de passer à l’étape d’après. Ce bouton est toujours là est parce qu’il ne coûte presque rien, mais comme le site requiert JavaScript de toute façon, il ne sert à rien. |
Je viens de demander l’avis des autres devs car je propose d’ancrer le fait que le site nécessite JavaScript. Nous n’avons pas d’objectif à ce que le site fonctionne sans JS (même si je suis sympatisant de cette idée), et dans les faits, le site requiert JS. |
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.
Effectivement, le site n'étant pas vraiment utilisable sans JS autant simplifier tout ça
Par contre je n'ai pas vu de <noscript>
donc peut-être ajouter une bannière pour prévenir l'utilisateur qui serait dans ce cas là ?
Ces boutons sont présents dans l’espoir que les utilisateurs sans JS puissent utiliser le site.
Le JavaScript est nécessaire pour que le site fonctionne. Autant le reconnaître et l’assumer. Autrement, on pourrait croire que les utilisateurs sans JS peuvent utiliser le site, alors qu’ils n’ont pas accès au menu déroulant d’authentification, ou au module de recherche d’emploi.