New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Factor inputextender and content out of time and globecoordinate experts #51
Conversation
The messages are new and will be translated again. The old messages will be deleted. No problem. |
@tobijat @thiemowmde ping |
The `messageGetter` was called with a params array as second parameter. That behavior matched the documentation, but the function usually passed as `messageGetter` (`mw.msg`) expects the params to be given as individual arguments.
|
||
$.extend( ExpertExtender.CalendarHint.prototype, { | ||
/** | ||
* @type {util.MessageProvider} |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Indenting
Generally looks sane to me, though I really am not up to speed enough with the JS for that to be very meaningful |
This introduces ExpertExtender with some modules. It makes it possible to reuse the inputextender in other experts, like quantity or monolingualtext.
This looks good from an architecture point of view. I can't say much about the details of the code. |
Factor inputextender and content out of time and globecoordinate experts
This introduces ExpertExtender with some modules. It makes it possible to reuse
the inputextender in other experts, like quantity or monolingualtext.
@siebrand, can you please have a look at the renaming of messages?