Regression: ul.links generated by theme_links() always appears as .inline

Yeah, this is a really annoying bug. I'll work on a patch.

Here's a patch :)

PS - I added examples to markup_test to test this.

Committed to CVS HEAD.

Interesting new graphical pattern, the "floating tabs".

Removing surplus padding in themes/bartik/css/style.css, #main-menu-links fixed this issue for me. Attached patch will do the needful, tested against Firefox 4 beta, chrome 7 and opera 10 for Ubuntu.

BTW, style.css has trailing white spaces (see 960490_floating_tabs_no_whitespace.patch) is it acceptable ?

With the patch in #6 the first tab is moved to the left, and looks not aligned with the rest of the page.

Shouldn't the patch be:
current: padding: 0 15px;
new: padding-left: 15px;

Either way padding 0 15px and padding-left works for me. FF-3.6.12.

The attachment (drupal7-padding.png) shows the result of the different options.
(ps: what to do, to show the image in the comment, is that possible for me?)

Sorry, didn't test this in Firefox. My mistake.

@sivaji The padding is not excess. There should be 15px on both the right and the left, so it lines up with everything else.

Here's a patch to fix it.

Ah. At last a patch that actually makes sense :)

I think the underlying problem is that in #903460: Bartik Tabs broken in Firefox, Jeff Burnz refactored the Bartik tabs to use "good old-fashioned" floats rather than display:inline, yet we've given the menu the inline class in this issue. I sent the patch through browsershots.org and the tabs are now positioned correctly.

#10 is fine with me. Works just as well.

Appears that HEAD is still broken.

Committed to CVS HEAD. Thanks.