[Extra zoom mode] Fix some localizable strings after r229878
authorwenson_hsieh@apple.com <wenson_hsieh@apple.com@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Sat, 24 Mar 2018 00:33:52 +0000 (00:33 +0000)
committerwenson_hsieh@apple.com <wenson_hsieh@apple.com@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Sat, 24 Mar 2018 00:33:52 +0000 (00:33 +0000)
https://bugs.webkit.org/show_bug.cgi?id=183963

Reviewed by Tim Horton.

The WEB_UI_STRING macro was incorrectly used for certain localized strings pertaining to extra zoom mode. To
correct this, make the "Done" string use WEB_UI_STRING with actual UI-facing text; since day, year and month
labels in the date picker are less generalizable to other UI, leave these as unique keys, but change them to
use WEB_UI_STRING_KEY instead.

Additionally, remove now-unused localizable strings for the text input view controller.

* English.lproj/Localizable.strings:
* platform/LocalizedStrings.cpp:
(WebCore::formControlDoneButtonTitle):
(WebCore::datePickerDayLabelTitle):
(WebCore::datePickerMonthLabelTitle):
(WebCore::datePickerYearLabelTitle):

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

Source/WebCore/ChangeLog
Source/WebCore/English.lproj/Localizable.strings
Source/WebCore/platform/LocalizedStrings.cpp
Source/WebCore/platform/LocalizedStrings.h

index 679683f..d5b5d76 100644 (file)
@@ -1,3 +1,24 @@
+2018-03-23  Wenson Hsieh  <wenson_hsieh@apple.com>
+
+        [Extra zoom mode] Fix some localizable strings after r229878
+        https://bugs.webkit.org/show_bug.cgi?id=183963
+
+        Reviewed by Tim Horton.
+
+        The WEB_UI_STRING macro was incorrectly used for certain localized strings pertaining to extra zoom mode. To
+        correct this, make the "Done" string use WEB_UI_STRING with actual UI-facing text; since day, year and month
+        labels in the date picker are less generalizable to other UI, leave these as unique keys, but change them to
+        use WEB_UI_STRING_KEY instead.
+
+        Additionally, remove now-unused localizable strings for the text input view controller.
+
+        * English.lproj/Localizable.strings:
+        * platform/LocalizedStrings.cpp:
+        (WebCore::formControlDoneButtonTitle):
+        (WebCore::datePickerDayLabelTitle):
+        (WebCore::datePickerMonthLabelTitle):
+        (WebCore::datePickerYearLabelTitle):
+
 2018-03-23  Chris Dumez  <cdumez@apple.com>
 
         NetworkStateNotifier::updateStateWithoutNotifying() is inefficient
index a852d99..361bc52 100644 (file)
 "Cut (Undo action name)" = "Cut";
 
 /* Day label in date picker */
-"Day label in date picker" = "Day label in date picker";
+"Day (Date picker for extra zoom mode)" = "Day";
 
 /* Fullscreen Deceptive Website Warning Sheet Title */
 "Deceptive Website Warning" = "Deceptive Website Warning";
 "Dictation (Undo action name)" = "Dictation";
 
 /* Title of the Done button for zoomed form controls. */
-"Done (extra zoomed form controls)" = "Done (extra zoomed form controls)";
+"Done" = "Done";
 
 /* Download Audio To Disk context menu item */
 "Download Audio" = "Download Audio";
 "Missing Plug-in" = "Missing Plug-in";
 
 /* Month label in date picker */
-"Month label in date picker" = "Month label in date picker";
+"Month (Date picker for extra zoom mode)" = "Month";
 
 /* Media Mute context menu item */
 "Mute" = "Mute";
 /* Title of the label to show on a snapshotted plug-in */
 "Snapshotted Plug-In" = "Snapshotted Plug-In";
 
-/* Title of the dictation button for zoomed form controls. */
-"Speak" = "Speak";
-
 /* Speech context sub-menu item */
 "Speech" = "Speech";
 
 /* WebKitErrorInternal description */
 "WebKit encountered an internal error" = "WebKit encountered an internal error";
 
-/* Title of the writing button for zoomed form controls. */
-"Write" = "Write";
-
 /* Year label in date picker */
-"Year label in date picker" = "Year label in date picker";
+"Year (Date picker for extra zoom mode)" = "Year";
 
 /* message in authentication panel */
 "Your login information will be sent securely." = "Your login information will be sent securely.";
index cf02ade..41ed394 100644 (file)
@@ -1038,7 +1038,7 @@ String webCryptoMasterKeyKeychainComment()
 
 String formControlDoneButtonTitle()
 {
-    return WEB_UI_STRING("Done (extra zoomed form controls)", "Title of the Done button for zoomed form controls.");
+    return WEB_UI_STRING("Done", "Title of the Done button for zoomed form controls.");
 }
 
 String formControlCancelButtonTitle()
@@ -1061,29 +1061,19 @@ String formControlSearchButtonTitle()
     return WEB_UI_STRING("Search", "Title of the Search button for zoomed form controls.");
 }
 
-String textInputModeWriteButtonTitle()
-{
-    return WEB_UI_STRING("Write", "Title of the writing button for zoomed form controls.");
-}
-
-String textInputModeSpeechButtonTitle()
-{
-    return WEB_UI_STRING("Speak", "Title of the dictation button for zoomed form controls.");
-}
-
 String datePickerDayLabelTitle()
 {
-    return WEB_UI_STRING("Day label in date picker", "Day label in date picker");
+    return WEB_UI_STRING_KEY("Day", "Day (Date picker for extra zoom mode)", "Day label in date picker");
 }
 
 String datePickerMonthLabelTitle()
 {
-    return WEB_UI_STRING("Month label in date picker", "Month label in date picker");
+    return WEB_UI_STRING_KEY("Month", "Month (Date picker for extra zoom mode)", "Month label in date picker");
 }
 
 String datePickerYearLabelTitle()
 {
-    return WEB_UI_STRING("Year label in date picker", "Year label in date picker");
+    return WEB_UI_STRING_KEY("Year", "Year (Date picker for extra zoom mode)", "Year label in date picker");
 }
 
 #endif
index 42bba25..b714a8e 100644 (file)
@@ -322,8 +322,6 @@ namespace WebCore {
     WEBCORE_EXPORT String formControlHideButtonTitle();
     WEBCORE_EXPORT String formControlGoButtonTitle();
     WEBCORE_EXPORT String formControlSearchButtonTitle();
-    WEBCORE_EXPORT String textInputModeWriteButtonTitle();
-    WEBCORE_EXPORT String textInputModeSpeechButtonTitle();
     WEBCORE_EXPORT String datePickerDayLabelTitle();
     WEBCORE_EXPORT String datePickerMonthLabelTitle();
     WEBCORE_EXPORT String datePickerYearLabelTitle();