r69615 MediaWiki - Code Review archive

Repository:MediaWiki
Revision:r69614‎ | r69615 | r69616 >
Date:14:21, 20 July 2010
Author:adam
Status:ok
Tags:
Comment:
Removing double indentation on r68520
Modified paths:
  • /trunk/extensions/UsabilityInitiative/UsabilityInitiative.hooks.php (modified) (history)
  • /trunk/extensions/UsabilityInitiative/js/plugins/jquery.wikiEditor.previewDialog.js (modified) (history)

Diff [purge]

Index: trunk/extensions/UsabilityInitiative/js/plugins/jquery.wikiEditor.previewDialog.js
@@ -68,49 +68,49 @@
6969 width: $( 'body' ).width() - 300,
7070 position: ['center', 'top'],
7171 open: function() {
72 - // Gets the latest copy of the wikitext
73 - var wikitext = context.fn.getContents();
74 - var $dialog = $( '#' + dialogID );
75 - $dialog
76 - .css( 'position', 'relative' )
77 - .css( 'height', $( 'body' ).height() - 200 )
78 - .parent()
79 - .css( 'top', '25px' );
80 - // $dialog.dialog( 'option', 'width', $( 'body' ).width() - 300 );
81 - // Aborts when nothing has changed since the last preview
82 - if ( context.modules.preview.previewText == wikitext ) {
83 - return;
84 - }
85 -
86 - $dialog.find( '.wikiEditor-preview-dialog-contents' ).empty();
87 - $dialog.find( '.wikiEditor-ui-loading' ).show();
88 - $.post(
89 - wgScriptPath + '/api.php',
90 - {
91 - 'action': 'parse',
92 - 'title': wgPageName,
93 - 'text': wikitext,
94 - 'prop': 'text',
95 - 'pst': '',
96 - 'format': 'json'
97 - },
98 - function( data ) {
99 - if (
100 - typeof data.parse == 'undefined' ||
101 - typeof data.parse.text == 'undefined' ||
102 - typeof data.parse.text['*'] == 'undefined'
103 - ) {
104 - return;
105 - }
106 - context.modules.preview.previewText = wikitext;
107 - $dialog.find( '.wikiEditor-ui-loading' ).hide();
108 - $dialog.find( '.wikiEditor-preview-dialog-contents' )
109 - .html( '<h1 class="firstHeading" id="firstHeading">'+wgTitle+'</h1>' +
110 - data.parse.text['*'] )
111 - .find( 'a:not([href^=#])' ).click( function() { return false; } );
112 - },
113 - 'json'
114 - );
 72+ // Gets the latest copy of the wikitext
 73+ var wikitext = context.fn.getContents();
 74+ var $dialog = $( '#' + dialogID );
 75+ $dialog
 76+ .css( 'position', 'relative' )
 77+ .css( 'height', $( 'body' ).height() - 200 )
 78+ .parent()
 79+ .css( 'top', '25px' );
 80+ // $dialog.dialog( 'option', 'width', $( 'body' ).width() - 300 );
 81+ // Aborts when nothing has changed since the last preview
 82+ if ( context.modules.preview.previewText == wikitext ) {
 83+ return;
 84+ }
 85+
 86+ $dialog.find( '.wikiEditor-preview-dialog-contents' ).empty();
 87+ $dialog.find( '.wikiEditor-ui-loading' ).show();
 88+ $.post(
 89+ wgScriptPath + '/api.php',
 90+ {
 91+ 'action': 'parse',
 92+ 'title': wgPageName,
 93+ 'text': wikitext,
 94+ 'prop': 'text',
 95+ 'pst': '',
 96+ 'format': 'json'
 97+ },
 98+ function( data ) {
 99+ if (
 100+ typeof data.parse == 'undefined' ||
 101+ typeof data.parse.text == 'undefined' ||
 102+ typeof data.parse.text['*'] == 'undefined'
 103+ ) {
 104+ return;
 105+ }
 106+ context.modules.preview.previewText = wikitext;
 107+ $dialog.find( '.wikiEditor-ui-loading' ).hide();
 108+ $dialog.find( '.wikiEditor-preview-dialog-contents' )
 109+ .html( '<h1 class="firstHeading" id="firstHeading">'+wgTitle+'</h1>' +
 110+ data.parse.text['*'] )
 111+ .find( 'a:not([href^=#])' ).click( function() { return false; } );
 112+ },
 113+ 'json'
 114+ );
115115 }
116116 },
117117 resizeme: false
Index: trunk/extensions/UsabilityInitiative/UsabilityInitiative.hooks.php
@@ -86,7 +86,7 @@
8787 array( 'src' => 'js/plugins.combined.js', 'version' => 453 ),
8888 ),
8989 'minified' => array(
90 - array( 'src' => 'js/plugins.combined.min.js', 'version' => 471 ),
 90+ array( 'src' => 'js/plugins.combined.min.js', 'version' => 472 ),
9191 ),
9292 ),
9393 );

Past revisions this follows-up on

RevisionCommit summaryAuthorDate
r68520Adding the previewDialog module for prototyping new ideas for previewadam16:03, 24 June 2010

Status & tagging log