Last modified: 2014-03-05 21:19:23 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 T48853, the corresponding Phabricator task for complete and up-to-date bug report information.
Bug 46853 - mediawiki.jqueryMsg really should be default and merged with the mediawiki.Message() code
mediawiki.jqueryMsg really should be default and merged with the mediawiki.Me...
Status: NEW
Product: MediaWiki
Classification: Unclassified
JavaScript (Other open bugs)
1.21.x
All All
: Normal major (vote)
: ---
Assigned To: Nobody - You can work on this!
:
Depends on:
Blocks: code_quality
  Show dependency treegraph
 
Reported: 2013-04-03 18:50 UTC by Bartosz Dziewoński
Modified: 2014-03-05 21:19 UTC (History)
5 users (show)

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


Attachments

Description Bartosz Dziewoński 2013-04-03 18:50:01 UTC
mediawiki.jqueryMsg really should be default and merged with the mediawiki.Message() code.

Having two partially incompatible client-side parsers is nasty and leads to nasty hard-to-debug issues when mw.msg accepts messages that are considered invalid by mw.jqueryMsg (I've just fixed one in a gadget used at pl.wikipedia).

Let's just merge those already; the jqueryMsg code works well and is widely used. mediawiki.jqueryMsg module should probably become a no-op for backwards-compatibility.

(Also, let's please get rid of that horrific name.)
Comment 1 Matthew Flaschen 2013-04-03 20:47:00 UTC
Agreed.

I suggested the same thing on Wikitech: http://comments.gmane.org/gmane.science.linguistics.wikipedia.technical/68076 . No one opposed the idea, though Amir Aharoni mentioned https://github.com/wikimedia/jquery.i18n.  On Wikitech, that counts as overwhelming consensus.

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


Navigation
Links