#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 follow-up: 2 Changed 11 years ago by
Resolution: | → invalid |
---|---|
Status: | new → closed |
comment:2 Changed 11 years ago by
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 follow-up: 4 Changed 11 years ago by
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 Changed 11 years ago by
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 follow-up: 6 Changed 11 years ago by
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 Changed 11 years ago by
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.
It's not valid to use a 1.8.x theme with 1.9 JavaScript.