Opened 10 years ago
Closed 10 years ago
#31241 closed defect (bug) (fixed)
Taxonomy error messages show success colors
Reported by: | ninnypants | Owned by: | ocean90 |
---|---|---|---|
Milestone: | 4.2 | Priority: | normal |
Severity: | normal | Version: | 3.7 |
Component: | Taxonomy | Keywords: | good-first-bug has-patch commit |
Focuses: | ui | Cc: |
Description
When attempting to update a taxonomy fails the message is shown with a green update border instead of the red error border.
Steps to reproduce:
- Create two tags
- Edit the second tag
- Attempt to change the slug of the second tag to be the same as the first
After hitting update you will be take to the list of all tags and the message "Tag not updated." Will show with a green border.
Attachments (6)
Change History (19)
#1
@
10 years ago
- Milestone changed from Awaiting Review to 4.2
- Summary changed from Taxonomy error messages show succes colors to Taxonomy error messages show success colors
This ticket was mentioned in Slack in #core by ninnypants. View the logs.
10 years ago
#5
in reply to:
↑ 4
@
10 years ago
Replying to mzak:
31241.diff wont cover all other error instances. Maybe try using keys from the $messages
array further up in that file.
This ticket was mentioned in Slack in #core by ninnypants. View the logs.
10 years ago
#8
@
10 years ago
The messages can be filtered, and there are four different notice types (success, warning, error, info) - what could we do to support custom messages and varying notice types? That is probably a bigger picture thing, and applies to messages in other places, but I want us to be thinking about that.
Added conditional to check what message is returned, and set the class accordingly