#9584 closed bug (fixed)
Tabs: Refresh issue when tabs are moved to bottom
Reported by: | SledgehammerPL | Owned by: | SledgehammerPL |
---|---|---|---|
Priority: | minor | Milestone: | 1.10.4 |
Component: | ui.tabs | Version: | 1.10.3 |
Keywords: | Cc: | ||
Blocked by: | Blocking: |
Description
Browsers: IE and Chrome (other not tested)
OS: XP, Win8 (other not tested)
How to reproduce bug:
It's hard to me to show it on jsFiddle, because the problem is with ajax loaded content.
The base is here : http://jsfiddle.net/Sledgehammer/hCdRy/9/
The idea of the problem: tab-index.html contains only this:
<ul> <li><a href="deny.html">FOOBAR</a></li> </ul> <span style="cursor:pointer" onclick="$.addTab();">PRESS HERE TO ADD TAB</span>
After loading page almost looks ok. But firing addTab() by clicking PRESS HERE TO ADD TAB should just add new Tab. But it makes strange new Tab (without) borders, and changes <li> FOOBAR</li> from first tab into a new tablist and new tab
if you want, you can see the problem here: http://www.kredyty.brzezina.pl/debug/
Change History (9)
comment:1 follow-up: 2 Changed 9 years ago by
Owner: | set to SledgehammerPL |
---|---|
Status: | new → pending |
comment:2 follow-up: 3 Changed 9 years ago by
Status: | pending → new |
---|
Replying to scott.gonzalez:
There's no reason that you need to use ajax to show whatever bug you're describing. Just inject the markup directly. You should be able to reduce this quite a bit.
- yes, you're right, I reduced two functions not connected to the problem.
- I'm not sure what you mean "inject the markup".
comment:3 follow-up: 4 Changed 9 years ago by
Status: | new → pending |
---|
Replying to SledgehammerPL:
- I'm not sure what you mean "inject the markup".
Don't rely on remote files, for example - http://jsfiddle.net/tj_vantoll/ZjgbK/
comment:4 Changed 9 years ago by
Status: | pending → new |
---|
Replying to tj.vantoll:
Replying to SledgehammerPL:
- I'm not sure what you mean "inject the markup".
Don't rely on remote files, for example - http://jsfiddle.net/tj_vantoll/ZjgbK/
Ok, this is not exactly the same, but let's try to fix this
http://jsfiddle.net/Sledgehammer/hCdRy/14/
Expected result is simply add new tab and when clicked on new tab -> shows new content loaded from external link
But unexpectedly previous content (contained <li> tags) is lost... And press RIGHT TAB 1 after adding NEW TAB - it contains another tablist
comment:5 Changed 9 years ago by
Status: | new → open |
---|---|
Summary: | Jquery Tabs creates unwanted tabs from external contents → Tabs: Refresh issue when tabs are moved to bottom |
Here's a reduced test case - http://jsfiddle.net/tj_vantoll/maC8B/. Something about the <ul><li><a href>
structure on the other tab is throwing this off.
The tabs also have to be on the bottom for this issue to happen as well. I'll have to dig into this more before determining if this is something we should support, but it seems like a legit problem so I'll mark it valid for now.
comment:6 Changed 9 years ago by
Resolution: | → fixed |
---|---|
Status: | open → closed |
Tabs: Keep reference to existing tablist during refresh. Fixes #9584 - Tabs: Refresh issue when tabs are moved to bottom.
Changeset: e14f75ed480e5b036bb47ab3398d1e0df28a128a
comment:7 Changed 9 years ago by
Milestone: | none → 1.11.0 |
---|
comment:8 Changed 9 years ago by
Tabs: Keep reference to existing tablist during refresh. Fixes #9584 - Tabs: Refresh issue when tabs are moved to bottom. (cherry picked from commit e14f75ed480e5b036bb47ab3398d1e0df28a128a)
Changeset: 8eefd2032fb5c0e60c7a9899ba51bc7f33aaebc8
comment:9 Changed 9 years ago by
Milestone: | 1.11.0 → 1.10.4 |
---|
There's no reason that you need to use ajax to show whatever bug you're describing. Just inject the markup directly. You should be able to reduce this quite a bit.