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.
Now that we introduced the "search" module into core the block has to be cared for. I think as it is right now the text that accompanies it is somewhat duplicate information. It would be a safe assumption to make that people know how to use a search box, and can make the link if the button next to it says Search.
Therefor this patch to remove it, apart from the points mentioned above it is also better aesthetically - as it lines out better with the given block title.
Before
After
Comment | File | Size | Author |
---|---|---|---|
#4 | remove.site_.search.text_.patch | 1011 bytes | Bojhan |
#4 | removed_it.png | 2.89 KB | Bojhan |
remove.site_.search.text_.patch | 740 bytes | Bojhan | |
Before-textremoval.png | 3.79 KB | Bojhan | |
Before-textremoval.png | 3.79 KB | Bojhan |
Comments
Comment #1
Bojhan CreditAttribution: Bojhan commentedAfter patch
Comment #2
sun+1
Can we remove both, please? The button titles "Search" already... So both titles/labels look horrible IMHO.
Comment #3
sunTagging. #512464: Auto-enable a block's 'status' with a non-empty 'region' when a module is installed would really for issues like this.
Comment #4
Bojhan CreditAttribution: Bojhan commentedRemoved it, with the edit everywhere patch it probably doesn't look as nice as it would to a normal user. But no need to fix that issue here, though - lets get a good UX strategy on that issue first.
Comment #5
alexanderpas CreditAttribution: alexanderpas commentedhow about setting "Search this site" as the defualt block title? (no need for bikeshedding here, just decide.)
Comment #6
sunBojhan agreed in IRC that both titles/labels are superfluous.
Ready to fly.
Comment #7
yoroy CreditAttribution: yoroy commentedI almost created the exact same patch yesterday. yes please.
Comment #8
Dries CreditAttribution: Dries commentedCommitted to CVS HEAD. Thanks!