search

backdrop / backdrop-issues

Issue tracker for Backdrop core.
144 stars 38 forks source link

[META][UX] Start using the new 'info' message type. #3312

Open klonos opened 5 years ago

klonos commented 5 years ago

In #3155 where this feature was implemented, we have identified some success (status) messages that would be better as info message instead. This issue if a follow-up to convert these, and perhaps other ones in core.

PR by @klonos: https://github.com/backdrop/backdrop/pull/2318

klonos commented 5 years ago

...ok, initial PR up for review. I have only worked on messages on the Project Installer.

klonos commented 5 years ago

...I just filed another PR, in which I explore the option of adding some of the Project Installer messages in the Form level. The idea basically is to have messages with problems/info related to the form be closer to the thing they refer to, and to separate them from any possible "global" messages. Here's a before:

screen shot 2018-10-21 at 8 13 11 am

...and an after:

screen shot 2018-10-21 at 8 16 13 am

PS: shamelessly plugging #3329

klonos commented 5 years ago

...I am making this a meta-issue.