Buttonset button margin shouldn't be applied to the last button in the set

Can you please provide an actual page showing this instead of ASCII art?

Working on it, still figuring out how best to attach images. I have a fix ready for pull requesting as well, so my creating this ticket is mostly a formality. Right now I'm on my way home, I didn't really expect anyone to find this until I got home so I left it half done. I clearly underestimated the community here, sorry 'bout that. :)

Working on it, still figuring out how best to attach images.

Please do not provide a screenshot. Use jsFiddle or jsbin to create an actual page showing the problem.

I have a fix ready for pull requesting as well, so my creating this ticket is mostly a formality.

Glad to hear you have a pending fix :-)

Tickets are not formalities. They're important for verification, discussion, version tracking, changelogs, etc.

Right now I'm on my way home, I didn't really expect anyone to find this until I got home so I left it half done. I clearly underestimated the community here, sorry 'bout that. :)

No problem :-) I'm going to set this back to pending until we get the visual test case. Thanks.

Tickets are not formalities. They're important for verification, discussion, version tracking, changelogs, etc.

Yeah, I got that. By 'formalities' I meant RingTFM and trying to follow the instructions. ;)

Huh, for whatever reason I can't seem to reproduce the problem in a minimal example - it styles nicely without ugly hacks, unlike in the development environment where I found myself having a problem with it. I'll try to figure it out at the next opportunity, which will probably be Tuesday. Can I just leave this as pending with me as the owner until then, or what's the policy for this kind of thing?

And if I'm the one changing the status to new all the time, please let me know how to not do that when I post a comment... :x

You're not doing anything wrong. Every time you reply it'll automatically get set back to new. If you don't reply for 2 weeks, the ticket will automatically close.

I'll set back to pending for you to investigate the scenario(s) that cause this problem.

So I created a demo:

• http://f.kth.se/~emlun/jquery-ui/buttonset-fix-margin/before/
• http://f.kth.se/~emlun/jquery-ui/buttonset-fix-margin/after/

The only differences between these two are http://pastebin.com/Seiics2f . The example shows that the change has some merit, though perhaps not a great deal.

Sadly, the "after" version does not render properly in IE9 - the buttonset buttons get spaced too far apart since IE does not understand the :last-child selector. I suppose that disqualifies this change since it would break buttonsets in IE. Rearranging the :not(:last-child) and :last-child rules does not help either.

See the accepted answer at http://stackoverflow.com/questions/1293369/using-last-child-in-css . I did not realize this when opening this ticket. :)

I copied your "before" page into jsFiddle and I don't see the problem: http://jsfiddle.net/5CjAV/

Perhaps it's because you're in quirks mode.

@emiun It might be easier to use this as a starting point for your test case: http://jsfiddle.net/tj_vantoll/RKwuk/. It has the latest versions of jQuery and jQuery loaded, is in standards mode, and has some example markup already in place.

Replying to [comment:9 scott.gonzalez]:

I copied your "before" page into jsFiddle and I don't see the problem: http://jsfiddle.net/5CjAV/ Perhaps it's because you're in quirks mode.

Yeah, I've probably gotten lost in my own changes while trying to fix the original problem. After I sat down to straighten these things out for the sake of this ticket everything seems to work just fine. I guess there really isn't any problem with the theme css as it is. Whatever it was that caused the problem turned out to be easy to override after all.

Anyway, I no longer see any reason to keep pursuing the change I originally suggested. Thanks for your patience. :)