Esanders has uploaded a new change for review.

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

Change subject: Create RTL version of minimal demo
......................................................................

Create RTL version of minimal demo

Change-Id: I79f33e9e38260ea76a9144d6e17a8ae9b998ff5a
---
M Gruntfile.js
M build/tasks/buildloader.js
M demos/ve/demo.minimal.js
A demos/ve/minimal-rtl.html
M demos/ve/minimal.html
M demos/ve/minimal.html.template
6 files changed, 107 insertions(+), 11 deletions(-)


  git pull ssh://gerrit.wikimedia.org:29418/VisualEditor/VisualEditor 
refs/changes/42/204942/1

diff --git a/Gruntfile.js b/Gruntfile.js
index 7fa6183..b37ed7c 100644
--- a/Gruntfile.js
+++ b/Gruntfile.js
@@ -128,7 +128,7 @@
                                pathPrefix: '../',
                                i18n: [ 'i18n/', 'lib/oojs-ui/i18n/' ],
                                indent: '\t\t',
-                               bidi: false
+                               dir: 'ltr'
                        },
                        desktopDemo: {
                                targetFile: 'demos/ve/desktop.html',
@@ -204,7 +204,22 @@
                                pathPrefix: '../../',
                                i18n: [ 'dist/i18n/', 'lib/oojs-ui/i18n/' ],
                                indent: '\t\t',
-                               bidi: false,
+                               dir: 'ltr',
+                               langList: false
+                       },
+                       minimalDemoRtl: {
+                               targetFile: 'demos/ve/minimal-rtl.html',
+                               template: 'demos/ve/minimal.html.template',
+                               modules: modules,
+                               load: [
+                                       'visualEditor.standalone.apex.dist',
+                                       'visualEditor.standalone.read'
+                               ],
+                               run: [ 'visualEditor.minimal.standalone.demo' ],
+                               pathPrefix: '../../',
+                               i18n: [ 'dist/i18n/', 'lib/oojs-ui/i18n/' ],
+                               indent: '\t\t',
+                               dir: 'rtl',
                                langList: false
                        },
                        test: {
diff --git a/build/tasks/buildloader.js b/build/tasks/buildloader.js
index 84d398f..117eefb 100644
--- a/build/tasks/buildloader.js
+++ b/build/tasks/buildloader.js
@@ -19,7 +19,7 @@
                        run = this.data.run || [],
                        env = this.data.env || {},
                        placeholders = this.data.placeholders || {},
-                       bidi = this.data.bidi !== undefined ? this.data.bidi : 
true,
+                       dir = this.data.dir,
                        langList = this.data.langList !== undefined ? 
this.data.langList : true,
                        text = grunt.file.read( this.data.template ),
                        done = this.async(),
@@ -32,11 +32,16 @@
                function styleTag( group, src ) {
                        var rtlFilepath = src.file.replace( /\.css$/, 
'.rtl.css' );
 
-                       if ( bidi && grunt.file.exists( rtlFilepath ) ) {
-                               return indent + '<link rel=stylesheet href="' + 
pathPrefix + src.file + '" class="stylesheet-ltr' +
-                                       ( group ? ' stylesheet-' + group : '' ) 
+ '">\n' +
-                                       indent + '<link rel=stylesheet href="' 
+ pathPrefix + rtlFilepath + '" class="stylesheet-rtl' +
-                                       ( group ? ' stylesheet-' + group : '' ) 
+ '">';
+                       if ( grunt.file.exists( rtlFilepath ) ) {
+                               if ( !dir ) {
+                                       return indent + '<link rel=stylesheet 
href="' + pathPrefix + src.file + '" class="stylesheet-ltr' +
+                                               ( group ? ' stylesheet-' + 
group : '' ) + '">\n' +
+                                               indent + '<link rel=stylesheet 
href="' + pathPrefix + rtlFilepath + '" class="stylesheet-rtl' +
+                                               ( group ? ' stylesheet-' + 
group : '' ) + '">';
+                               } else if ( dir === 'rtl' ) {
+                                       return indent + '<link rel=stylesheet 
href="' + pathPrefix + rtlFilepath + '"' +
+                                               ( group ? ' class="stylesheet-' 
+ group + '"' : '' ) + '>';
+                               }
                        }
                        return indent + '<link rel=stylesheet href="' + 
pathPrefix + src.file + '"' +
                                ( group ? ' class="stylesheet-' + group + '"' : 
'' ) + '>';
@@ -129,6 +134,7 @@
 
                placeholders.styles = styles.join( '\n\n' );
                placeholders.scripts = scripts.join( '\n\n' );
+               placeholders.dir = dir;
 
                grunt.util.async.forEachSeries(
                        Object.keys( placeholders ),
diff --git a/demos/ve/demo.minimal.js b/demos/ve/demo.minimal.js
index 3cec76f..93f3d52 100644
--- a/demos/ve/demo.minimal.js
+++ b/demos/ve/demo.minimal.js
@@ -13,8 +13,9 @@
        // Create a document model for a new surface
        target.addSurface(
                ve.dm.converter.getModelFromDom(
-                       ve.createDocumentFromHtml( '<p><b>Hello,</b> 
<i>World!</i></p>' )
+                       ve.createDocumentFromHtml( '<p><b>Hello,</b> 
<i>World!</i></p>' ),
                        // Optional: Document language, directionality (ltr/rtl)
+                       { lang: $.i18n().locale, dir: $( 'body' ).css( 
'direction' ) }
                )
        );
 
diff --git a/demos/ve/minimal-rtl.html b/demos/ve/minimal-rtl.html
new file mode 100644
index 0000000..3dcae7b
--- /dev/null
+++ b/demos/ve/minimal-rtl.html
@@ -0,0 +1,74 @@
+<!DOCTYPE html>
+<html>
+       <!--
+       /*!
+        * VisualEditor minimal demo
+        *
+        * @copyright 2011-2015 VisualEditor Team and others; see 
http://ve.mit-license.org
+        */
+       -->
+       <head>
+               <meta charset="utf-8">
+               <meta name="viewport" content="initial-scale=1.0, 
user-scalable=no" />
+               <title>VisualEditor Standalone Demo</title>
+
+               <!-- oojs-ui-apex -->
+               <link rel=stylesheet 
href="../../lib/oojs-ui/oojs-ui-apex.vector.rtl.css">
+               <link rel=stylesheet 
href="../../lib/oojs-ui/oojs-ui-apex-icons-editing-core.vector.rtl.css">
+               <link rel=stylesheet 
href="../../lib/oojs-ui/oojs-ui-apex-icons-editing-advanced.vector.rtl.css">
+               <link rel=stylesheet 
href="../../lib/oojs-ui/oojs-ui-apex-icons-editing-styling.vector.rtl.css">
+               <link rel=stylesheet 
href="../../lib/oojs-ui/oojs-ui-apex-icons-editing-list.vector.rtl.css">
+
+               <!-- visualEditor.dist.apex -->
+               <link rel=stylesheet 
href="../../dist/visualEditor-apex.rtl.css" class="stylesheet-ve">
+
+               <!-- visualEditor.standalone.read -->
+               <link rel=stylesheet 
href="../../src/init/sa/styles/ve.init.sa.css" class="stylesheet-read">
+
+               <!-- visualEditor.minimal.standalone.demo -->
+               <link rel=stylesheet href="../../demos/ve/demo.minimal.css">
+
+       </head>
+       <body dir="rtl">
+               <div class="ve-instance"></div>
+               <div class="ve-demo-output">
+                       <input type="button" class="ve-demo-convert" 
value="&darr; Convert to HTML">
+                       <textarea class="ve-demo-html"></textarea>
+               </div>
+
+               <!-- jquery -->
+               <script src="../../lib/jquery/jquery.js"></script>
+
+               <!-- oojs -->
+               <script src="../../lib/oojs/oojs.jquery.js"></script>
+
+               <!-- oojs-ui -->
+               <script src="../../lib/oojs-ui/oojs-ui.js"></script>
+
+               <!-- jquery.i18n.dist -->
+               <script src="../../dist/lib/jquery.i18n.js"></script>
+
+               <!-- jquery.uls.data.dist -->
+               <script src="../../dist/lib/jquery.uls.data.js"></script>
+
+               <!-- jquery.client -->
+               <script src="../../lib/jquery.client/jquery.client.js"></script>
+
+               <!-- papaparse -->
+               <script src="../../lib/papaparse/papaparse.js"></script>
+
+               <!-- oojs-ui-apex -->
+               <script src="../../lib/oojs-ui/oojs-ui-apex.js"></script>
+
+               <!-- visualEditor.dist -->
+               <script src="../../dist/visualEditor.js"></script>
+
+               <script>
+                       ve.messagePaths = 
["../../dist/i18n/","../../lib/oojs-ui/i18n/"];
+               </script>
+
+               <!-- visualEditor.minimal.standalone.demo -->
+               <script src="../../demos/ve/demo.minimal.js"></script>
+
+       </body>
+</html>
diff --git a/demos/ve/minimal.html b/demos/ve/minimal.html
index dc547bd..b412a6f 100644
--- a/demos/ve/minimal.html
+++ b/demos/ve/minimal.html
@@ -29,7 +29,7 @@
                <link rel=stylesheet href="../../demos/ve/demo.minimal.css">
 
        </head>
-       <body>
+       <body dir="ltr">
                <div class="ve-instance"></div>
                <div class="ve-demo-output">
                        <input type="button" class="ve-demo-convert" 
value="&darr; Convert to HTML">
diff --git a/demos/ve/minimal.html.template b/demos/ve/minimal.html.template
index ab2c32f..cc5e3a3 100644
--- a/demos/ve/minimal.html.template
+++ b/demos/ve/minimal.html.template
@@ -15,7 +15,7 @@
 <!-- STYLES -->
 
        </head>
-       <body>
+       <body dir="<!-- DIR -->">
                <div class="ve-instance"></div>
                <div class="ve-demo-output">
                        <input type="button" class="ve-demo-convert" 
value="&darr; Convert to HTML">

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

Gerrit-MessageType: newchange
Gerrit-Change-Id: I79f33e9e38260ea76a9144d6e17a8ae9b998ff5a
Gerrit-PatchSet: 1
Gerrit-Project: VisualEditor/VisualEditor
Gerrit-Branch: master
Gerrit-Owner: Esanders <esand...@wikimedia.org>

_______________________________________________
MediaWiki-commits mailing list
MediaWiki-commits@lists.wikimedia.org
https://lists.wikimedia.org/mailman/listinfo/mediawiki-commits

Reply via email to