jenkins-bot has submitted this change and it was merged. Change subject: Flexiskin: Upload file button - solved issue with missing tooltip and icon ......................................................................
Flexiskin: Upload file button - solved issue with missing tooltip and icon Change-Id: I4af7e74a4fd95008a2c5596d2e7b85f9648811e1 --- M Flexiskin/extension.json M Flexiskin/i18n/en.json M Flexiskin/i18n/qqq.json M Flexiskin/resources/BS.Flexiskin/Menuitems/General.js M Flexiskin/resources/BS.Flexiskin/Menuitems/Header.js M Flexiskin/resources/bluespice.flexiskin.css 6 files changed, 15 insertions(+), 7 deletions(-) Approvals: Robert Vogel: Looks good to me, approved Mglaser: Looks good to me, but someone else must approve Raimond Spekking: Looks good to me, but someone else must approve jenkins-bot: Verified diff --git a/Flexiskin/extension.json b/Flexiskin/extension.json index 2e747d3..6eb481f 100644 --- a/Flexiskin/extension.json +++ b/Flexiskin/extension.json @@ -94,7 +94,8 @@ "bs-flexiskin-error-templatenotexists", "bs-flexiskin-usebackground", "bs-flexiskin-labelcurrentbackground", - "bs-extjs-upload" + "bs-extjs-upload", + "bs-flexiskin-button-upload-image-tooltip" ] } }, diff --git a/Flexiskin/i18n/en.json b/Flexiskin/i18n/en.json index 2acb4cc..3cb9f31 100644 --- a/Flexiskin/i18n/en.json +++ b/Flexiskin/i18n/en.json @@ -66,5 +66,6 @@ "apihelp-bs-flexiskin-upload-description": "Allows to upload files to a flexiskin", "apihelp-bs-flexiskin-upload-store-description": "Lists uploaded files of a flexiskin", "action-flexiskin-viewspecialpage": "view specialpage \"Flexiskin\"", - "flexiskin": "Skin preferences" + "flexiskin": "Skin preferences", + "bs-flexiskin-button-upload-image-tooltip": "Upload new image" } diff --git a/Flexiskin/i18n/qqq.json b/Flexiskin/i18n/qqq.json index 326e2b3..3e837a3 100644 --- a/Flexiskin/i18n/qqq.json +++ b/Flexiskin/i18n/qqq.json @@ -69,5 +69,6 @@ "apihelp-bs-flexiskin-upload-description": "Describes an API endpoint that is used to upload files to a flexiskin", "apihelp-bs-flexiskin-upload-store-description": "Describes an API endpoint that lists uploaded files of a flexiskin", "action-flexiskin-viewspecialpage": "Part of error message when access denied to view specialpage \"Flexiskin\"", - "flexiskin": "Title for special page of extension \"Flexiskin\"" + "flexiskin": "Title for special page of extension \"Flexiskin\"", + "bs-flexiskin-button-upload-image-tooltip": "Tooltip: upload button for logo, background, ..." } diff --git a/Flexiskin/resources/BS.Flexiskin/Menuitems/General.js b/Flexiskin/resources/BS.Flexiskin/Menuitems/General.js index b890814..2908670 100644 --- a/Flexiskin/resources/BS.Flexiskin/Menuitems/General.js +++ b/Flexiskin/resources/BS.Flexiskin/Menuitems/General.js @@ -122,9 +122,9 @@ } ); this.buttonUploadBackground = Ext.create( 'Ext.Button', { - tooltip: mw.message( 'bs-flexiskin-button-upload-file' ).plain(), + tooltip: mw.message( 'bs-flexiskin-button-upload-image-tooltip' ).plain(), width: '20px', - iconCls: 'icon-upload3', + iconCls: 'bs-icon-upload', style: { marginLeft: '10px', color: 'white' diff --git a/Flexiskin/resources/BS.Flexiskin/Menuitems/Header.js b/Flexiskin/resources/BS.Flexiskin/Menuitems/Header.js index 8aa27aa..2a4fa95 100644 --- a/Flexiskin/resources/BS.Flexiskin/Menuitems/Header.js +++ b/Flexiskin/resources/BS.Flexiskin/Menuitems/Header.js @@ -40,9 +40,9 @@ }); this.buttonUploadLogo = Ext.create( 'Ext.Button', { - tooltip: mw.message( 'bs-flexiskin-button-upload-file' ).plain(), + tooltip: mw.message( 'bs-flexiskin-button-upload-image-tooltip' ).plain(), width: '20px', - iconCls: 'icon-upload3', + iconCls: 'bs-icon-upload', style: { marginLeft: '10px', color: 'white' diff --git a/Flexiskin/resources/bluespice.flexiskin.css b/Flexiskin/resources/bluespice.flexiskin.css index 1272cf5..c7303af 100644 --- a/Flexiskin/resources/bluespice.flexiskin.css +++ b/Flexiskin/resources/bluespice.flexiskin.css @@ -6,4 +6,9 @@ } #bs-flexiskin-config-settings{ border-right: 1px solid black; +} + +span.x-btn-icon-el.bs-icon-upload { + font-size: 12px; + text-align: left; } \ No newline at end of file -- To view, visit https://gerrit.wikimedia.org/r/319294 To unsubscribe, visit https://gerrit.wikimedia.org/r/settings Gerrit-MessageType: merged Gerrit-Change-Id: I4af7e74a4fd95008a2c5596d2e7b85f9648811e1 Gerrit-PatchSet: 1 Gerrit-Project: mediawiki/extensions/BlueSpiceExtensions Gerrit-Branch: master Gerrit-Owner: Dvogel hallowelt <daniel.vo...@hallowelt.com> Gerrit-Reviewer: Ljonka <l.verhovs...@gmail.com> Gerrit-Reviewer: Mglaser <gla...@hallowelt.biz> Gerrit-Reviewer: Pwirth <wi...@hallowelt.biz> Gerrit-Reviewer: Raimond Spekking <raimond.spekk...@gmail.com> Gerrit-Reviewer: Robert Vogel <vo...@hallowelt.biz> Gerrit-Reviewer: Siebrand <siebr...@kitano.nl> Gerrit-Reviewer: jenkins-bot <> _______________________________________________ MediaWiki-commits mailing list MediaWiki-commits@lists.wikimedia.org https://lists.wikimedia.org/mailman/listinfo/mediawiki-commits