Last modified: 2014-01-03 01:34:31 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 T61099, the corresponding Phabricator task for complete and up-to-date bug report information.
Bug 59099 - mw.notify() with type to get different ui with predefined styles
mw.notify() with type to get different ui with predefined styles
Status: PATCH_TO_REVIEW
Product: MediaWiki
Classification: Unclassified
JavaScript (Other open bugs)
1.23.0
All All
: Low enhancement (vote)
: ---
Assigned To: Nobody - You can work on this!
: design
Depends on:
Blocks:
  Show dependency treegraph
 
Reported: 2013-12-29 11:45 UTC by Fomafix
Modified: 2014-01-03 01:34 UTC (History)
5 users (show)

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


Attachments
mw.notify() with type for watch and patrol (3.46 KB, patch)
2013-12-29 11:45 UTC, Fomafix
Details

Description Fomafix 2013-12-29 11:45:32 UTC
Created attachment 14188 [details]
mw.notify() with type for watch and patrol

mw.notify() is used for success and for fail notifications. But it has always the same style.

It would be nice to have a set of predefined ui styles for different types. The types should orient at the API results: For example:

* type "notify": Green. Success. Default style.
* type "warn": Yellow. Success but with warning.
* type "error": Red. Fail.

I created a patch to demonstrate the idea. The styles use different background colors for the different types. It would be better to have self-explanatory icons.
Comment 1 Andre Klapper 2014-01-02 11:45:25 UTC
Hi! Thanks for your patch!

You are welcome to use Developer access
  https://www.mediawiki.org/wiki/Developer_access
to submit this as a Git branch directly into Gerrit:
  https://www.mediawiki.org/wiki/Git/Tutorial

Putting your branch in Git makes it easier to review it quickly. If you don't want to set up Git/Gerrit, you can also use https://tools.wmflabs.org/gerrit-patch-uploader/
Thanks again! We appreciate your contribution.
Comment 2 Gerrit Notification Bot 2014-01-02 22:40:51 UTC
Change 105104 had a related patch set uploaded by Gerrit Patch Uploader:
mw.notify() with message type and predefined styles

https://gerrit.wikimedia.org/r/105104
Comment 3 Bartosz Dziewoński 2014-01-03 00:16:51 UTC
(Fomafix, there are some comments on the gerrit change – I'm posting here since I'm not sure if you're watching it. You should get a labs/gerrit account (aka "Developer access") to be able to get automated mail about new comments from gerrit and to reply there, even if you're using the patch uploader to submit patches :) )

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


Navigation
Links