If there are too many comments it's hard to read them all, and sometimes it can be disastrous to miss an comment. It would be very useful for QA, RM and RMaints to see easily if there is something they should not miss on a bug report.
Created attachment 145236 [details] [review] Add red background to important comments
Created attachment 145237 [details] [review] Bug 32614: Highlight important comments This patch is suggesting to have an 'important' tag that could be attach to comments, and that would highlight them. * There is a new note at the top right "There are 2 important comments in this bug!" * The 'important' tag is red * The comment flagged with the 'important' tag will have a red background for better visilibity
This is how it would look like: https://snipboard.io/UrsuAl.jpg
From: Your Name <you@example.com>
Created attachment 145253 [details] [review] Bug 32614: Highlight important comments This patch is suggesting to have an 'important' tag that could be attach to comments, and that would highlight them. * There is a new note at the top right "There are 2 important comments in this bug!" * The 'important' tag is red * The comment flagged with the 'important' tag will have a red background for better visilibity
Created attachment 145269 [details] [review] Bug 32614: Highlight important comments This patch is suggesting to have an 'important' tag that could be attach to comments, and that would highlight them. * There is a new note at the top right "There are 2 important comments in this bug!" * The 'important' tag is red * The comment flagged with the 'important' tag will have a red border for better visilibity
This is how it looks like with the last patch: https://snipboard.io/KqBHWP.jpg Replaced background-color with border-color.
Patch is applied to test on bugzilla, but it doesn't seem to be doing anything Chris
Did you remove the assets? rm data/assets/*.js data/assets/*.css
Yep its made new ones -rw-r--r-- 1 www-data www-data 273640 Jan 15 23:12 01d1b8200ec3d6070e858c3636a8f000.js So not quite sure whats up.
For what it's worth, it seems to be working for me?
Yes, it's working for me as well! :-)
Looks good to me too. Marking as "RESOLVED FIXED". Please reopen if I missed something.