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.
After update a module the menu icons disappear.
Comment | File | Size | Author |
---|---|---|---|
#6 | fix_icons_not_showing-1399516-6.patch | 546 bytes | ewills |
Comments
Comment #1
Luna Vulpo CreditAttribution: Luna Vulpo commentedIf thing that is because in menu_icons.css in background-image is "public://" instead real url
Comment #2
acrollet CreditAttribution: acrollet commentedHrmm,
I'm having trouble reproducing this issue - can you clear your caches and let me know if it remains the same? If so, please go to admin/config/media/file-system and let me know your Public file system setting?
thanks much.
Comment #3
Luna Vulpo CreditAttribution: Luna Vulpo commentedI cleared cache bud it doesn't help.
I have in admin/config/media/file-system "sites/default/files"
I installed a dev version - same effect.
I workaround a problem adding in css file lines like this:
where xxx is nuber.
Comment #4
acrollet CreditAttribution: acrollet commentedWhat is the content of your file_public_path variable? (in the variables table, or use the command 'drush vget file_public_path')
Comment #5
thecandide CreditAttribution: thecandide commentedI have the same issue. My file_public_path variable is "s:19:"sites/default/files";" which should be correct. The workaround also fixes the issue for me. Could it be a problem with the the "public://"?
When I replace "public://" in "background-image: url(/public://styles/menu_icon/public/menu_icons/menu_icon_1040.jpg);" with "background-image: url(/sites/default/files/styles/menu_icon/public/menu_icons/menu_icon_1040.jpg);" it works fine.
Thank you for your time.
Comment #6
ewills CreditAttribution: ewills commentedYeah I have this issue too - the attached patch should fix it. Nice and easy - the public:// url just needed to be converted to a "proper" path using file_create_url()
Comment #7
acrollet CreditAttribution: acrollet commentedSee #1407716: change to file API or rely on imagecache for the preferred resolution to this issue.
Comment #8
Hanno CreditAttribution: Hanno commentedWorks for me as well. Re-opened this as this patch seems a good temporary solution. For sure we need to rethink the code as in #1407716: change to file API or rely on imagecache
Comment #9
acrollet CreditAttribution: acrollet commentedCommited patch in #6 as part of http://drupalcode.org/project/menu_icons.git/commit/2a87da8, thanks!