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

Fix #13076: MatMenu component was broken since Angular Material upgrade #1960

Merged
merged 1 commit into from
Jul 24, 2024

Conversation

marob
Copy link
Contributor

@marob marob commented Jul 23, 2024

Description

Le code de center-matmenu.directive.ts (110 lignes de code) provient visiblement de https://stackoverflow.com/a/54134809/499994 et ne servait qu'à centrer le menu....

Le centrage a été remplacé par 2 lignes de CSS.

Quelques modifications dans le code HTML/TS pour garder le bouton/menu au rollover d'une autre ligne et pour bien gérer la coche/décoche de la checkbox du menu (l'event change n'étant visiblement pas déclenché suffisamment tôt pour s'effectuer avant que le menu ne soit détruit, on utilise click à la place)

Type de changement

  • Correction

Checklist

Sélectionner les éléments de la checklist

  • Mon code suit le style de code de ce projet.
  • J'ai commenté mon code, en particulier dans les classes et les méthodes difficile à comprendre.
  • J'ai fait les changements correspondant dans la documentation RAML.
  • J'ai fait les changements correspondant dans la documentation Métier.
  • J'ai fait les changements correspondant dans la documentation Technique.
  • J'ai rajouté les tests unitaires vérifiant mes fonctionnalités.
  • J'ai rajouté les tests de non régression vérifiant mes fonctionnalités.
  • Les tests unitaires nouveaux et existants passent avec succès localement.
  • Toutes les dépendances ont été mergées en priorité

Contributeur

  • VAS (Vitam Accessible en Service)

@GiooDev GiooDev added this to the IT 139 milestone Jul 23, 2024
@marob marob changed the title Fix 13076: MatMenu component was broken since Angular Material upgrade Fix #13076: MatMenu component was broken since Angular Material upgrade Jul 23, 2024
@marob marob merged commit 3482747 into develop Jul 24, 2024
7 of 8 checks passed
@marob marob deleted the fix_13076 branch July 24, 2024 08:39
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants