cisstech / platon

Platform for Learning and Teaching Online
Other
14 stars 3 forks source link

Dialog service delay implementation #17

Closed Ofghanirre closed 1 year ago

Ofghanirre commented 1 year ago

Je propose une modification qui consiste à intégrer un délai par défaut de disparition pour les alertes et messages fournies par le DialogService du projet

Ce fix vient du fait qu'on est rapidement spammé de notifications lors d'édition de ressource, point trouvé inconfortable par les utilisateurs actuels:

image

À l'image de la lib NGZorro, j'ai établis le délai moyen à 4500ms

https://ng.ant.design/components/notification/en

codecov[bot] commented 1 year ago

Codecov Report

Merging #17 (abbea0d) into main (0ba89be) will not change coverage. The diff coverage is n/a.

@@           Coverage Diff           @@
##             main      #17   +/-   ##
=======================================
  Coverage   89.13%   89.13%           
=======================================
  Files           2        2           
  Lines          46       46           
  Branches       11       11           
=======================================
  Hits           41       41           
  Misses          1        1           
  Partials        4        4           

:mega: We’re building smart automated test selection to slash your CI/CD build times. Learn more

mciissee commented 1 year ago

Good job ✅ Il faut juste faire attention au format des commits en suivant la description ici https://github.com/cisstech/platon/blob/main/CONTRIBUTING.md. Ca permet de faciliter la génération du CHANGELOG

Ofghanirre commented 1 year ago

Ok je regarde ça tout de suite!