Opened 11 years ago

Closed 11 years ago

Last modified 11 years ago

#8255 closed bug (notabug)

jquery ui tabs : tab selected is missing classes applied and classes removed.

Reported by: tadudek Owned by:
Priority: minor Milestone: 1.9.0
Component: ui.tabs Version: git (not yet released)
Keywords: Cc:
Blocked by: Blocking:

Description

It seems that with the milestone version of (using m7) that the selected tab list item incorrectly has classes applied:

http://jsfiddle.net/tadudek/pkbcP/6/ 1.64

Selected Tab
class="ui-corner-top ui-tabs-selected ui-state-active"

ui-corner-top  //appeares in both
ui-tabs-selected
ui-state-active //appeares in both

http://jsfiddle.net/tadudek/pkbcP/11/ 1.9m7

Selected Tab
class="ui-state-default ui-corner-top ui-tabs-active ui-state-active"

ui-state-default //looks like it's not removed
ui-corner-top //appeares in both
ui-tabs-active 
ui-state-active //appeares in both

MISSING CLASS :ui-tabs-selected
not-selected tabs for both versions tested
class="ui-corner-top ui-state-default"

This causes the bottom of the selected tab to still look detached. I had to manually copy in the entire 1.9m7 release into the jsfiddle javascript block because I didn't see anyone hosting this version.

Hope this helps!

Change History (6)

comment:1 Changed 11 years ago by Scott González

Resolution: invalid
Status: newclosed

It's not valid to use a 1.8.x theme with 1.9 JavaScript.

comment:2 in reply to:  1 Changed 11 years ago by tadudek

Replying to scott.gonzalez:

It's not valid to use a 1.8.x theme with 1.9 JavaScript.

ouch, not backwards compatible? Is there a themeroller for 1.9 yet?

comment:3 Changed 11 years ago by Scott González

No ThemeRoller yet. Themes are never backward compatible; we don't even support using a 1.8.17 theme with 1.8.18 JavaScript.

comment:4 in reply to:  3 Changed 11 years ago by tadudek

Replying to scott.gonzalez:

No ThemeRoller yet. Themes are never backward compatible; we don't even support using a 1.8.17 theme with 1.8.18 JavaScript.

Well, thought I had something there for ya, but thanks. I appreciate the quick reply Scott.

As a note: I had to move to 1.9 to get the tab bug fix for: http://bugs.jqueryui.com/ticket/5069. But can't use the fixed js with earlier css versions; which in my case is customized. So. I appreciate the help, you've been on top of my questions all day, but I guess I'll have to wait for the full release, or manually fix (i.e. create workaround) the previous stable version to deal with the tab building problem. Again, thanks for the help and direction.

comment:5 Changed 11 years ago by Scott González

The theme-related settings are all in jquery.ui.theme.css, so if you just use your existing one with the new jquery.ui.tabs.css (and any other widget css), it *should* work. No promises though :-)

Here's a diff of the themes directory from 1-8-stable to master: https://gist.github.com/2370542

comment:6 in reply to:  5 Changed 11 years ago by tadudek

Replying to scott.gonzalez:

The theme-related settings are all in jquery.ui.theme.css, so if you just use your existing one with the new jquery.ui.tabs.css (and any other widget css), it *should* work. No promises though :-)

Here's a diff of the themes directory from 1-8-stable to master: https://gist.github.com/2370542

Scott, just wanted to follow up (not sure if there is a better place than on your ticketing system), I took your idea, and from building a simple mockup example it works great. I'm not sure how it'd affect everything in all cases, but again, I'm just mocking some use case examples up for my offshore team. So Thanks. All I did was:

<link type="text/css" href="${pageContext.request.contextPath}/tmihome/styles/jquery_ui.css/custom-theme/jquery-ui-1.8.16.custom.css" rel="stylesheet" />

<link type="text/css" href="${pageContext.request.contextPath}/tmihome/styles/jquery_ui.css/1.9m7/base/jquery.ui.tabs.css" rel="stylesheet" />

and bam.. looks good. Much appreciated.

Note: See TracTickets for help on using tickets.