Convert theme functions to use drupal_attributes()

Nice start Jacine! Subscribing.

1.

+  //print_r($link); die;

2.

+function theme_menu_tree($tree, $attributes = array()) {
+function theme_menu_local_tasks($attributes = array()) {

do we need the new $attributes parameter?

3.

-function theme_menu_local_action($link) {
-  return '<li>' . $link . "</li>\n";
+function theme_menu_local_action($link = array()) {
+  $link['attributes'] = array();
+  return '<li' . drupal_attributes($link['attributes']) . '>' . $link['value'] . "</li>\n";

there is no improvement here. I guess the $link['attributes'] = array(); line should be changed to

if (!isset($link['attributes'])) {
  $link['attributes'] = array();
}

or

  $link['attributes'][] = 'something';

4. in theme_menu_local_tasks() as I see the secondary local tasks will have a "tabs primary secondary" class after the patch..

subscribing. good initiative. We're already doing something similar in the RDF patches.

What are the plans to deal with the tpl files? Let themer alter the array of CSS classes before drupal_attributes?

menu.inc-attributes.patch queued for re-testing.