From 2f8bee998ec6cee95493776787fc983488ae52a2 Mon Sep 17 00:00:00 2001 From: Brent Simmons Date: Tue, 12 Feb 2019 22:56:44 -0800 Subject: [PATCH] Place the comment in the right place in DetailWebView. --- .../MainWindow/Detail/DetailWebView.swift | 16 ++++++++-------- 1 file changed, 8 insertions(+), 8 deletions(-) diff --git a/NetNewsWire/MainWindow/Detail/DetailWebView.swift b/NetNewsWire/MainWindow/Detail/DetailWebView.swift index 1ce082904..ff450b7e9 100644 --- a/NetNewsWire/MainWindow/Detail/DetailWebView.swift +++ b/NetNewsWire/MainWindow/Detail/DetailWebView.swift @@ -9,19 +9,17 @@ import AppKit import WebKit -// There’s no API for affecting a WKWebView’s contextual menu. -// (WebView had API for this.) -// -// This a minor hack. It hides unwanted menu items. -// The menu item identifiers are not documented anywhere; -// they could change, and this code would need updating. - final class DetailWebView: WKWebView { // MARK: NSView override func willOpenMenu(_ menu: NSMenu, with event: NSEvent) { - + // There’s no API for affecting a WKWebView’s contextual menu. + // (WebView had API for this.) + // + // This a minor hack. It hides unwanted menu items. + // The menu item identifiers are not documented anywhere; + // they could change, and this code would need updating. for menuItem in menu.items { if shouldHideMenuItem(menuItem) { menuItem.isHidden = true @@ -47,6 +45,8 @@ final class DetailWebView: WKWebView { } } +// MARK: - Private + private extension NSUserInterfaceItemIdentifier { static let DetailMenuItemIdentifierReload = NSUserInterfaceItemIdentifier(rawValue: "WKMenuItemIdentifierReload")