Come together with the global Drupal community in Rotterdam, 28 Sept – 1 Oct 2026. Sessions, contribution, connection, and Early Bird savings until 8 June.theme_languageicons_icon() > Please use that function, to support cdn.
| Comment | File | Size | Author |
|---|---|---|---|
| #5 | 969722-04-use_theme_image_to_render_icons-d7.patch | 1.57 KB | Freso |
| #2 | 969722-use_theme_image_to_render_icons-02-d7.patch | 1.53 KB | Freso |
| #1 | 969722-use_theme_image_to_render_icons-01-d6.patch | 1.51 KB | Freso |
Comments
Comment #1
Freso commentedI just tried implementing this feature request on 6.x-2.x. Please try the attached patch and report back whether it works. (I haven't tried it out meself.)
Comment #2
Freso commentedAnd the patch for 7.x-1.x.
Comment #3
thehong commentedPatch look good, please commit to stable branch first.
Comment #4
Freso commentedNo, patches always go to HEAD first. Also, did you try/test the patch or just review it? I'll need the former before I'll apply it. (It's okay if you only test 6.x-2.x though, as 7.x-1.x isn't fully working at the moment.)
Comment #5
Freso commentedThe patch didn't work, but this one does, and was thus committed. :)
Now I just need either a proper review of the patch for 6.x-2.x, or some time to test it myself. (My focus is mainly on getting Language icons ready for D7, so I cannot say when I'll have time for that right now.)
Comment #6
Freso commentedAnd finally patched up the D6 version and committed it to the 6.x-2.x branch.