wikimedia/mediawiki-extensions-VisualEditor
1
0
Fork 0
mirror of https://gerrit.wikimedia.org/r/mediawiki/extensions/VisualEditor synced 2024-09-24 10:48:42 +00:00
Code Issues Packages Projects Releases Wiki Activity

Merge "Use better align change descriptions"

Browse source
This commit is contained in:
jenkins-bot 2018-05-16 09:49:05 +00:00 committed by Gerrit Code Review
parent bbc5170428 7254fbe2cf
commit d70ddfb85a
2 changed files with 19 additions and 8 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

5
extension.json
View file

@ -985,6 +985,11 @@
"visualeditor",
"visualeditor-aliencontextitem-title",
"visualeditor-aliennode-tooltip",
"visualeditor-align-desc-center",
"visualeditor-align-desc-default",
"visualeditor-align-desc-left",
"visualeditor-align-desc-none",
"visualeditor-align-desc-right",
"visualeditor-align-widget-center",
"visualeditor-align-widget-default",
"visualeditor-align-widget-left",

22
modules/ve-mw/dm/nodes/ve.dm.MWImageNode.js
View file

@ -147,14 +147,20 @@ ve.dm.MWImageNode.static.describeChanges = function ( attributeChanges, attribut
};
ve.dm.MWImageNode.static.describeChange = function ( key, change ) {
if ( key === 'align' ) {
return ve.msg( 'visualeditor-changedesc-align',
// Messages used:
// visualeditor-align-widget-left, visualeditor-align-widget-right,
// visualeditor-align-widget-center, visualeditor-align-widget-default
ve.msg( 'visualeditor-align-widget-' + change.from ),
ve.msg( 'visualeditor-align-widget-' + change.to )
);
switch ( key ) {
case 'align':
return ve.msg( 'visualeditor-changedesc-align',
// Messages used:
// visualeditor-align-desc-left, visualeditor-align-desc-right,
// visualeditor-align-desc-center, visualeditor-align-desc-default,
// visualeditor-align-desc-none
ve.msg( 'visualeditor-align-desc-' + change.from ),
ve.msg( 'visualeditor-align-desc-' + change.to )
);
case 'originalClasses':
case 'unrecognizedClasses':
return;
// TODO: Handle valign
}
// Parent method
return ve.dm.Node.static.describeChange.apply( this, arguments );