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

Retirer les boutons appliquer dans les filtres #3898

Merged
merged 1 commit into from
Apr 25, 2024
Merged

Conversation

francoisfreitag
Copy link
Contributor

@francoisfreitag francoisfreitag commented Apr 8, 2024

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.

@francoisfreitag francoisfreitag self-assigned this Apr 8, 2024
@francoisfreitag francoisfreitag added the no-changelog Ne doit pas figurer dans le journal des changements. label Apr 8, 2024
@francoisfreitag
Copy link
Contributor Author

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.
@xavfernandez
Copy link
Contributor

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...

Oui, c'est l'étape d'après 😅

Après, garder ce bouton ne coûte pas très cher non plus 🤷‍♂️

@francoisfreitag
Copy link
Contributor Author

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.

@francoisfreitag francoisfreitag requested a review from a team April 8, 2024 15:49
@francoisfreitag
Copy link
Contributor Author

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.
Si c’est OK pour vous, autant s’épargner les faux-semblants.

@francoisfreitag francoisfreitag changed the title Simplifier le JS pour cacher les boutons appliquer dans les filtres Retirer les boutons appliquer dans les filtres Apr 11, 2024
Copy link
Contributor

@rsebille rsebille left a 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à ?

@francoisfreitag francoisfreitag added this pull request to the merge queue Apr 25, 2024
Merged via the queue into master with commit 4a8d4c3 Apr 25, 2024
10 checks passed
@francoisfreitag francoisfreitag deleted the ff/rm-js branch April 25, 2024 10:17
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
no-changelog Ne doit pas figurer dans le journal des changements.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants