Last modified: 2014-09-18 05:48:11 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 T64744, the corresponding Phabricator task for complete and up-to-date bug report information.
Bug 62744 - implement light inline button style in mediawiki.ui
implement light inline button style in mediawiki.ui
Status: RESOLVED FIXED
Product: MediaWiki
Classification: Unclassified
MediaWiki UI (Other open bugs)
unspecified
All All
: Normal normal (vote)
: ---
Assigned To: Nobody - You can work on this!
https://trello.com/c/wBiXSreC
:
Depends on:
Blocks: 70913 mediawiki.ui
  Show dependency treegraph
 
Reported: 2014-03-17 18:55 UTC by spage
Modified: 2014-09-18 05:48 UTC (History)
8 users (show)

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


Attachments

Description spage 2014-03-17 18:55:01 UTC
The Flow discussion system uses a style for its post interaction buttons "Reply • Edit • Thank" that is even lighter-weight than mw-ui-quiet buttons (bug 62556). This could/should move to core, e.g. a mediawiki.ui.buttons.light module.
Comment 1 Matthew Flaschen 2014-03-24 21:33:28 UTC
This doesn't necessarily need its own module (if we put it in core, I'm inclined that it should go into mediawiki.button).
Comment 2 Matthew Flaschen 2014-03-24 21:33:40 UTC
I mean mediawiki.ui.button
Comment 3 Gerrit Notification Bot 2014-03-26 16:47:36 UTC
Change 121101 had a related patch set uploaded by Bencmq:
Add mw-ui-light button style

https://gerrit.wikimedia.org/r/121101
Comment 4 spage 2014-03-27 01:30:49 UTC
Looking at the complexity of the patch...
The problem with defining a .mw-ui-light style that modifies .mw-ui-button is that it must reset all of mw-ui-button's  background, border, font-weight, padding, etc. simply to return to plain inline text. It would be simpler and cleaner to allow the existing mw.ui-{constructive,destructive,progressive,neutral} styles to add their coloring on hover & click to any item. Then we don't need a .mw-ui-light style at all.

This approach would also make mw-ui-quiet simpler; note Shahyar Ghobadpour proposes to get rid of mw-ui-quiet in core.
Comment 5 Gerrit Notification Bot 2014-04-04 03:04:10 UTC
Change 121101 abandoned by Bencmq:
Add mw-ui-light button style

https://gerrit.wikimedia.org/r/121101
Comment 6 Jon 2014-08-28 23:17:12 UTC
Is this bug still relevant?
Comment 7 Matthew Flaschen 2014-09-04 22:08:34 UTC
Seems to be addressed by the current anchors patch (https://gerrit.wikimedia.org/r/#/c/155856/1).  Will propose tagging as such.
Comment 8 Matthew Flaschen 2014-09-04 22:16:55 UTC
I actually meant to link the original anchor patch (https://gerrit.wikimedia.org/r/#/c/150577/14), which is merged.
Comment 9 Jon 2014-09-16 23:26:09 UTC
I'm confused. is this issue fixed or not? If not could you more clearly explain the issue?
Comment 10 Matthew Flaschen 2014-09-18 05:48:11 UTC
Yes, I think it's addressed by https://tools.wmflabs.org/styleguide/desktop/section-6.html#section-6.1 (the classes used for that probably need to be tweaked per bug 70818, but I assume the feature will be kept).

If not, please reopen and explain.

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


Navigation
Links