r51621 MediaWiki - Code Review archive

Repository:MediaWiki
Revision:< r51620‎ | r51621 | r51622 >
Date:23:41, 8 June 2009
Author:tparscal
Status:deferred
Tags:
Comment:
Foxed some i18n issues, reogranized messages a bit, fixed group label bug.
Modified paths:
  • /trunk/extensions/UsabilityInitiative/EditToolbar/EditToolbar.hooks.php (modified) (history)
  • /trunk/extensions/UsabilityInitiative/EditToolbar/EditToolbar.i18n.php (modified) (history)
  • /trunk/extensions/UsabilityInitiative/EditToolbar/EditToolbar.js (modified) (history)

Diff [purge]

Index: trunk/extensions/UsabilityInitiative/EditToolbar/EditToolbar.hooks.php
——@@ -42,46 +42,50 @@
4343 $toolbar = '<div id="edittoolbar"></div>';
4444 // List of messages to be sent to the client for use in the toolbar
4545 $messages = array(
46 - /* Sections */
 46+ /* Main Section */
 47+ 'edittoolbar-tool-format-bold',
 48+ 'edittoolbar-tool-format-bold-example',
 49+ 'edittoolbar-tool-format-italic',
 50+ 'edittoolbar-tool-format-italic-example',
 51+ 'edittoolbar-tool-insert-ilink',
 52+ 'edittoolbar-tool-insert-ilink-example',
 53+ 'edittoolbar-tool-insert-xlink',
 54+ 'edittoolbar-tool-insert-xlink-example',
 55+ 'edittoolbar-tool-insert-file',
 56+ 'edittoolbar-tool-insert-file-pre',
 57+ 'edittoolbar-tool-insert-file-example',
 58+ 'edittoolbar-tool-insert-reference',
 59+ 'edittoolbar-tool-insert-reference-example',
 60+ 'edittoolbar-tool-insert-signature',
 61+ /* Formatting Section */
4762 'edittoolbar-section-format',
 63+ 'edittoolbar-tool-format-heading',
 64+ 'edittoolbar-tool-format-heading-1',
 65+ 'edittoolbar-tool-format-heading-2',
 66+ 'edittoolbar-tool-format-heading-3',
 67+ 'edittoolbar-tool-format-heading-4',
 68+ 'edittoolbar-tool-format-heading-5',
 69+ 'edittoolbar-tool-format-heading-example',
 70+ 'edittoolbar-group-format-list',
 71+ 'edittoolbar-tool-format-ulist',
 72+ 'edittoolbar-tool-format-ulist-example',
 73+ 'edittoolbar-tool-format-olist',
 74+ 'edittoolbar-tool-format-olist-example',
 75+ 'edittoolbar-group-format-size',
 76+ 'edittoolbar-tool-format-big',
 77+ 'edittoolbar-tool-format-big-example',
 78+ 'edittoolbar-tool-format-small',
 79+ 'edittoolbar-tool-format-small-example',
 80+ 'edittoolbar-tool-format-superscript',
 81+ 'edittoolbar-tool-format-superscript-example',
 82+ 'edittoolbar-tool-format-subscript',
 83+ 'edittoolbar-tool-format-subscript-example',
 84+ /* Insert Section */
4885 'edittoolbar-section-insert',
 86+ /* Special Characters Section */
4987 'edittoolbar-section-characters',
 88+ /* Help Section */
5089 'edittoolbar-section-help',
51 - /* Main Section */
52 - 'edittoolbar-format-bold',
53 - 'edittoolbar-format-bold-example',
54 - 'edittoolbar-format-italic',
55 - 'edittoolbar-format-italic-example',
56 - 'edittoolbar-insert-ilink',
57 - 'edittoolbar-insert-ilink-example',
58 - 'edittoolbar-insert-xlink',
59 - 'edittoolbar-insert-xlink-example',
60 - 'edittoolbar-insert-file',
61 - 'edittoolbar-insert-file-pre',
62 - 'edittoolbar-insert-file-example',
63 - 'edittoolbar-insert-reference',
64 - 'edittoolbar-insert-reference-example',
65 - 'edittoolbar-insert-signature',
66 - /* Formatting Section */
67 - 'edittoolbar-format-ulist',
68 - 'edittoolbar-format-ulist-example',
69 - 'edittoolbar-format-olist',
70 - 'edittoolbar-format-olist-example',
71 - 'edittoolbar-format-heading',
72 - 'edittoolbar-format-heading-1',
73 - 'edittoolbar-format-heading-2',
74 - 'edittoolbar-format-heading-3',
75 - 'edittoolbar-format-heading-4',
76 - 'edittoolbar-format-heading-5',
77 - 'edittoolbar-format-heading-example',
78 - 'edittoolbar-format-superscript',
79 - 'edittoolbar-format-superscript-example',
80 - 'edittoolbar-format-subscript',
81 - 'edittoolbar-format-subscript-example',
82 - 'edittoolbar-format-big',
83 - 'edittoolbar-format-big-example',
84 - 'edittoolbar-format-small',
85 - 'edittoolbar-format-small-example',
8690 );
8791 // Transforms messages into javascript object members
8892 foreach ( $messages as $i => $message ) {
Index: trunk/extensions/UsabilityInitiative/EditToolbar/EditToolbar.js
——@@ -126,11 +126,11 @@
127127 .attr( 'class', 'group' )
128128 .appendTo( $(this) );
129129 // Checks if a there's a label for this group
130 - if ( 'label' in section.groups[group] ) {
 130+ if ( 'label' in section.groups[group] || 'labelMsg' in section.groups[group] ) {
131131 groupDiv.append(
132132 $( '<div />' )
133133 .attr( 'class', 'label' )
134 - .text( ( section.groups[group].label || gM( section.groups[group].labelMsg ) ) )
 134+ .text( section.groups[group].label || gM( section.groups[group].labelMsg ) )
135135 )
136136 }
137137 // Creates generic action
——@@ -255,14 +255,14 @@
256256 'format': {
257257 tools: {
258258 'bold': {
259 - labelMsg: 'edittoolbar-format-bold',
 259+ labelMsg: 'edittoolbar-tool-format-bold',
260260 type: 'button',
261261 icon: 'format-bold.png',
262262 action: {
263263 type: 'encapsulate',
264264 options: {
265265 pre: "'''",
266 - periMsg: 'edittoolbar-format-bold-example',
 266+ periMsg: 'edittoolbar-tool-format-bold-example',
267267 post: "'''"
268268 }
269269 }
——@@ -271,14 +271,14 @@
272272 section: 'main',
273273 group: 'format',
274274 id: 'italic',
275 - labelMsg: 'edittoolbar-format-italic',
 275+ labelMsg: 'edittoolbar-tool-format-italic',
276276 type: 'button',
277277 icon: 'format-italic.png',
278278 action: {
279279 type: 'encapsulate',
280280 options: {
281281 pre: "''",
282 - periMsg: 'edittoolbar-format-italic-example',
 282+ periMsg: 'edittoolbar-tool-format-italic-example',
283283 post: "''"
284284 }
285285 }
——@@ -288,47 +288,47 @@
289289 'insert': {
290290 tools: {
291291 'xlink': {
292 - labelMsg: 'edittoolbar-insert-xlink',
 292+ labelMsg: 'edittoolbar-tool-insert-xlink',
293293 type: 'button',
294294 icon: 'insert-xlink.png',
295295 action: {
296296 type: 'encapsulate',
297297 options: {
298298 pre: "[",
299 - periMsg: 'edittoolbar-insert-xlink-example',
 299+ periMsg: 'edittoolbar-tool-insert-xlink-example',
300300 post: "]"
301301 }
302302 }
303303 },
304304 'ilink': {
305 - labelMsg: 'edittoolbar-insert-ilink',
 305+ labelMsg: 'edittoolbar-tool-insert-ilink',
306306 type: 'button',
307307 icon: 'insert-ilink.png',
308308 action: {
309309 type: 'encapsulate',
310310 options: {
311311 pre: "[[",
312 - periMsg: 'edittoolbar-insert-ilink-example',
 312+ periMsg: 'edittoolbar-tool-insert-ilink-example',
313313 post: "]]"
314314 }
315315 }
316316 },
317317 'file': {
318 - labelMsg: 'edittoolbar-insert-file',
 318+ labelMsg: 'edittoolbar-tool-insert-file',
319319 type: 'button',
320320 icon: 'insert-file.png',
321321 action: {
322322 type: 'encapsulate',
323323 options: {
324324 pre: "[[",
325 - preMsg: 'edittoolbar-insert-file-pre',
326 - periMsg: 'edittoolbar-insert-file-example',
 325+ preMsg: 'edittoolbar-tool-insert-file-pre',
 326+ periMsg: 'edittoolbar-tool-insert-file-example',
327327 post: "]]"
328328 }
329329 }
330330 },
331331 'reference': {
332 - labelMsg: 'edittoolbar-insert-reference',
 332+ labelMsg: 'edittoolbar-tool-insert-reference',
333333 filters: [ 'body.ns-subject' ],
334334 type: 'button',
335335 icon: 'insert-reference.png',
——@@ -336,13 +336,13 @@
337337 type: 'encapsulate',
338338 options: {
339339 pre: "<ref>",
340 - periMsg: 'edittoolbar-insert-reference-example',
 340+ periMsg: 'edittoolbar-tool-insert-reference-example',
341341 post: "</ref>"
342342 }
343343 }
344344 },
345345 'signature': {
346 - labelMsg: 'edittoolbar-insert-signature',
 346+ labelMsg: 'edittoolbar-tool-insert-signature',
347347 filters: [ 'body.ns-talk' ],
348348 type: 'button',
349349 icon: 'insert-signature.png',
——@@ -364,60 +364,60 @@
365365 'heading': {
366366 tools: {
367367 'heading': {
368 - labelMsg: 'edittoolbar-format-heading',
 368+ labelMsg: 'edittoolbar-tool-format-heading',
369369 type: 'select',
370370 list: {
371371 'heading-1' : {
372 - labelMsg: 'edittoolbar-format-heading-1',
 372+ labelMsg: 'edittoolbar-tool-format-heading-1',
373373 action: {
374374 type: 'encapsulate',
375375 options: {
376376 pre: "=",
377 - periMsg: 'edittoolbar-format-heading-example',
 377+ periMsg: 'edittoolbar-tool-format-heading-example',
378378 post: "="
379379 }
380380 }
381381 },
382382 'heading-2' : {
383 - labelMsg: 'edittoolbar-format-heading-2',
 383+ labelMsg: 'edittoolbar-tool-format-heading-2',
384384 action: {
385385 type: 'encapsulate',
386386 options: {
387387 pre: "==",
388 - periMsg: 'edittoolbar-format-heading-example',
 388+ periMsg: 'edittoolbar-tool-format-heading-example',
389389 post: "=="
390390 }
391391 }
392392 },
393393 'heading-3' : {
394 - labelMsg: 'edittoolbar-format-heading-3',
 394+ labelMsg: 'edittoolbar-tool-format-heading-3',
395395 action: {
396396 type: 'encapsulate',
397397 options: {
398398 pre: "===",
399 - periMsg: 'edittoolbar-format-heading-example',
 399+ periMsg: 'edittoolbar-tool-format-heading-example',
400400 post: "==="
401401 }
402402 }
403403 },
404404 'heading-4' : {
405 - labelMsg: 'edittoolbar-format-heading-4',
 405+ labelMsg: 'edittoolbar-tool-format-heading-4',
406406 action: {
407407 type: 'encapsulate',
408408 options: {
409409 pre: "====",
410 - periMsg: 'edittoolbar-format-heading-example',
 410+ periMsg: 'edittoolbar-tool-format-heading-example',
411411 post: "===="
412412 }
413413 }
414414 },
415415 'heading-5' : {
416 - labelMsg: 'edittoolbar-format-heading-5',
 416+ labelMsg: 'edittoolbar-tool-format-heading-5',
417417 action: {
418418 type: 'encapsulate',
419419 options: {
420420 pre: "=====",
421 - periMsg: 'edittoolbar-format-heading-example',
 421+ periMsg: 'edittoolbar-tool-format-heading-example',
422422 post: "====="
423423 }
424424 }
——@@ -427,30 +427,30 @@
428428 }
429429 },
430430 'list': {
431 - label: 'List',
 431+ labelMsg: 'edittoolbar-group-format-list',
432432 tools: {
433433 'ulist': {
434 - labelMsg: 'edittoolbar-format-ulist',
 434+ labelMsg: 'edittoolbar-tool-format-ulist',
435435 type: 'button',
436436 icon: 'format-ulist.png',
437437 action: {
438438 type: 'encapsulate',
439439 options: {
440440 pre: "* ",
441 - periMsg: 'edittoolbar-format-ulist-example',
 441+ periMsg: 'edittoolbar-tool-format-ulist-example',
442442 post: ""
443443 }
444444 }
445445 },
446446 'olist': {
447 - labelMsg: 'edittoolbar-format-olist',
 447+ labelMsg: 'edittoolbar-tool-format-olist',
448448 type: 'button',
449449 icon: 'format-olist.png',
450450 action: {
451451 type: 'encapsulate',
452452 options: {
453453 pre: "# ",
454 - periMsg: 'edittoolbar-format-olist-example',
 454+ periMsg: 'edittoolbar-tool-format-olist-example',
455455 post: ""
456456 }
457457 }
——@@ -458,56 +458,56 @@
459459 }
460460 },
461461 'size': {
462 - label: 'Size',
 462+ labelMsg: 'edittoolbar-group-format-size',
463463 tools: {
464464 'big': {
465 - labelMsg: 'edittoolbar-format-big',
 465+ labelMsg: 'edittoolbar-tool-format-big',
466466 type: 'button',
467467 icon: 'format-big.png',
468468 action: {
469469 type: 'encapsulate',
470470 options: {
471471 pre: "<big>",
472 - periMsg: 'edittoolbar-format-big-example',
 472+ periMsg: 'edittoolbar-tool-format-big-example',
473473 post: "</big>"
474474 }
475475 }
476476 },
477477 'small': {
478 - labelMsg: 'edittoolbar-format-small',
 478+ labelMsg: 'edittoolbar-tool-format-small',
479479 type: 'button',
480480 icon: 'format-small.png',
481481 action: {
482482 type: 'encapsulate',
483483 options: {
484484 pre: "<small>",
485 - periMsg: 'edittoolbar-format-small-example',
 485+ periMsg: 'edittoolbar-tool-format-small-example',
486486 post: "</small>"
487487 }
488488 }
489489 },
490490 'superscript': {
491 - labelMsg: 'edittoolbar-format-superscript',
 491+ labelMsg: 'edittoolbar-tool-format-superscript',
492492 type: 'button',
493493 icon: 'format-superscript.png',
494494 action: {
495495 type: 'encapsulate',
496496 options: {
497497 pre: "<super>",
498 - periMsg: 'edittoolbar-format-superscript-example',
 498+ periMsg: 'edittoolbar-tool-format-superscript-example',
499499 post: "</super>"
500500 }
501501 }
502502 },
503503 'subscript': {
504 - labelMsg: 'edittoolbar-format-subscript',
 504+ labelMsg: 'edittoolbar-tool-format-subscript',
505505 type: 'button',
506506 icon: 'format-subscript.png',
507507 action: {
508508 type: 'encapsulate',
509509 options: {
510510 pre: "<sub>",
511 - periMsg: 'edittoolbar-format-subscript-example',
 511+ periMsg: 'edittoolbar-tool-format-subscript-example',
512512 post: "</sub>"
513513 }
514514 }
Index: trunk/extensions/UsabilityInitiative/EditToolbar/EditToolbar.i18n.php
——@@ -15,45 +15,48 @@
1616 'edittoolbar' => 'Editing Toolbar',
1717 'edittoolbar-desc' => 'Edit page toolbar with enhanced usability',
1818 'edittoolbar-preference' => 'Enable enhanced editing toolbar',
19 - /* Sections */
 19+ /* Main Section */
 20+ 'edittoolbar-tool-format-bold' => 'Bold',
 21+ 'edittoolbar-tool-format-bold-example' => 'Bold text',
 22+ 'edittoolbar-tool-format-italic' => 'Italic',
 23+ 'edittoolbar-tool-format-italic-example' => 'Italic text',
 24+ 'edittoolbar-tool-insert-ilink' => 'Internal link',
 25+ 'edittoolbar-tool-insert-ilink-example' => 'Link title',
 26+ 'edittoolbar-tool-insert-xlink' => 'External link (remember http:// prefix)',
 27+ 'edittoolbar-tool-insert-xlink-example' => 'http://www.example.com link title',
 28+ 'edittoolbar-tool-insert-file' => 'Embedded file',
 29+ 'edittoolbar-tool-insert-file-pre' => '$1File:',
 30+ 'edittoolbar-tool-insert-file-example' => 'Example.jpg',
 31+ 'edittoolbar-tool-insert-reference' => 'Reference',
 32+ 'edittoolbar-tool-insert-reference-example' => 'Insert footnote text here',
 33+ 'edittoolbar-tool-insert-signature' => 'Signature and timestamp',
 34+ /* Formatting Section */
2035 'edittoolbar-section-format' => 'Format',
 36+ 'edittoolbar-tool-format-heading' => 'Heading',
 37+ 'edittoolbar-tool-format-heading-1' => 'Level 1',
 38+ 'edittoolbar-tool-format-heading-2' => 'Level 2',
 39+ 'edittoolbar-tool-format-heading-3' => 'Level 3',
 40+ 'edittoolbar-tool-format-heading-4' => 'Level 4',
 41+ 'edittoolbar-tool-format-heading-5' => 'Level 5',
 42+ 'edittoolbar-tool-format-heading-example' => 'Heading text',
 43+ 'edittoolbar-group-format-list' => 'List',
 44+ 'edittoolbar-tool-format-ulist' => 'Bulleted list',
 45+ 'edittoolbar-tool-format-ulist-example' => 'Bulleted list item',
 46+ 'edittoolbar-tool-format-olist' => 'Numbered list',
 47+ 'edittoolbar-tool-format-olist-example' => 'Numbered list item',
 48+ 'edittoolbar-group-format-size' => 'Size',
 49+ 'edittoolbar-tool-format-big' => 'Big',
 50+ 'edittoolbar-tool-format-big-example' => 'Big text',
 51+ 'edittoolbar-tool-format-small' => 'Small',
 52+ 'edittoolbar-tool-format-small-example' => 'Small text',
 53+ 'edittoolbar-tool-format-superscript' => 'Superscript',
 54+ 'edittoolbar-tool-format-superscript-example' => 'Superscript text',
 55+ 'edittoolbar-tool-format-subscript' => 'Subscript',
 56+ 'edittoolbar-tool-format-subscript-example' => 'Subscript text',
 57+ /* Insert Section */
2158 'edittoolbar-section-insert' => 'Insert',
 59+ /* Special Characters Section */
2260 'edittoolbar-section-characters' => 'Special Characters',
 61+ /* Help Section */
2362 'edittoolbar-section-help' => 'Help',
24 - 'edittoolbar-section-insert' => 'Insert',
25 - /* Main Section */
26 - 'edittoolbar-format-bold' => 'Bold',
27 - 'edittoolbar-format-bold-example' => 'Bold text',
28 - 'edittoolbar-format-italic' => 'Italic',
29 - 'edittoolbar-format-italic-example' => 'Italic text',
30 - 'edittoolbar-insert-ilink' => 'Internal link',
31 - 'edittoolbar-insert-ilink-example' => 'Link title',
32 - 'edittoolbar-insert-xlink' => 'External link (remember http:// prefix)',
33 - 'edittoolbar-insert-xlink-example' => 'http://www.example.com link title',
34 - 'edittoolbar-insert-file' => 'Embedded file',
35 - 'edittoolbar-insert-file-pre' => '$1File:',
36 - 'edittoolbar-insert-file-example' => 'Example.jpg',
37 - 'edittoolbar-insert-reference' => 'Reference',
38 - 'edittoolbar-insert-reference-example' => 'Insert footnote text here',
39 - 'edittoolbar-insert-signature' => 'Signature and timestamp',
40 - /* Formatting Section */
41 - 'edittoolbar-format-ulist' => 'Bulleted list',
42 - 'edittoolbar-format-ulist-example' => 'Bulleted list item',
43 - 'edittoolbar-format-olist' => 'Numbered list',
44 - 'edittoolbar-format-olist-example' => 'Numbered list item',
45 - 'edittoolbar-format-heading' => 'Heading',
46 - 'edittoolbar-format-heading-1' => 'Level 1',
47 - 'edittoolbar-format-heading-2' => 'Level 2',
48 - 'edittoolbar-format-heading-3' => 'Level 3',
49 - 'edittoolbar-format-heading-4' => 'Level 4',
50 - 'edittoolbar-format-heading-5' => 'Level 5',
51 - 'edittoolbar-format-heading-example' => 'Heading text',
52 - 'edittoolbar-format-superscript' => 'Superscript',
53 - 'edittoolbar-format-superscript-example' => 'Superscript text',
54 - 'edittoolbar-format-subscript' => 'Subscript',
55 - 'edittoolbar-format-subscript-example' => 'Subscript text',
56 - 'edittoolbar-format-big' => 'Big',
57 - 'edittoolbar-format-big-example' => 'Big text',
58 - 'edittoolbar-format-small' => 'Small',
59 - 'edittoolbar-format-small-example' => 'Small text',
6063 );

Status & tagging log