Problem/Motivation
When working on a View that has many displays with overridden sections, it can be extremely difficult to notice / remember which sections are overridden on which displays. The current style changes for an overridden section is to apply an italic font style to that section. With the default font, an italic style is way too subtle to be noticed easily.
Proposed resolution
We should highlight the entire section so that the overridden section is much more obvious at a glance. For color-blind accessibility reasons, changing the background color would not be sufficient alone. Instead we should supply both a color and a symbol.
User interface changes
In this patch I've added a colored border around the overridden section. Since the affected element does not currently have a border, the new border can act as the symbol which denotes overridden, and the color is helpful to draw the user focus to that area.
This patch adjusts both the views_ui module css, as well as the Stable theme views_ui css since the Stable theme itself overrides the views_ui css. If it is inappropriate for a patch to affect both components, please let me know. I'm happy to submit multiple patches, or otherwise adjust as needed.
Example of change:
Greyscale:
Comment | File | Size | Author |
---|---|---|---|
#17 | after_patch.png | 57.38 KB | smustgrave |
#17 | before_patch.png | 56.93 KB | smustgrave |
#14 | diff_reroll_2908862_3-14.txt | 1.3 KB | ankithashetty |
#14 | 2908862-14.patch | 1.07 KB | ankithashetty |
views-admin-css.png | 117.22 KB | daggerhart |
Comments
Comment #2
daggerhart CreditAttribution: daggerhart commentedComment #3
daggerhart CreditAttribution: daggerhart commentedNot sure why I never uploaded this patch. Here it is for all you composers out there.
Comment #11
kerasai CreditAttribution: kerasai at Kerasai commentedLooks great.
Thanks for your contribution!
Comment #12
lauriiiThis would be great issue to discuss during one of our UX calls.
Comment #13
quietone CreditAttribution: quietone as a volunteer commentedThe patch does not apply to Drupal 9.3.x.
Comment #14
ankithashettyRe-rolled the patch in #3, thanks!
Comment #17
smustgrave CreditAttribution: smustgrave at Mobomo commentedTested on 9.5 and seems to work fine.
Comment #18
alexpottThe usability review hinted at in #12 has not taken place yet. And the patch does not apply to 10.x since stable has been removed.
I'd also like to see how this looks on a view with lots and lots of things overridden. I have a feeling this will get very very heavy.