Support for Drupal 7 is ending on 5 January 2025—it’s time to migrate to Drupal 10! Learn about the many benefits of Drupal 10 and find migration tools in our resource center.Problem/Motivation
In the prior issue we added attribute filtering, but didn't update some of the UI text.
Proposed resolution
Update the text to indicate that attributes are filtered also
Remaining tasks
Patch.
Review.
User interface changes
The format configuration UI text.
API changes
None.
Data model changes
none
| Comment | File | Size | Author |
|---|---|---|---|
| #6 | 2579357-6.patch | 1.2 KB | pwolanin |
| #2 | Screen Shot 2015-10-02 at 4.30.45 PM.png | 148.67 KB | pwolanin |
| #2 | 2579357-2.patch | 1.2 KB | pwolanin |
Comments
Comment #2
pwolanin CreditAttribution: pwolanin as a volunteer and at Acquia commented1st pass at a patch.
Here's how the UI looks:
Comment #6
pwolanin CreditAttribution: pwolanin as a volunteer and at Acquia commentedNo idea why it didn't apply. Here's a re-roll.
Comment #8
pwolanin CreditAttribution: pwolanin as a volunteer and at Acquia commentedoh, I am an idiot
Comment #10
pwolanin CreditAttribution: pwolanin as a volunteer and at Acquia commentedadding tag since this changes a t() string
Comment #11
droplet CreditAttribution: droplet commentednote that there's some info is hidden due to the bug
Comment #12
Wim LeersLooks sensible :)
Comment #13
Wim LeersComment #14
Fabianx CreditAttribution: Fabianx as a volunteer and at Tag1 Consulting commentedComment #15
webchickOh dear lord that's an overwhelming amount of text. :\
I mean. I guess it's "ok" in that it's a uid-1 facing page. Still, assigning for UX review. If no better suggestion is made in the next ~24 hours I'll go ahead and commit it and we can tweak it in subsequent minor releases.
Comment #16
alexpottLooks good. Committed 43a7d10 and pushed to 8.0.x. Thanks!