WMDE-Fisch has uploaded a new change for review. (
https://gerrit.wikimedia.org/r/343084 )
Change subject: Use mw prefix on all HelpDialog CSS classes
......................................................................
Use mw prefix on all HelpDialog CSS classes
Change-Id: I2d72da98f78ad77534399a19c0f9e0208fef3af7
---
M modules/ext.TwoColConflict.HelpDialog.js
M tests/qunit/TwoColConflict.HelpDialog.test.js
2 files changed, 8 insertions(+), 12 deletions(-)
git pull ssh://gerrit.wikimedia.org:29418/mediawiki/extensions/TwoColConflict
refs/changes/84/343084/1
diff --git a/modules/ext.TwoColConflict.HelpDialog.js
b/modules/ext.TwoColConflict.HelpDialog.js
index 02b9f23..39ed83e 100644
--- a/modules/ext.TwoColConflict.HelpDialog.js
+++ b/modules/ext.TwoColConflict.HelpDialog.js
@@ -54,7 +54,7 @@
* @return {string}
*/
getCssPrefix: function () {
- return this.config.name.toLowerCase();
+ return 'mw-' + this.config.name.toLowerCase();
},
/**
@@ -71,7 +71,7 @@
slide.$element
.append(
$( '<div>' ).addClass(
- 'mw-' + this.getCssPrefix() +
+ this.getCssPrefix() +
'-help-dialog-image-' +
imageMode + ' ' + imageClass
)
)
@@ -131,12 +131,12 @@
*/
HelpDialog.init = function ( config ) {
var windowManager = new OO.ui.WindowManager(),
- dialog;
+ dialog = new HelpDialog( config );
$( 'body' )
.append( windowManager.$element )
.click( function ( event ) {
- if ( $( event.target ).hasClass(
config.name.toLowerCase() + '-help-dialog' ) ) {
+ if ( $( event.target ).hasClass(
dialog.getCssPrefix() + '-help-dialog' ) ) {
HelpDialog.hide();
}
} );
@@ -149,14 +149,12 @@
label: mw.msg( config.next ),
flags: [ 'primary', 'progressive' ],
modes: [ 'initial', 'middle' ],
- classes: [ config.name.toLowerCase() +
'-help-next' ]
+ classes: [ dialog.getCssPrefix() + '-help-next'
]
},
{ action: 'previous', flags: 'safe', label: mw.msg(
config.prev ), modes: [ 'middle', 'last' ], classes: [ config.name +
'-help-previous' ] },
- { label: mw.msg( config.close ), flags: 'safe', modes:
'initial', classes: [ config.name.toLowerCase() + '-help-close-start' ] },
- { label: mw.msg( config.close ), flags: 'primary',
modes: 'last', classes: [ config.name.toLowerCase() + '-help-close-end' ] }
+ { label: mw.msg( config.close ), flags: 'safe', modes:
'initial', classes: [ dialog.getCssPrefix() + '-help-close-start' ] },
+ { label: mw.msg( config.close ), flags: 'primary',
modes: 'last', classes: [ dialog.getCssPrefix() + '-help-close-end' ] }
];
-
- dialog = new HelpDialog( config );
HelpDialog.show = function () {
if ( !windowManager.hasWindow( dialog ) ) {
diff --git a/tests/qunit/TwoColConflict.HelpDialog.test.js
b/tests/qunit/TwoColConflict.HelpDialog.test.js
index 2e76215..88c60cf 100644
--- a/tests/qunit/TwoColConflict.HelpDialog.test.js
+++ b/tests/qunit/TwoColConflict.HelpDialog.test.js
@@ -1,6 +1,6 @@
( function ( mw ) {
var helpDialog = mw.libs.twoColConflict.HelpDialog,
- dialog;
+ dialog = helpDialog.getDialogInstance();
QUnit.module( 'ext.TwoColConflict.HelpDialog' );
@@ -35,8 +35,6 @@
}
]
} );
-
- dialog = helpDialog.getDialogInstance();
helpDialog.show();
helpDialog.hide();
--
To view, visit https://gerrit.wikimedia.org/r/343084
To unsubscribe, visit https://gerrit.wikimedia.org/r/settings
Gerrit-MessageType: newchange
Gerrit-Change-Id: I2d72da98f78ad77534399a19c0f9e0208fef3af7
Gerrit-PatchSet: 1
Gerrit-Project: mediawiki/extensions/TwoColConflict
Gerrit-Branch: master
Gerrit-Owner: WMDE-Fisch <[email protected]>
_______________________________________________
MediaWiki-commits mailing list
[email protected]
https://lists.wikimedia.org/mailman/listinfo/mediawiki-commits
