Skip to content

HTTPS clone URL

Subversion checkout URL

You can clone with HTTPS or Subversion.

Download ZIP

Loading…

vertical controlgroup inside collapsible set display issue #2905

Closed
beloitdavisja opened this Issue · 9 comments

5 participants

@beloitdavisja

Example:
http://jsfiddle.net/BGAN5/

All the controlgroup buttons correctly display except for the first button of the last collapsible group (in above example, Menu Item 1 under Menu 5). jQM is adding the ui-corner-bottom class erroneously to this element.

@toddparker

Can you use the latest, not RC2 in your test page. Template here: http://jsbin.com/akaxah/6/edit

@beloitdavisja

Apologies...first bug submission.

Interesting though - When I switch out RC2 for the latest, looks like it isn't getting the js/css files...

http://jsfiddle.net/BGAN5/2/

I used the latest on my scribble page locally, and the problem still exists.

@toddparker

I don't like how you can't see the head in fiddle. Here is your code using latest. The issue is still there:
http://jsbin.com/ahuput
http://jsbin.com/ahuput/edit

@beloitdavisja

Ah thanks. Unfamiliar with using jsbin. Was hitting the Render button and getting an undefined page. At least now you can see what I'm seeing.

@MauriceG

So I'm not the only one who see that undefined page... But what have you done that it's work?

@toddparker hacking the hashes has no success

@toddparker

Shut off the preview and remove the hash that they add and it works great.

@Madhuka

even in horizontal, this issue exist.

@MauriceG MauriceG referenced this issue from a commit in MauriceG/jquery-mobile
@MauriceG MauriceG addresses issue #2905 controlgroups in collapsible-set: leave child-a…
…nchors in collapsible-sets untouched; removing triple l option name; avoiding duplicate setting ui-corner-bottom for last collapsible-header in set
0f1edce
@gseguin gseguin referenced this issue from a commit
Commit has since been removed from the repository and is no longer available.
@gseguin gseguin referenced this issue from a commit
Commit has since been removed from the repository and is no longer available.
@gseguin gseguin referenced this issue from a commit in gseguin/jquery-mobile
@gseguin gseguin Decoupled collapsible and collapsible-set
That fixes issue #2905 as well
6d4274f
@agcolom
Collaborator

@toddparker This looks fixed to me. The 1st button in last collapsible looks fine now.

@toddparker

Closing as fixed. Thanks @agcolom

@toddparker toddparker closed this
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Something went wrong with that request. Please try again.