Contextual menu Hide and Show Link Previews should not have a symbol
authordino@apple.com <dino@apple.com@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Mon, 12 Aug 2019 22:23:03 +0000 (22:23 +0000)
committerdino@apple.com <dino@apple.com@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Mon, 12 Aug 2019 22:23:03 +0000 (22:23 +0000)
https://bugs.webkit.org/show_bug.cgi?id=200645
<rdar://problem/54129647>

Reviewed by Wenson Hsieh.

Don't use an image on the UIMenuItem.

* UIProcess/API/Cocoa/_WKElementAction.mm:
(+[_WKElementAction imageForElementActionType:]): Return nil for Show/Hide Link Previews.

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

Source/WebKit/ChangeLog
Source/WebKit/UIProcess/API/Cocoa/_WKElementAction.mm

index e196646..e9f7d9e 100644 (file)
@@ -1,3 +1,16 @@
+2019-08-12  Dean Jackson  <dino@apple.com>
+
+        Contextual menu Hide and Show Link Previews should not have a symbol
+        https://bugs.webkit.org/show_bug.cgi?id=200645
+        <rdar://problem/54129647>
+
+        Reviewed by Wenson Hsieh.
+
+        Don't use an image on the UIMenuItem.
+
+        * UIProcess/API/Cocoa/_WKElementAction.mm:
+        (+[_WKElementAction imageForElementActionType:]): Return nil for Show/Hide Link Previews.
+
 2019-08-12  Chris Dumez  <cdumez@apple.com>
 
         Unreviewed, add missing WTF::initializeMainThread() call to fix some crashes on the bots after r248533.
index e3397b4..4b5ff1b 100644 (file)
@@ -240,7 +240,7 @@ static void addToReadingList(NSURL *targetURL, NSString *title)
     case _WKElementActionTypeDownload:
         return [UIImage systemImageNamed:@"arrow.down.circle"];
     case _WKElementActionToggleShowLinkPreviews:
-        return [UIImage systemImageNamed:@"safari"];
+        return nil; // Intentionally empty.
     }
 }