REGRESSION (r129738): Calendar picker is too wide when the input is rtl
authorkeishi@webkit.org <keishi@webkit.org@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Wed, 10 Oct 2012 11:29:49 +0000 (11:29 +0000)
committerkeishi@webkit.org <keishi@webkit.org@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Wed, 10 Oct 2012 11:29:49 +0000 (11:29 +0000)
https://bugs.webkit.org/show_bug.cgi?id=98881

.:

Reviewed by kent Tamura.

* ManualTests/forms/calendar-picker.html: Added isCalendarRTL parameters.

Source/WebCore:

Reviewed by Kent Tamura.

Calendar picker should use param.isCalendarRTL instead of param.isRTL but some were left.

No new tests. Covered by ManualTests/forms/calendar-picker.html.

* Resources/pagepopups/calendarPicker.js:
(CalendarPicker.prototype.fixWindowSize): Calendar picker should use param.isCalendarRTL instead of param.isRTL.
(DaysTable.prototype._handleKey): Ditto.

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@130888 268f45cc-cd09-0410-ab3c-d52691b4dbfc

ChangeLog
ManualTests/forms/calendar-picker.html
Source/WebCore/ChangeLog
Source/WebCore/Resources/pagepopups/calendarPicker.js

index fbe5288..9e17da9 100644 (file)
--- a/ChangeLog
+++ b/ChangeLog
@@ -1,3 +1,12 @@
+2012-10-10  Keishi Hattori  <keishi@webkit.org>
+
+        REGRESSION (r129738): Calendar picker is too wide when the input is rtl
+        https://bugs.webkit.org/show_bug.cgi?id=98881
+
+        Reviewed by kent Tamura.
+
+        * ManualTests/forms/calendar-picker.html: Added isCalendarRTL parameters.
+
 2012-10-10  Christophe Dumez  <christophe.dumez@intel.com>
 
         [EFL][WK2] Port MiniBrowser to Elementary
index 1f94ee4..be243ca 100644 (file)
@@ -64,7 +64,8 @@ var japaneseArguments = {
 };
 var arabicArguments = {
     locale: 'ar',
-    isRTL: true,
+    isRTL: false,
+    isCalendarRTL: true,
     monthLabels : ['يناير', 'فبراير', 'مارس', 'أبريل', 'مايو', 'يونيو',
                    'يوليو', 'أغسطس', 'سبتمبر', 'أكتوبر', 'نوفمبر', 'ديسمبر'],
     dayLabels : ['ح', 'ن', 'ث', 'ر', 'خ', 'ج', 'س'],
@@ -158,6 +159,7 @@ var longDatalistArguments = {
 var arabicDatalistArguments = {
     locale: 'ar',
     isRTL: true,
+    isCalendarRTL: true,
     monthLabels : ['يناير', 'فبراير', 'مارس', 'أبريل', 'مايو', 'يونيو',
                    'يوليو', 'أغسطس', 'سبتمبر', 'أكتوبر', 'نوفمبر', 'ديسمبر'],
     dayLabels : ['ح', 'ن', 'ث', 'ر', 'خ', 'ج', 'س'],
@@ -180,6 +182,7 @@ var arabicDatalistArguments = {
 var arabicLongDatalistArguments = {
     locale: 'ar',
     isRTL: true,
+    isCalendarRTL: true,
     monthLabels : ['يناير', 'فبراير', 'مارس', 'أبريل', 'مايو', 'يونيو',
                    'يوليو', 'أغسطس', 'سبتمبر', 'أكتوبر', 'نوفمبر', 'ديسمبر'],
     dayLabels : ['ح', 'ن', 'ث', 'ر', 'خ', 'ج', 'س'],
index 7ccdde8..c68cebf 100644 (file)
@@ -1,3 +1,18 @@
+2012-10-10  Keishi Hattori  <keishi@webkit.org>
+
+        REGRESSION (r129738): Calendar picker is too wide when the input is rtl
+        https://bugs.webkit.org/show_bug.cgi?id=98881
+
+        Reviewed by Kent Tamura.
+
+        Calendar picker should use param.isCalendarRTL instead of param.isRTL but some were left.
+
+        No new tests. Covered by ManualTests/forms/calendar-picker.html.
+
+        * Resources/pagepopups/calendarPicker.js:
+        (CalendarPicker.prototype.fixWindowSize): Calendar picker should use param.isCalendarRTL instead of param.isRTL.
+        (DaysTable.prototype._handleKey): Ditto.
+
 2012-10-10  Alexander Pavlov  <apavlov@chromium.org>
 
         Web Inspector: Semantically incorrect CSS rules result in broken source code data
index b9bd081..c2d3deb 100644 (file)
@@ -375,7 +375,7 @@ CalendarPicker.prototype.fixWindowSize = function() {
     var DaysAreaContainerBorder = 1;
     var yearMonthEnd;
     var daysAreaEnd;
-    if (global.params.isRTL) {
+    if (global.params.isCalendarRTL) {
         var startOffset = this._element.offsetLeft + this._element.offsetWidth;
         yearMonthEnd = startOffset - yearMonthRightElement.offsetLeft;
         daysAreaEnd = startOffset - (daysAreaElement.offsetLeft + daysAreaElement.offsetWidth) + maxCellWidth * 7 + DaysAreaContainerBorder;
@@ -1089,7 +1089,7 @@ DaysTable.prototype._handleKey = function(event) {
         return;
     }
 
-    if (key == (global.params.isRTL ? "Right" : "Left")) {
+    if (key == (global.params.isCalendarRTL ? "Right" : "Left")) {
         if (x == 0) {
             if (y == 0) {
                 if (!this._maybeSetPreviousMonth())
@@ -1111,7 +1111,7 @@ DaysTable.prototype._handleKey = function(event) {
             y--;
         this.updateSelection(event, x, y);
 
-    } else if (key == (global.params.isRTL ? "Left" : "Right")) {
+    } else if (key == (global.params.isCalendarRTL ? "Left" : "Right")) {
         if (x == 6) {
             if (y == DaysTable._Weeks - 1) {
                 if (!this._maybeSetNextMonth())