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.
Updated: Comment #0
Problem/Motivation
Search icon is not retina compatible (uses png instead of svg, unlike the toolbar).
Proposed resolution
Use the loupe-icon from libricons. This solution has also been used in #1963886: Use HiDPI icons in the toolbar.
Remaining tasks
None
User interface changes
Slight change in the search icon. Search icon now look better on retina-ready devices.
Retina:
Retina Hover:
Non-Retina:
Non-Retina Hover:
API changes
None
Comment | File | Size | Author |
---|---|---|---|
#1 | search_icon_retina-2181487-1.patch | 4.68 KB | sqndr |
#1 | search-icon-retina.png | 273 KB | sqndr |
#1 | search-icon-retina-hover.png | 272.96 KB | sqndr |
#1 | search-icon-non-retina-hover.png | 277.13 KB | sqndr |
#1 | search-icon-non-retina.png | 277.28 KB | sqndr |
Comments
Comment #1
sqndr CreditAttribution: sqndr commentedComment #2
Wim LeersJust fixing the tag :)
Comment #3
pfrenssenVery nice work. This is also related to #2142655: [policy, no patch] Agree on a technique for loading SVG images in HTML that degrades to PNG.
Comment #4
dermarioI could have a look on it for review.
Comment #5
dermarioSomething seems to wrong with the patch-file. If i apply it, it gives me the following error:
:7: trailing whitespace.
�PNG
:24: trailing whitespace.
�PNG
Checking patch core/misc/icons/424242/loupe.png...
Checking patch core/misc/icons/424242/loupe.svg...
Checking patch core/misc/icons/505050/loupe.png...
Checking patch core/misc/icons/505050/loupe.svg...
With git diff i can see a windows linebreak in loupe.pngs - declaration that wasn't there before:
Before:
After patching:
I am not sure if that is relevant at all but i will let someone else check.
Comment #6
Wim Leers#5: the "trailing whitespace" errors can safely be ignored because they occur in a PNG file, which is a binary file, where whitespace only exists if it needs to exist. If you have a Windows linebreak on your system, then my best guess is that that's a Windows-specific problem. No such problems on Linux/OS X.
Comment #7
LewisNymanComment #8
Wim LeersComment #9
Bojhan CreditAttribution: Bojhan commentedComment #10
LewisNymanCode looks good. RTBC++
Comment #11
webchickCommitted and pushed to 8.x. Thanks!
Welcome to the core team, sqndr! :D
Comment #13
triple5 CreditAttribution: triple5 commentedNice work, however, when using the bartik theme as the base for a new theme, (for example copying bartik from core to e.g. themes/custom/exampletheme) this file is not found, shouldn't this file reside in the theme folder or the bartik folder instead of core?