Nikerabbit has uploaded a new change for review.

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


Change subject: Apply light formatting in page mode
......................................................................

Apply light formatting in page mode

Bug: 45548
Change-Id: I1e131ad18008fa05784c4f67efd798a9c0598484
---
M Translate.php
M resources/js/ext.translate.messagetable.js
A resources/js/ext.translate.parsers.js
3 files changed, 85 insertions(+), 4 deletions(-)


  git pull ssh://gerrit.wikimedia.org:29418/mediawiki/extensions/Translate 
refs/changes/36/53036/1

diff --git a/Translate.php b/Translate.php
index ebc809a..a4174df 100644
--- a/Translate.php
+++ b/Translate.php
@@ -322,6 +322,7 @@
                'mediawiki.util',
                'jquery.appear',
                'mediawiki.jqueryMsg',
+               'ext.translate.parsers',
        ),
        'messages' => array(
                'translate-messagereview-submit',
@@ -382,6 +383,13 @@
        ),
 ) + $resourcePaths;
 
+$wgResourceModules['ext.translate.parsers'] = array(
+       'scripts' => 'resources/js/ext.translate.parsers.js',
+       'dependencies' => array(
+               'mediawiki.util',
+       ),
+) + $resourcePaths;
+
 $wgResourceModules['ext.translate.selecttoinput'] = array(
        'scripts' => 'resources/js/ext.translate.selecttoinput.js',
 ) + $resourcePaths;
diff --git a/resources/js/ext.translate.messagetable.js 
b/resources/js/ext.translate.messagetable.js
index ed85667..20a0202 100644
--- a/resources/js/ext.translate.messagetable.js
+++ b/resources/js/ext.translate.messagetable.js
@@ -246,16 +246,14 @@
                                                        lang: sourceLanguage,
                                                        dir: sourceLanguageDir
                                                } )
-                                               // FIXME this should be parsed 
wiki text
-                                               .text( message.definition ),
+                                               .html( 
mw.translate.formatMessageGently( message.definition, message.key ) ),
                                        $( '<div>' )
                                                .addClass( 'six columns 
tux-proofread-translation' )
                                                .attr( {
                                                        lang: targetLanguage,
                                                        dir: targetLanguageDir
                                                } )
-                                               // FIXME this should be parsed 
wiki text
-                                               .text( message.translation || 
'' )
+                                               .html( 
mw.translate.formatMessageGently( message.translation || '', message.key ) )
                                );
 
                        this.$container.append( $message );
diff --git a/resources/js/ext.translate.parsers.js 
b/resources/js/ext.translate.parsers.js
new file mode 100644
index 0000000..d1c0847
--- /dev/null
+++ b/resources/js/ext.translate.parsers.js
@@ -0,0 +1,75 @@
+/*
+ * Stuff related to manipulating and parsing message content.
+ * @author Niklas Laxström, 2013
+ * @license GPL2+
+ */
+
+( function ( $, mw ) {
+       'use strict';
+       
+       mw.translate = mw.translate || {};
+
+       mw.translate = $.extend( mw.translate, {
+               /**
+                * Formats some common wikitext elements
+                * @param {String} text Message text
+                * @param {String} [key] Message key
+                * @return {String} Formatted text in html
+                */
+               formatMessageGently: function ( text, key ) {
+                       // Try to keep simple.
+                       text = $( '<div>' ).text( text ).html();
+
+                       // Hack for page translation page titles
+                       if ( key && key.match( /\/Page_display_title$/ ) ) {
+                               text = '=' + text + '=';
+                       }
+
+                       text = text.replace( /^(=+)(.*?)(=+)/, function ( 
match, p1, p2, p3 ) {
+                               var len = Math.min( p1.length, p2.length, 6 );
+                               return $( '<div>' ).append( $( '<h' + len + '>' 
).html( p2 ) ).html();
+                       } );
+
+                       text = text.replace( /(^\*.*(\n|$))+/gm, function ( 
match ) {
+                               match = match.replace( /^\*(.*)/gm, function ( 
match, p1 ) {
+                                       return $( '<div>' ).append( $( '<li>' 
).html( p1 ) ).html();
+                               } );
+                               return $( '<div>' ).append( $( '<ul>' ).html( 
match ) ).html();
+                       } );
+
+                       text = text.replace( /(^#.*(\n|$))+/gm, function ( 
match ) {
+                               match = match.replace( /^#(.*)/gm, function ( 
match, p1 ) {
+                                       return $( '<div>' ).append( $( '<li>' 
).html( p1 ) ).html();
+                               } );
+                               return $( '<div>' ).append( $( '<ol>' ).html( 
match ) ).html();
+                       } );
+
+                       text = text.replace( /\[\[(.+?)\|(.+?)\]\]/g, function 
( match, p1, p2 ) {
+                               var link = $( '<a>' ).html( p2 ).prop( 'href', 
mw.util.wikiGetlink( p1 ) );
+                               return $( '<div>' ).append( link ).html();
+                       } );
+
+                       text = text.replace( /\[\[(.+?)\]\]/g, function ( 
match, p1 ) {
+                               var link = $( '<a>' ).html( p1 ).prop( 'href', 
mw.util.wikiGetlink( p1 ) );
+                               return $( '<div>' ).append( link ).html();
+                       } );
+
+                       text = text.replace( /\[([^ ]+) (.+?)\]/g, function ( 
match, p1, p2 ) {
+                               var link = $( '<a>' ).html( p2 ).prop( 'href', 
p1 );
+                               return $( '<div>' ).append( link ).html();
+                       } );
+
+                       text = text.replace( /'''(.+?)'''/g, function ( match, 
p1 ) {
+                               return $( '<div>' ).append( $( '<b>' ).html( p1 
) ).html();
+                       } );
+
+                       text = text.replace( /''(.+?)''/g, function ( match, p1 
) {
+                               return $( '<div>' ).append( $( '<i>' ).html( p1 
) ).html();
+                       } );
+
+                       text = text.replace( /\n\n/gm, '<br>' );
+                       return text;
+               }
+       } );
+
+} ( jQuery, mediaWiki ) );

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

Gerrit-MessageType: newchange
Gerrit-Change-Id: I1e131ad18008fa05784c4f67efd798a9c0598484
Gerrit-PatchSet: 1
Gerrit-Project: mediawiki/extensions/Translate
Gerrit-Branch: master
Gerrit-Owner: Nikerabbit <[email protected]>

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

Reply via email to