Localize strings for action menus
authortimothy_horton@apple.com <timothy_horton@apple.com@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Tue, 11 Nov 2014 19:14:07 +0000 (19:14 +0000)
committertimothy_horton@apple.com <timothy_horton@apple.com@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Tue, 11 Nov 2014 19:14:07 +0000 (19:14 +0000)
https://bugs.webkit.org/show_bug.cgi?id=138591
<rdar://problem/18815343>

Reviewed by Dan Bates.

* English.lproj/Localizable.strings:
Add a bunch of localizable strings.

* WebView/WebActionMenuController.mm:
(-[WebActionMenuController _createActionMenuItemForTag:withHitTestResult:]):
Make use of the new strings.

* UIProcess/mac/WKActionMenuController.mm:
(-[WKActionMenuController _showTextIndicator]):
(-[WKActionMenuController _hideTextIndicator]):
(-[WKActionMenuController _createActionMenuItemForTag:]):
Make use of the new strings.

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

Source/WebCore/ChangeLog
Source/WebCore/English.lproj/Localizable.strings
Source/WebKit/mac/ChangeLog
Source/WebKit/mac/WebView/WebActionMenuController.mm
Source/WebKit2/ChangeLog
Source/WebKit2/UIProcess/mac/WKActionMenuController.mm

index a91bbc623a65fd4bb467186949af9e76685cd3fe..5d64af5a9cca157df80bff822611539dae204a72 100644 (file)
@@ -1,3 +1,14 @@
+2014-11-11  Tim Horton  <timothy_horton@apple.com>
+
+        Localize strings for action menus
+        https://bugs.webkit.org/show_bug.cgi?id=138591
+        <rdar://problem/18815343>
+
+        Reviewed by Dan Bates.
+
+        * English.lproj/Localizable.strings:
+        Add a bunch of localizable strings.
+
 2014-11-11  Myles C. Maxfield  <mmaxfield@apple.com>
 
         Cleanup from r175379
index 77dd707147c5cf2ec2bd50102232045d9aaabb6f..011f41faf3ed361f631f0d10241b94d1fa9eb3a0 100644 (file)
 /* accessibility label for video element controller */
 "video playback" = "video playback";
 
+/* "Open Link in Default Browser" action menu item */
+"Open (action menu item)" = "Open";
+
+/* "Preview Link" action menu item */
+"Preview (action menu item)" = "Preview";
+
+/* "Add to Reading List" action menu item */
+"Add to Reading List (action menu item)" = "Add to Reading List";
+
+/* "Copy" action menu item in image menu */
+"Copy (image action menu item)" = "Copy";
+
+/* "Add to Photos" action menu item */
+"Add to Photos (action menu item)" = "Add to Photos";
+
+/* "Save to Downloads" action menu item in image menu */
+"Save to Downloads (image action menu item)" = "Save to Downloads";
+
+/* "Share" action menu item in image menu */
+"Share (image action menu item)" = "Share";
+
+/* "Copy" action menu item in text menu */
+"Copy (text action menu item)" = "Copy";
+
+/* "Look Up" action menu item */
+"Look Up (action menu item)" = "Look Up";
+
+/* "Paste" action menu item */
+"Paste (action menu item)" = "Paste";
+
+/* "Suggestions" action menu item */
+"Suggestions (action menu item)" = "Suggestions";
+
+/* "Copy" action menu item in video menu */
+"Copy (video action menu item)" = "Copy";
+
+/* "Share" action menu item in video menu */
+"Share (video action menu item)" = "Share";
+
+/* "Save to Downloads" action menu item in video menu */
+"Save to Downloads (video action menu item)" = "Save to Downloads";
index 99b71ae62621ce1867dfaedd9d8ffcf0de1f82a6..45d6fb366f0089e69c98b6f89709a05898e563dd 100644 (file)
@@ -1,3 +1,15 @@
+2014-11-11  Tim Horton  <timothy_horton@apple.com>
+
+        Localize strings for action menus
+        https://bugs.webkit.org/show_bug.cgi?id=138591
+        <rdar://problem/18815343>
+
+        Reviewed by Dan Bates.
+
+        * WebView/WebActionMenuController.mm:
+        (-[WebActionMenuController _createActionMenuItemForTag:withHitTestResult:]):
+        Make use of the new strings.
+
 2014-11-11  Myles C. Maxfield  <mmaxfield@apple.com>
 
         Cleanup from r175379
index 1b4965d409fc23ebd8fe6f232f6dfc7672bc20c8..c019b250d37c1ae46a826f3d4ea47a786b90d85e 100644 (file)
@@ -441,49 +441,48 @@ static DictionaryPopupInfo performDictionaryLookupForRange(Frame* frame, Range&
     NSImage *image = nil;
     id representedObject = nil;
 
-    // FIXME: We should localize these strings.
     switch (tag) {
     case WebActionMenuItemTagOpenLinkInDefaultBrowser:
         selector = @selector(_openURLFromActionMenu:);
-        title = @"Open";
+        title = WEB_UI_STRING_KEY("Open", "Open (action menu item)", "action menu item");
         image = webKitBundleImageNamed(@"OpenInNewWindowTemplate");
         representedObject = [hitTestResult objectForKey:WebElementLinkURLKey];
         break;
 
     case WebActionMenuItemTagPreviewLink:
         selector = @selector(_quickLookURLFromActionMenu:);
-        title = @"Preview";
+        title = WEB_UI_STRING_KEY("Preview", "Preview (action menu item)", "action menu item");
         image = [NSImage imageNamed:@"NSActionMenuQuickLook"];
         representedObject = hitTestResult;
         break;
 
     case WebActionMenuItemTagAddLinkToSafariReadingList:
         selector = @selector(_addToReadingListFromActionMenu:);
-        title = @"Add to Safari Reading List";
+        title = WEB_UI_STRING_KEY("Add to Reading List", "Add to Reading List (action menu item)", "action menu item");
         image = [NSImage imageNamed:NSImageNameBookmarksTemplate];
         representedObject = [hitTestResult objectForKey:WebElementLinkURLKey];
         break;
 
     case WebActionMenuItemTagCopyText:
         selector = @selector(_copySelection:);
-        title = @"Copy";
+        title = WEB_UI_STRING_KEY("Copy", "Copy (text action menu item)", "text action menu item");
         image = [NSImage imageNamed:@"NSActionMenuCopy"];
         break;
 
     case WebActionMenuItemTagLookupText:
         selector = @selector(_lookupText:);
-        title = @"Look Up";
+        title = WEB_UI_STRING_KEY("Look Up", "Look Up (action menu item)", "action menu item");
         image = [NSImage imageNamed:@"NSActionMenuLookup"];
         break;
 
     case WebActionMenuItemTagPaste:
         selector = @selector(_paste:);
-        title = @"Paste";
+        title = WEB_UI_STRING_KEY("Paste", "Paste (action menu item)", "action menu item");
         image = [NSImage imageNamed:@"NSActionMenuPaste"];
         break;
 
     case WebActionMenuItemTagTextSuggestions:
-        title = @"Suggestions";
+        title = WEB_UI_STRING_KEY("Suggestions", "Suggestions (action menu item)", "action menu item");
         image = [NSImage imageNamed:@"NSActionMenuSpelling"];
         break;
 
index 12de68f53c09eb361b958df61036ce920b981cb1..ce61ee05c9a2cad1d37029c111aaebde4288bc1c 100644 (file)
@@ -1,3 +1,17 @@
+2014-11-11  Tim Horton  <timothy_horton@apple.com>
+
+        Localize strings for action menus
+        https://bugs.webkit.org/show_bug.cgi?id=138591
+        <rdar://problem/18815343>
+
+        Reviewed by Dan Bates.
+
+        * UIProcess/mac/WKActionMenuController.mm:
+        (-[WKActionMenuController _showTextIndicator]):
+        (-[WKActionMenuController _hideTextIndicator]):
+        (-[WKActionMenuController _createActionMenuItemForTag:]):
+        Make use of the new strings.
+
 2014-11-11  Myles C. Maxfield  <mmaxfield@apple.com>
 
         Cleanup from r175379
index 89dfad1a532d95a515bd9f2a985a1745625e26eb..fcd253b3a3935f773f3090055bc6954eae167ecb 100644 (file)
@@ -44,6 +44,7 @@
 #import <ImageKit/ImageKit.h>
 #import <WebCore/DataDetectorsSPI.h>
 #import <WebCore/GeometryUtilities.h>
+#import <WebCore/LocalizedStrings.h>
 #import <WebCore/NSSharingServiceSPI.h>
 #import <WebCore/NSSharingServicePickerSPI.h>
 #import <WebCore/NSViewSPI.h>
@@ -723,88 +724,87 @@ static NSString *pathToPhotoOnDisk(NSString *suggestedFilename)
     NSString *title = nil;
     NSImage *image = nil;
 
-    // FIXME: These titles need to be localized.
     switch (tag) {
     case kWKContextActionItemTagOpenLinkInDefaultBrowser:
         selector = @selector(_openURLFromActionMenu:);
-        title = @"Open";
+        title = WEB_UI_STRING_KEY("Open", "Open (action menu item)", "action menu item");
         image = webKitBundleImageNamed(@"OpenInNewWindowTemplate");
         break;
 
 #if WK_API_ENABLED
     case kWKContextActionItemTagPreviewLink:
         selector = @selector(_keepPreviewOpenFromActionMenu:);
-        title = @"Preview";
+        title = WEB_UI_STRING_KEY("Preview", "Preview (action menu item)", "action menu item");
         image = [NSImage imageNamed:@"NSActionMenuQuickLook"];
         break;
 #endif
 
     case kWKContextActionItemTagAddLinkToSafariReadingList:
         selector = @selector(_addToReadingListFromActionMenu:);
-        title = @"Add to Safari Reading List";
+        title = WEB_UI_STRING_KEY("Add to Reading List", "Add to Reading List (action menu item)", "action menu item");
         image = [NSImage imageNamed:NSImageNameBookmarksTemplate];
         break;
 
     case kWKContextActionItemTagCopyImage:
         selector = @selector(_copyImage:);
-        title = @"Copy";
+        title = WEB_UI_STRING_KEY("Copy", "Copy (image action menu item)", "image action menu item");
         image = [NSImage imageNamed:@"NSActionMenuCopy"];
         break;
 
     case kWKContextActionItemTagAddImageToPhotos:
         selector = @selector(_addImageToPhotos:);
-        title = @"Add to Photos";
+        title = WEB_UI_STRING_KEY("Add to Photos", "Add to Photos (action menu item)", "action menu item");
         image = [NSImage imageNamed:@"NSActionMenuAddToPhotos"];
         break;
 
     case kWKContextActionItemTagSaveImageToDownloads:
         selector = @selector(_saveImageToDownloads:);
-        title = @"Save to Downloads";
+        title = WEB_UI_STRING_KEY("Save to Downloads", "Save to Downloads (image action menu item)", "image action menu item");
         image = [NSImage imageNamed:@"NSActionMenuSaveToDownloads"];
         break;
 
     case kWKContextActionItemTagShareImage:
-        title = @"Share";
+        title = WEB_UI_STRING_KEY("Share (image action menu item)", "Share (image action menu item)", "image action menu item");
         image = [NSImage imageNamed:@"NSActionMenuShare"];
         break;
 
     case kWKContextActionItemTagCopyText:
         selector = @selector(_copySelection:);
-        title = @"Copy";
+        title = WEB_UI_STRING_KEY("Copy", "Copy (text action menu item)", "text action menu item");
         image = [NSImage imageNamed:@"NSActionMenuCopy"];
         break;
 
     case kWKContextActionItemTagLookupText:
         selector = @selector(_lookupText:);
-        title = @"Look Up";
+        title = WEB_UI_STRING_KEY("Look Up", "Look Up (action menu item)", "action menu item");
         image = [NSImage imageNamed:@"NSActionMenuLookup"];
         break;
 
     case kWKContextActionItemTagPaste:
         selector = @selector(_paste:);
-        title = @"Paste";
+        title = WEB_UI_STRING_KEY("Paste", "Paste (action menu item)", "action menu item");
         image = [NSImage imageNamed:@"NSActionMenuPaste"];
         break;
 
     case kWKContextActionItemTagTextSuggestions:
-        title = @"Suggestions";
+        title = WEB_UI_STRING_KEY("Suggestions", "Suggestions (action menu item)", "action menu item");
         image = [NSImage imageNamed:@"NSActionMenuSpelling"];
         break;
 
     case kWKContextActionItemTagCopyVideoURL:
         selector = @selector(_copyVideoURL:);
-        title = @"Copy";
+        title = WEB_UI_STRING_KEY("Copy", "Copy (video action menu item)", "video action menu item");
         image = [NSImage imageNamed:@"NSActionMenuCopy"];
         break;
 
     case kWKContextActionItemTagSaveVideoToDownloads:
         selector = @selector(_saveVideoToDownloads:);
-        title = @"Save to Downloads";
+        title = WEB_UI_STRING_KEY("Share", "Share (video action menu item)", "video action menu item");
         image = [NSImage imageNamed:@"NSActionMenuSaveToDownloads"];
         break;
 
     case kWKContextActionItemTagShareVideo:
-        title = @"Share";
+        title = WEB_UI_STRING_KEY("Save to Downloads", "Save to Downloads (video action menu item)", "video action menu item");
         image = [NSImage imageNamed:@"NSActionMenuShare"];
         break;