r99683 MediaWiki - Code Review archive

Repository:MediaWiki
Revision:< r99682‎ | r99683 | r99684 >
Date:14:40, 13 October 2011
Author:junaidpv
Status:ok (Comments)
Tags:
Comment:
Narayam: Add Tamil Bamini keyboard layout implementation. For bug 31665.
Modified paths:
  • /trunk/extensions/Narayam/Narayam.i18n.php (modified) (history)
  • /trunk/extensions/Narayam/Narayam.php (modified) (history)
  • /trunk/extensions/Narayam/js/ext.narayam.rules.ta-bamini.js (added) (history)

Diff [purge]

Index: trunk/extensions/Narayam/Narayam.i18n.php
——@@ -43,6 +43,7 @@
4444 'narayam-si-wijesekara' => 'Sinhala Wijesekara',
4545 'narayam-ta99' => 'Tamil99',
4646 'narayam-ta' => 'Tamil Transliteration',
 47+ 'narayam-ta-bamini' => 'Tamil Bamini',
4748 'narayam-te-inscript' => 'Telugu InScript',
4849 'narayam-bn-avro' => 'Bengali Avro',
4950 'narayam-bn-inscript' => 'Bengali InScript',
Index: trunk/extensions/Narayam/js/ext.narayam.rules.ta-bamini.js
——@@ -0,0 +1,85 @@
 2+/**
 3+ * Regular expression rules table for Tamil Bamini Keyboard layout.
 4+ * @author Junaid P V ([[user:Junaidpv]])
 5+ * @date 2011-10-13
 6+ * @credits http://help.keymandesktop.com/keyboards/keyboard_ekwbamuni.php
 7+ * License: GPLv3, CC-BY-SA 3.0
 8+ */
 9+
 10+var rules = [
 11+['`', '', 'ஹ'],
 12+['q', '', 'ங'],
 13+['w', '', 'ற'],
 14+['e', '', 'ந'],
 15+['r', '', 'ச'],
 16+['t', '', 'வ'],
 17+['y', '', 'ல'],
 18+['u', '', 'ர'],
 19+//['i', '', ''],
 20+['o', '', 'ழ'],
 21+['p', '', 'ி'],
 22+['\\[', '', 'ஜ'],
 23+['\\]', '', 'ஸ'],
 24+['\\\\', '', 'ஷ'],
 25+['a', '', 'ய'],
 26+['s', '', 'ள'],
 27+['d', '', 'ன'],
 28+['f', '', 'க'],
 29+['g', '', 'ப'],
 30+['h', '', 'ா'],
 31+['j', '', 'த'],
 32+['k', '', 'ம'],
 33+['l', '', 'ட'],
 34+[';', '', '்'],
 35+['z', '', 'ண'],
 36+['x', '', 'ஒ'],
 37+['c', '', 'உ'],
 38+['v', '', 'எ'],
 39+['b', '', 'டி'],
 40+['m', '', 'அ'],
 41+[',', '', 'இ'],
 42+['/', '', 'ஃ'],
 43+
 44+['@', '', ';'],
 45+['#', '', 'சூ'],
 46+['\\$', '', 'கூ'],
 47+['%', '', 'மூ'],
 48+['\\^', '', 'டூ'],
 49+['&', '', 'ரூ'],
 50+['Q', '', 'ஞ'],
 51+['W', '', 'று'],
 52+['E', '', 'நு'],
 53+['R', '', 'சு'],
 54+['T', '', 'வு'],
 55+['Y', '', 'லு'],
 56+['U', '', 'ரு'],
 57+['I', '', 'ஐ'],
 58+['O', '', 'ழு'],
 59+['P', '', 'ீ'],
 60+['\\|', '', "'"],
 61+['A', '', 'யு'],
 62+['S', '', 'ளு'],
 63+['D', '', 'னு'],
 64+['F', '', 'கு'],
 65+['G', '', 'பு'],
 66+['H', '', 'ர்'],
 67+['J', '', 'து'],
 68+['K', '', 'மு'],
 69+['L', '', 'டு'],
 70+['Z', '', 'ணு'],
 71+['X', '', 'ஓ'],
 72+['C', '', 'ஊ'],
 73+['V', '', 'ஏ'],
 74+['B', '', 'டீ'],
 75+['M', '', 'ஆ'],
 76+['\\<', '', 'ஈ'],
 77+['\\>', '', ',']
 78+];
 79+
 80+jQuery.narayam.addScheme( 'ta-bamini', {
 81+ 'namemsg': 'narayam-ta-bamini',
 82+ 'extended_keyboard': false,
 83+ 'lookbackLength': 0,
 84+ 'keyBufferLength': 0,
 85+ 'rules': rules
 86+} );
Index: trunk/extensions/Narayam/Narayam.php
——@@ -94,6 +94,7 @@
9595 'ta' => array(
9696 'ta' => 'ext.narayam.rules.ta',
9797 'ta99' => 'ext.narayam.rules.ta99',
 98+ 'ta-bamini' => 'ext.narayam.rules.ta-bamini',
9899 ),
99100 'te' => array(
100101 'te-inscript' => 'ext.narayam.rules.te-inscript',
——@@ -272,6 +273,11 @@
273274 'messages' => array( 'narayam-ta99' ),
274275 'dependencies' => 'ext.narayam.rules.ta', // make sure ta99 loads after ta
275276 );
 277+$wgResourceModules['ext.narayam.rules.ta-bamini'] = $narayamTpl + array(
 278+ 'scripts' => 'js/ext.narayam.rules.ta-bamini.js',
 279+ 'messages' => array( 'narayam-ta-bamini' ),
 280+ 'dependencies' => 'ext.narayam.rules.ta', // make sure ta99 loads after ta
 281+);
276282 $wgResourceModules['ext.narayam.rules.te-inscript'] = $narayamTpl + array(
277283 'scripts' => 'js/ext.narayam.rules.te-inscript.js',
278284 'messages' => array( 'narayam-te-inscript' ),

Follow-up revisions

RevisionCommit summaryAuthorDate
r102305MFT r99387 r99395 r99456 r99457 r99682 r99683 r100013 r100024 r100108 r100212...nikerabbit18:21, 7 November 2011

Comments

#Comment by Santhosh.thottingal (talk | contribs)   03:27, 14 October 2011

Srikanth said there are differences in the reference used for this keymap (Refer c3 in Bug 31665). So removing i18ndeploy tag.

Status & tagging log