A new look for system messages was proposed in https://groups.drupal.org/node/283223#Messages and was introduced in https://drupal.org/node/1986408.
However the current implementation does not match the original design and comes with a border join which is simple ugly.
While we should have a thicker block (border) on the left currently we have a ragged line instead becouse of the different border colors and the the way browsers render the joins of borders.
We could get a nice / clean look in two ways:
- Change our mind on the expected look of messages and use the same color for all four sides of the border.
- Adding some extra css markup to have what was originally designed.
Personally I vote to the second solution and do it if there is not something (important) against it.
Comment | File | Size | Author |
---|---|---|---|
#4 | 2095929-system-messages-border-4.patch | 2.48 KB | longwave |
#1 | d8-messages--error-reworked.png | 41.14 KB | thamas |
#1 | drupal-system-messages-border-fix-2095929-1.patch | 2.48 KB | thamas |
d8-messages--error.png | 42.91 KB | thamas |
Comments
Comment #1
thamasHere is a patch with a simple box-shadow trick. It is not perfect 'cause the border radius is applied to both sides of the shadow. But it is not so noticeable like the current implementation and needed just some additional lines of code. I feel I could live with it. ;)
If we want a "pixel-perfect" solution :) (without plus rounds) we have to use a :before pseudo-element with a bit more lines of css code.
Comment #2
LewisNyman CreditAttribution: LewisNyman commentedComment #3
LewisNyman CreditAttribution: LewisNyman commentedThis looks a lot better. Nice catch.
We don't need to have 'px' units after 0. That's all I could find.
Comment #4
longwaveLooks better to me too. Attached patch changes 0px to 0 as suggested.
Comment #5
LewisNyman CreditAttribution: LewisNyman commentedFantastic
Comment #6
thamasThanks for the review and the correction! I should pay attention to what I do… :)
Comment #7
webchickCommitted and pushed to 8.x. Thanks!