Style classes were available only for switch by hostname, I included for else also.

In if($setcurrentactive && $lang==localizer_get_uilocale()) removed $setcurrentactive because it was always FALSE for me so class currentuilocale wasn't shown.
Maybe it is needed for something else but I can't figure out what for.

I have also removed active in class because I think that if someone will want a different style than it will create at least a.currentuilocale and maybe a.uilocale.

I don't like a.active class because it changes color of a link in default (Garland) theme.

It could be a good idea to remove link (tekst stays but it is not link) for current locale but it's too much messing with the code for my skill level.

CommentFileSizeAuthor
#1 localizer_module_1.patch1.85 KBWuk
localizer_module_0.patch1.84 KBWuk

Comments

Wuk’s picture

Wuk commented
StatusFileSize
new localizer_module_1.patch1.85 KB

I just find out (I looked more carefully) that all menus are using a.active so it's better to have active as backup.

Here is patch with a.active included.

Wuk’s picture

Wuk commented

This one is correct. In previous post I manually changed patch file and got errors.

Roberto Gerola’s picture

Roberto Gerola commented
Status: Needs review » Fixed

Thanks.
Integrated in the latest cvs version of localizer.module file

Anonymous’s picture

(not verified) commented
Status: Fixed » Closed (fixed)