Active
Project:
Language Icons
Version:
7.x-1.x-dev
Component:
Code
Priority:
Major
Category:
Bug report
Assigned:
Unassigned
Reporter:
Created:
22 Dec 2010 at 11:54 UTC
Updated:
16 Dec 2022 at 07:14 UTC
Jump to comment: Most recent, Most recent file
Comments
Comment #1
klonos...coming from #904948: No icons are showing
Comment #2
Freso commentedAlright, please test this patch. It hides (and disables) the options altogether, and ignores them in the processing of language links as well. Could quite possibly use better wording - suggestions are most welcome.
I don't think this issue can have a happy ending until #1009360: Make locale_block_view return a renderable array instead of HTML has been resolved.
Comment #3
Freso commentedUpdated the patch a bit. I'll probably commit this in the morrow unless outrage ensues before then.
Comment #4
klonosSorry Frederik, this got out of my radar because you said you'd commit it next day, so I said I'd just wait to test it when the next dev was available. You didn't commit though, so I didn't test ;)
Anyways..., the patch does fix the effect of #1067766: img tag messed-up for icon images in Node links (both title & alt attributes contain a nested img tag instead of text), but it also introduces the disabling of the option to disable node links which I need. If others have this need too, then you can use my temporary fix/workaround in post #3 over at #1067766: img tag messed-up for icon images in Node links (both title & alt attributes contain a nested img tag instead of text).
This is trivial (I understand that it is merely cosmetic), but may I also report that the
'#collapsed' => TRUE,attribute in the 'Add language icons' fieldset has no effect (at least in firefox), since the fieldset is always presented expanded.Lastly, may I ask which is this bug (issue # or link please) you mention that that cannot currently be resolved?
Comment #5
Freso commentedThe bug that this is waiting on is referred/linked to in comment #2.
But the whole point of this issue is that, as it is, Drupal 7 doesn't care. There is no difference between node's language links and the language switcher block's language links. Thus, the options only serve to give a false impression of what's possible with the module.
That's a bug that should be filed against Drupal's core (possibly jQuery), then. (It worked fine in Chromium, when I tested there.)
Comment #6
klonosThanx for pointing out the core bug. I am heading over there so I can help with testing etc. As for the '#collapsed' => TRUE bug, I'll test some more because it might be a browser specific thing as you suggest (I'm only seeing it in latest firefox 4 x64 nightlies ATM), then file a core bug for it if it actually checks out.
Comment #7
Freso commentedI just pushed my commit with the patch previously posted, only now with a link here.
Postponing until #1009360: Make locale_block_view return a renderable array instead of HTML is fixed, or another work-around shows itself.
Comment #8
batigol commentedhttp://drupal.org/node/1009360#comment-5459878 is working ?
Comment #9
klonosYeah, there's a patch for D8 that can be manually applied to D7 too. Could we give that a try?
Comment #10
Freso commentedAs long as the patch isn't committed, there's no reason in "giving it a try". Its form is not final until it's been added to core.
Comment #11
klonosYeah, I meant to only add my comment as a reference to that issue and the patch so we can give it a try once it got fixed. While I was at it, I also switched status to active because I thought you'd want to give it a try sooner than waiting for it to be committed (you know how long it sometimes takes for core patches to get in). If it did work, then instead of me saying that this *might* solve another issue we'd post a comment saying that it actually does solve it and perhaps this way we'd also "push" for the fix to get in sooner ;)
Comment #12
jddeli commentedI can not check or uncheck the options
Add language icons
Link types to add language icons.
options Node links
options Language switcher block
These options are currently disabled due to a bug that cannot currently be resolved. They may be reintroduced at a later stage.
Comment #13
klonosΕ πατρίδα!...
- Μην αλλάζεις το πεδίο "Assigned" εκτός και αν πρόκειται να δουλέψεις ο ίδιος πάνω στον κώδικα με σκοπό να κάνεις upload ένα αρχείο .patch.
- Μην αλλάζεις το πεδίο "Version". Ότι διορθώνεται πάει πρώτα στην πιο πρόσφατη dev έκδοση και έπειτα (όταν έχουν μαζευτεί αρκετές διορθώσεις) βγαίνει μια καινούρια έκδοση.
- Το συγκεκριμένο issue έχει τεθεί σε κατάσταση αναμονής (Status: "postponed"), με βάση το γεγονός ότι δεν μπορούμε να λύσουμε το πρόβλημα #1009360: Make locale_block_view return a renderable array instead of HTML. Για περισσότερα, διάβασε παραπάνω στο #7. Είδη πρότεινα στον maintainer του module να δοκιμάσουμε κάτι με το δοκιμαστικό patch που υπάρχει για το θέμα που σου αναφέρω και δεν δέχτηκε να το κάνουμε έως ότου το πρόβλημα έχει λυθεί πλήρως σε μια επόμενη έκδοση του Drupal 7 (για να γίνει αυτό, θα πρέπει πρώτα το πραγματικό πρόβλημα να έχει λυθεί στο Drupal 8). Οπότε μην πειράζεις ούτε το πεδίο "Status".
Βλέπω ότι είσαι μέλος εδώ και πάνω από 3 χρόνια (περισσότερο από 'μένα). Θα 'πρεπε να γνωρίζεις ότι αν συνεχίσεις να αλλάζεις τα issues απλά επειδή έχεις αναλάβει ένα project το οποίο σε επείγει, το μόνο που θα καταφέρεις είναι να τσατίσεις τους πάντες και θα σταματήσουν να σου απαντούν (στην καλύτερη των περιπτώσεων). Ήδη ρισκάρω να με "επιπλήξουν" που σου απαντώ στα Ελληνικά εδώ (δεν γνωρίζω αν επιτρέπεται άλλη γλώσσα εκτός από Αγγλικά), αλλά βλέπω ότι σου είπαν τα ίδια πράγματα που σου λέω και εγώ αλλού και δεν πρέπει να το κατάλαβες.
Αυτάαα...
Comment #14
Josias commentedYour language icons block can be enabled in your page.tpl.php with:
Drupal 7
Drupal 6
Comment #15
Con An commentedis there anything new in this issue?
i tried Josias solution. but nothing..
do i have to do something else or just use the code above?
--ε! πατρίδα!..
Comment #16
e.ku commented@Freso , i'm beginner in using drupal and when i'v tried to use two language on my website the error message appears says that the icon language is disable due to bug ..... .
would you tell me please how can i use that patch file that you mention above ? and is it solve the problems or not yet???
thanks
Comment #17
klonos@Freso: Hey Frederik! I guess you never got the time to test things as you said in #1009360-11: Make locale_block_view return a renderable array instead of HTML. Well, there's a new patch available now. Care to take a look?
Comment #18
susheel_c commentedAny updates here? Seems like a simple enough thing to fix (but I'm no programmer)... can you let us know when it'll be pushed into a release of the mod?
Comment #20
nujood commentedComment #21
klonos...please do not assign the issue to yourself unless you are going to work on it (provide a patch).
@Freso: Hey Frederik! My question in #17 still stands ;)
Comment #22
mike-michal commentedHello everybody!
In Theme function theme_languageicons_icon($variables) change returned value to: return theme('image', $image);
It worked for me. But why the returned value was only language iso name? Don't f***n know :)
Comment #23
sami.etogo commentedHello "Ozyrys"! I am new in Drupal. Where can I find the "theme_languageicons_icon($variables)"? Thank you!
Comment #24
joelpittet@sami.etogo Here's a decent explanation:
https://www.drupal.org/node/457740
Comment #25
mariocantor commentedThis solution works for me, maybe is not the best, but works..
1. go to http://www.flag-sprites.com/
2. Configure your sprite (flags you need)
3. download Css and Flags.png. Move the flags.png to your files directory
4.Create a Blog and make table using html and add the flags
5. add the flag classes to your css (provide by the site http://www.flag-sprites.com/), you must change the flags.png url.
6. now you have your own flags language block...
this is block example that I created
-I use US and UAE flags for English an Arabic
Comment #27
anybodyAs this is now fixed for D8 this patch is marked as needs backport to D7.
Comment #28
dravenkwith core 8.5.3, the bug still exists.

Comment #29
prophet108 commentedI updated this module (on an old D7 site) and now i don't have language switcher icons anymore and only see "These options are currently disabled due to a bug that cannot currently be resolved. They may be reintroduced at a later stage." Is there a patch for the D7 version yet? Thank you.
Comment #30
graham leach commentedHello,
Here's my solution. Added to the bottom of theme.css
Comment #31
anybody