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

ui: notification #258

Merged
merged 1 commit into from
Apr 30, 2019
Merged

ui: notification #258

merged 1 commit into from
Apr 30, 2019

Conversation

Garfield-fr
Copy link
Contributor

Signed-off-by: Bertrand Zuchuat [email protected]

@pronguen
Copy link
Contributor

pronguen commented Apr 17, 2019

Fonctionnement ok!

Contenu

  • Il faudrait retravailler les textes. Les messages sont structurés en 2 parties, à voir si c'est bien nécessaire, et si oui, harmoniser le titre, c'est-à-dire la première section du message (ex: lors de la validation d'une demande, nous avons "Search L'exemplaire est en transit", alors qu'il n'y a pas de rapport avec la recherche). A première vue, ce titre ne semble pas indispensable. Est-il possible de le supprimer?
  • Il manque des traductions de messages (interface patron).
  • Couleurs: le texte de la 2e section du message apparaît en vert contrairement à la première section. Peu esthétique. Je propose dans un premier temps d'avoir tous les messages en noir/gris sur blanc. Si l'on souhaite utiliser des couleurs, faire que l'entier du message (fond + texte) soit en vert, orange, bleu ou rouge (cf. alertes bootstrap):
    alertes bootstrap

@Garfield-fr
Copy link
Contributor Author

Pour implémenter ces notifications, nous utilisons le system "Toast" de Bootstrap:
https://getbootstrap.com/docs/4.3/components/toasts/

  • Il faudrait analyser les choses, mais il n'est pas facile de reproduire les couleurs comme sur les alertes, car cela n'est pas prévu de base.
  • Si on veut supprimer le titre, on peut le remplacer par un espace, mais il y aura toujours cette structure.
  • Je pense que les traductions n'ont pas été récupérées sur transifex (valider la traduction complète de ui)

@iGormilhit
Copy link

@pronguen @Garfield-fr

Pour la couleur, est-ce que ça pourrait être dans la pastille carrée qu'il y a dans les exemples du toast bootstrap ?

Pour la structure, il faudrait trouver en effet trouver le moyen que chaque élément soit pertinent...

@Garfield-fr
Copy link
Contributor Author

Le dialogue a été modifié:

  • Suppression de l'entête
  • Couleur de fond en fonction des types + texte en blanc

* NEW Implements a new notification system (like Toast boostrap).
* FIX closes rero#232

Signed-off-by: Bertrand Zuchuat <[email protected]>
@BadrAly BadrAly merged commit 168dcd6 into rero:dev Apr 30, 2019
@Garfield-fr Garfield-fr deleted the flash-message branch June 28, 2019 09:12
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