By asiafan on

Hi, I'd like to use this theme but has one little annoying thing, that the language switcher lists flags and language names too in separate lines, so having 2 languages is a list of four, and looks bad as if there were 4 lang.s. All the other themes display the flag+name in one line, not in two.. I checked the paragraph containing this part and its the same in all other themes, and have only two "li" elements and only a space between a flag and country name, so I dont understand why Friendselectric makes a four line bulleted list when theres only 2 list items in the code. I dont know where to find this error, so any help is appreciated, thanks.

the code of the lang. switcher starts with: <div class="content"><div class="item-list"><ul><li><span class="i18n-link"><a href="/en/user/1/edit" class="active"><img src="/modules/i18n/flags/en.png" class="i18n-icon" width="16" height="12" alt="English" /></a>&nbsp;<a href="/en/user/1/edit" class="active">English</a></span></li> <li> "same for other language" </li>

in the FE css theres
.item-list ul li a, li.expanded a, li.collapsed a, li.leaf a {
margin: 0;
padding: 3px 1px 3px 20px;
display: block;
}

which makes those (ugly double) linebreaks i think, but I still have no idea why the word "english" is a new list item????

⋅ Categories: Drupal 4.7.x

Comments

wonderland’s picture

wonderland commented

Hi,

I had the same problem. It actually is a issue in the friendselectric theme (also see http://drupal.org/node/27211). Therefore it should be fixed in the theme, but as I don't know enough about css and themes to do that, I fixed it in the output of the i18n module.

In the end of i18n.inc file, you will find a function theme_i18n_link with the following lines:

 $output .= l(theme('i18n_language_icon', $lang), $target, $attributes, NULL, NULL, FALSE, TRUE);
 $output .= $separator;
 $output .= l($text, $target, $attributes, NULL, NULL, FALSE, TRUE);

Just change these lines to

 $output .= l(theme('i18n_language_icon', $lang).$separator.$text, $target, $attributes, NULL, NULL, FALSE, TRUE);
  //$output .= $separator;
  //$output .= l($text, $target, $attributes, NULL, NULL, FALSE, TRUE);

If anyone knows a proper fix for the theme, please let us know.

..- Hendrik