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

Supprime du CSS dédié à une très ancienne version du menu responsive #5033

Merged

Conversation

francois-ferrandis
Copy link
Contributor

La classe .arrow-none n'est plus utilisée nulle part.

Aussi, il me semble probable qu'il y ait eu un .dropdown-menu à l'intérieur du #menu-agent à une époque, mais visiblement ça n'est plus le cas.

@francois-ferrandis francois-ferrandis self-assigned this Jan 30, 2025
Copy link
Contributor

@adipasquale adipasquale left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

bien vu merci @francois-ferrandis !

on peut éventuellement inliner ces 8 lignes restantes de CSS dans le fichier parent application_agent.scss ? je me demande si elles sont vraiment utiles d’ailleurs.

@francois-ferrandis
Copy link
Contributor Author

bien vu merci @francois-ferrandis !

on peut éventuellement inliner ces 8 lignes restantes de CSS dans le fichier parent application_agent.scss ? je me demande si elles sont vraiment utiles d’ailleurs.

Elles sont bel et bien utiles, notamment pour le dropdown de changement de status ajouté dans #4882 pour le cursor, et l'ombre est pas moche à mes yeux, et ça semble compatible avec les recommandations du DSFR.

Et pour déplacer en vrac dans le fichier racine, je n'y vois pas tant d'avantage, et les fichiers avec du CSS en vrac ça m'angoisse. 😸

@francois-ferrandis francois-ferrandis enabled auto-merge (squash) February 3, 2025 10:28
@francois-ferrandis francois-ferrandis merged commit 80d861c into production Feb 3, 2025
15 checks passed
@francois-ferrandis francois-ferrandis deleted the frf/delete-old-css-for-responsive-menu branch February 3, 2025 10:32
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Status: ✅ Done
Development

Successfully merging this pull request may close these issues.

2 participants