class="active" on menu's

I use pretty much the same setup as you for my site: eng & fr with Nice Menus. I have not had a problem with the class="active" going missing - it gets added each time. I didn't modify any of the modules in any way.

Just so you know, I am using drupal 4.7.2, nice_menus 1.2.2.2 2006/03/30 and i18n.module 1.30 2006/05/31.

To do my menus, I used Primary menu for the english links and created a second menu called Liens primaires for the french. Each menu is in its own block, set to display only on en/* and fr/* respectively.

If that doesn't help, the only other thing I can suggest is to check that your CSS is correctly targeting active class in both menu sets. Make sure you didn't add something specific only to the english menus to the CSS rule.

Let me know if that helps.

Here there is a rewritten l() function in contrib.inc file that should work.
I've reworked a similar patch that I've written for my localizer module.
Not tested with i18n module, but it should work.
It works regardless of what link you have used in your menu structure :
with or without a locale prefix.

Hope this helps.

function l($text, $path, $attributes = array(), $query = NULL, $fragment
= NULL, $absolute = FALSE, $html = FALSE) {

        static $valid_locales;
  $valid_locales = locale_supported_languages();
  $valid_locales = $valid_locales['name'];


        $requestq=$_REQUEST['q'];       
        $exploded_path = explode('/', $requestq);

        $localeinpath = $exploded_path[0];
        if(!(($localeinpath!= '') && array_key_exists($localeinpath,
$valid_locales))){
                $localeinpath = '';
        }

        if($localeinpath != ''){
                array_shift($exploded_path);
                $pathwithoutlocale=implode("/", $exploded_path);
        }
        else
        {
                $pathwithoutlocale=$requestq;
        }


        if($localeinpath != '')
        {
                $isactive = ($path==$_GET['q']);
                $isactive = $isactive || (drupal_get_normal_path($pathwithoutlocale)==
$path);

                foreach ($valid_locales as $lang) {
                        $isactive = $isactive || (drupal_get_normal_path($lang . '/' .
$pathwithoutlocale)==$path);                    
                }

        }
        else
        {
                $isactive = ($path == $_GET['q']);
        }

  if ($isactive) {
    if (isset($attributes['class'])) {
      $attributes['class'] .= ' active';
    }
    else {
      $attributes['class'] = 'active';
    }
  }
  return '<a href="'. check_url(url($path, $query, $fragment,
$absolute)) .'"'. drupal_attributes($attributes) .'>'. ($html ? $text :
check_plain($text)) .'</a>';
}

This is closely related to http://drupal.org/node/80820 so please follow up there.

Roberto's suggestion will work -Thanks

I'm not including any core patches in i18n though.