Last modified: 2013-07-18 12:01:30 UTC

Wikimedia Bugzilla is closed!

Wikimedia migrated from Bugzilla to Phabricator. Bug reports are handled in Wikimedia Phabricator.
This static website is read-only and for historical purposes. It is not possible to log in and except for displaying bug reports and their history, links might be broken. See T50593, the corresponding Phabricator task for complete and up-to-date bug report information.
Bug 48593 - Icon-only-buttons broken in vector
Icon-only-buttons broken in vector
Status: RESOLVED FIXED
Product: MediaWiki
Classification: Unclassified
Interface (Other open bugs)
1.22.0
All All
: Normal minor (vote)
: ---
Assigned To: Rainer Rillke @commons.wikimedia
:
Depends on:
Blocks:
  Show dependency treegraph
 
Reported: 2013-05-17 22:30 UTC by Rainer Rillke @commons.wikimedia
Modified: 2013-07-18 12:01 UTC (History)
3 users (show)

See Also:
Web browser: ---
Mobile Platform: ---
Assignee Huggle Beta Tester: ---


Attachments
rendering of broken vector button (996 bytes, image/png)
2013-05-17 22:31 UTC, Rainer Rillke @commons.wikimedia
Details
expected result (1.04 KB, image/png)
2013-05-17 22:32 UTC, Rainer Rillke @commons.wikimedia
Details
new proposed content of resources/jquery.ui/themes/vector/jquery.ui.button.css (15.69 KB, text/css)
2013-05-17 22:45 UTC, Rainer Rillke @commons.wikimedia
Details
Patch to apply against resources/jquery.ui/themes/vector/jquery.ui.button.css (633 bytes, patch)
2013-05-17 22:45 UTC, Rainer Rillke @commons.wikimedia
Details

Description Rainer Rillke @commons.wikimedia 2013-05-17 22:30:51 UTC
Original Bug title:
Icon-only-buttons broken in vector

broken by https://bits.wikimedia.org/static-1.22wmf4/resources/jquery.ui/themes/vector/jquery.ui.button.css
body .ui-button {
/*...*/
  width: auto
/*...*/
}

which overwrites

button.ui-button-icon-only {
  width: 2.4em
}


Test case (can be executed in a JS console):

$button = $('<button>').text('required but not visible').button({
icons: {
primary: "ui-icon-locked"
},
text: false
});
mw.util.$content.prepend($button);
Comment 1 Rainer Rillke @commons.wikimedia 2013-05-17 22:31:55 UTC
Created attachment 12335 [details]
rendering of broken vector button
Comment 2 Rainer Rillke @commons.wikimedia 2013-05-17 22:32:24 UTC
Created attachment 12336 [details]
expected result
Comment 3 Rainer Rillke @commons.wikimedia 2013-05-17 22:45:16 UTC
Created attachment 12337 [details]
new proposed content of resources/jquery.ui/themes/vector/jquery.ui.button.css
Comment 4 Rainer Rillke @commons.wikimedia 2013-05-17 22:45:44 UTC
Created attachment 12338 [details]
Patch to apply against resources/jquery.ui/themes/vector/jquery.ui.button.css
Comment 5 Andre Klapper 2013-05-24 14:53:35 UTC
Thanks! Anybody is welcome to use Developer access
  https://www.mediawiki.org/wiki/Developer_access
to submit this patch as a Git branch directly into Gerrit:
  https://www.mediawiki.org/wiki/Git/Tutorial
so the patch can get reviewed and merged into the codebase.
Comment 6 Rainer Rillke @commons.wikimedia 2013-07-18 05:43:01 UTC
Obviously no one wanted. Isn't there a way to submit a patch to gerrit quickly? A web-interface like github has, perhaps?
Comment 7 Rainer Rillke @commons.wikimedia 2013-07-18 11:38:20 UTC
Gerrit notification:

https://gerrit.wikimedia.org/r/74345
Comment 8 Bartosz Dziewoński 2013-07-18 11:51:15 UTC
(In reply to comment #6)
> Obviously no one wanted. Isn't there a way to submit a patch to gerrit
> quickly? A web-interface like github has, perhaps?

No, but it may or may not be in the works. (Currently you can already edit commit messages from the web interface using the little pencil icon on its top-right corner.)
Comment 9 Gerrit Notification Bot 2013-07-18 11:56:06 UTC
Change 74345 had a related patch set uploaded by Matmarex:
Fixing width of icon-only-buttons in Vector skin

https://gerrit.wikimedia.org/r/74345
Comment 10 Bartosz Dziewoński 2013-07-18 12:00:25 UTC
Merged.
Comment 11 Gerrit Notification Bot 2013-07-18 12:01:30 UTC
Change 74345 merged by jenkins-bot:
Fixing width of icon-only-buttons in Vector skin

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

Note You need to log in before you can comment on or make changes to this bug.


Navigation
Links