Esanders has uploaded a new change for review.

  https://gerrit.wikimedia.org/r/232476

Change subject: Remove isBorderBox from editor overlay
......................................................................

Remove isBorderBox from editor overlay

Caused problems with the OOUI editor switcher.

Added targeted border-box rule to textarea which
was actually using it.

Change-Id: I120cb757b67405691a1cbe2608595e35d3f82132
---
M i18n/en.json
M i18n/qqq.json
M includes/Resources.php
M resources/mobile.editor.common/editor.less
M resources/mobile.editor.overlay/EditorOverlay.js
M resources/mobile.overlays/Overlay.less
6 files changed, 5 insertions(+), 3 deletions(-)


  git pull ssh://gerrit.wikimedia.org:29418/mediawiki/extensions/MobileFrontend 
refs/changes/76/232476/1

diff --git a/i18n/en.json b/i18n/en.json
index dae527d..914481a 100644
--- a/i18n/en.json
+++ b/i18n/en.json
@@ -121,7 +121,6 @@
        "mobile-frontend-editor-previewing-page": 
"<span><strong>Previewing</strong> $1</span>",
        "mobile-frontend-editor-redirect-title": "Redirecting to editor...",
        "mobile-frontend-editor-save": "Save",
-       "mobile-frontend-editor-source-editor": "Edit source",
        "mobile-frontend-editor-success": "Success! Your edit was saved.",
        "mobile-frontend-editor-success-landmark-1": "Congratulations, you are 
now a {{SITENAME}} editor!",
        "mobile-frontend-editor-success-new-page": "Congratulations on creating 
a new page!",
diff --git a/i18n/qqq.json b/i18n/qqq.json
index 8a07ff6..b9ba690 100644
--- a/i18n/qqq.json
+++ b/i18n/qqq.json
@@ -117,7 +117,6 @@
        "mobile-frontend-editor-previewing-page": "A heading saying what page 
is being previewed after editing. All text should be wrapped in a STRONG tag 
except the page title itself.\n\nParameters:\n* $1 - page 
title\n{{Related|Mobile-frontend-editor-page}}",
        "mobile-frontend-editor-redirect-title": "Shown as a user is redirected 
to the JavaScript editor after browsing to a URL with action=edit.\n\nUsed as 
page title.",
        "mobile-frontend-editor-save": "Caption for save button on edit 
form.\n\nShould be consistent with the following message:\n* 
{{msg-mw|mobile-frontend-editor-license}}\n{{Identical|Save}}",
-       "mobile-frontend-editor-source-editor": "Label for button that switches 
to the 'source editor' (a WikiText editing interface). Appears near 
{{msg-mw|mobile-frontend-editor-visual-editor}}.\n{{Identical|Edit source}}",
        "mobile-frontend-editor-success": "Text that displays when a page edit 
has been successfully saved.\n\nUsed as \"toast\" message.\n\nSee also:\n* 
{{msg-mw|Mobile-frontend-editor-success-landmark-1}}",
        "mobile-frontend-editor-success-landmark-1": "Special save success 
message shown to users on their 1st edit.\n\nSee also:\n* 
{{msg-mw|Mobile-frontend-editor-success}}",
        "mobile-frontend-editor-success-new-page": "Text that displays when a 
new page has been successfully saved.\n\nUsed as \"toast\" message.",
diff --git a/includes/Resources.php b/includes/Resources.php
index 3b404ad..cf14c51 100644
--- a/includes/Resources.php
+++ b/includes/Resources.php
@@ -635,7 +635,6 @@
                        'mobile-frontend-editor-previewing-page',
                        'mobile-frontend-editor-switch-confirm',
                        'mobile-frontend-editor-visual-editor',
-                       'mobile-frontend-editor-source-editor',
                        'mobile-frontend-editor-switch-editor',
                        'mobile-frontend-editor-anonwarning',
                ),
diff --git a/resources/mobile.editor.common/editor.less 
b/resources/mobile.editor.common/editor.less
index f030e7a..64d590b 100644
--- a/resources/mobile.editor.common/editor.less
+++ b/resources/mobile.editor.common/editor.less
@@ -7,6 +7,7 @@
        }
 
        .wikitext-editor {
+               .box-sizing(border-box);
                padding: 10px @contentMargin;
        }
 
diff --git a/resources/mobile.editor.overlay/EditorOverlay.js 
b/resources/mobile.editor.overlay/EditorOverlay.js
index 9088169..fb420aa 100644
--- a/resources/mobile.editor.overlay/EditorOverlay.js
+++ b/resources/mobile.editor.overlay/EditorOverlay.js
@@ -20,6 +20,9 @@
         * @extends EditorOverlayBase
         */
        EditorOverlay = EditorOverlayBase.extend( {
+               /** @inheritdoc **/
+               isBorderBox: false,
+               /** @inheritdoc **/
                templatePartials: $.extend( {}, 
EditorOverlayBase.prototype.templatePartials, {
                        content: mw.template.get( 'mobile.editor.overlay', 
'content.hogan' ),
                        messageBox: MessageBox.prototype.template,
diff --git a/resources/mobile.overlays/Overlay.less 
b/resources/mobile.overlays/Overlay.less
index 1ed4781..1fb6d5f 100644
--- a/resources/mobile.overlays/Overlay.less
+++ b/resources/mobile.overlays/Overlay.less
@@ -98,6 +98,7 @@
        background: #fff;
        z-index: @z-indexOverlay;
        padding-top: @headerHeight;
+       .box-sizing(border-box);
        display: none;
 
        &.visible {

-- 
To view, visit https://gerrit.wikimedia.org/r/232476
To unsubscribe, visit https://gerrit.wikimedia.org/r/settings

Gerrit-MessageType: newchange
Gerrit-Change-Id: I120cb757b67405691a1cbe2608595e35d3f82132
Gerrit-PatchSet: 1
Gerrit-Project: mediawiki/extensions/MobileFrontend
Gerrit-Branch: master
Gerrit-Owner: Esanders <[email protected]>

_______________________________________________
MediaWiki-commits mailing list
[email protected]
https://lists.wikimedia.org/mailman/listinfo/mediawiki-commits

Reply via email to