Opened 10 years ago
Closed 9 years ago
#35496 closed defect (bug) (fixed)
Quick Edit error messages UI improvements
| Reported by: | afercia | Owned by: | afercia |
|---|---|---|---|
| Priority: | normal | Milestone: | 4.9 |
| Component: | Quick/Bulk Edit | Version: | |
| Severity: | normal | Keywords: | has-screenshots has-ui-feedback has-patch |
| Cc: | Focuses: | ui |
Attachments (6)
Change History (17)
#3
@
10 years ago
I think having them red is good. I almost want full on error messages though with boxes and everything. They merge a little too much for me just as text. However, I do wonder if making them an error box will be overkill.
I wonder a little on positioning, the message seems to float without connecting to an action in a few of those screenshots. Could we use this as a chance to bring the message to the action also?
#4
@
10 years ago
+1 for a regularly styled error message, even if it does feel a little overkill. Attaching some ideas.
#5
@
10 years ago
+1 to that @melchoyce - looks so much better with a box and error message that way.
#7
@
9 years ago
- Keywords has-patch added; needs-patch removed
- Milestone Awaiting Review → 4.9
- Owner set to
- Status new → assigned
35496.diff changes the error messages to use inline notices. Also adds some missing periods to some error messages. A few screenshots:
#8
@
9 years ago
One note, we'll want to use the .notice-alt class here since the notice is on a white background.
#9
@
9 years ago
Actually, the notices are on a zebra striped background, depending on the line the background is white or light gray. Patch updated with the notice-alt style. Screenshot:
One thing I personally don't like so much about the notice-alt style is that it lacks the bottom box-shadow, making the notice look "flat". any specific reason for this?
![(please configure the [header_logo] section in trac.ini)](/chrome/site/your_project_logo.png)







Yep, just prepared a screenshot for Quick Edit and Reply errors for comments :)