Page MenuHomePhabricator

jquery UI buttonset in Vector Skin does not show up as a "set"
Closed, ResolvedPublic

Assigned To
Authored By
Rillke
Nov 28 2011, 5:58 PM
Referenced Files
F8549: buttonset.png
Nov 22 2014, 12:03 AM
F8548: buttonset.png
Nov 22 2014, 12:03 AM
F8546: buttonset.png
Nov 22 2014, 12:03 AM
F8547: buttonset.png
Nov 22 2014, 12:03 AM

Description

It should look like http://jqueryui.com/demos/button/#radio .

Using monobook theme it looks fine. In vector, however it looks like singe buttons.

To test it, you can insert the following html:

<div class="demo">
<form>
<div id="radio">

		<input type="radio" id="radio1" name="radio" /><label for="radio1">Choice 1</label>
		<input type="radio" id="radio2" name="radio" checked="checked" /><label for="radio2">Choice 2</label>
		<input type="radio" id="radio3" name="radio" /><label for="radio3">Choice 3</label>

</div>
</form>
</div><!-- End demo -->

And the followng JS:

mw.loader.using([
'jquery.ui.button'
], function(){ $( "#radio" ).buttonset(); });


Version: unspecified
Severity: minor

Details

Reference
bz32687
TitleReferenceAuthorSource BranchDest Branch
Add temporary debug output for T346876repos/phabricator/phabricator!16aklapperT346876-debugwmf/stable
Customize query in GitLab

Event Timeline

bzimport raised the priority of this task from to Low.Nov 22 2014, 12:03 AM
bzimport set Reference to bz32687.

Maybe fixed now. But it might be also just a race condition how CSS is loaded and applied.

Created attachment 12876
Screenshot of the "buttonset" in Firefox 22

Only "fixed" if you have no space between the <option>s because they are display:inline-block -- otherwise a tiny space remains.

Since padding of the content of the button is 1em, it is save to set the margin from -0.3em to -0.4em

Attached:

buttonset.png (338×522 px, 16 KB)

Created attachment 12877
Screenshot of the "buttonset" in Opera/9.80 Presto/2.12.388 Version/12.15 (Core 2.12.388)

Attached:

buttonset.png (339×504 px, 16 KB)

Created attachment 12878
Screenshot of the "buttonset" in IE 10

Attached:

buttonset.png (333×488 px, 17 KB)

Created attachment 12880
Screenshot of the "buttonset" in Chromium 27

Attached:

buttonset.png (332×519 px, 36 KB)

Change 74328 had a related patch set uploaded by Rillke:
Removing space between buttons in Buttonset in Vector

https://gerrit.wikimedia.org/r/74328

Change 74328 merged by jenkins-bot:
Removing space between buttons in Buttonset in Vector

https://gerrit.wikimedia.org/r/74328