Merge pull request #1185 from omnivore-app/fix/add-lookup-menu-to-pdf

Add the Lookup (define) menu item to PDF
This commit is contained in:
Jackson Harper 2022-09-12 12:19:43 +08:00 committed by GitHub
commit 1fafcd84bc
No known key found for this signature in database
GPG key ID: 4AEE18F83AFDEB23

View file

@ -88,6 +88,7 @@ import Utils
}
.onShouldShowMenuItemsForSelectedText(perform: { pageView, menuItems, selectedText in
let copy = menuItems.first(where: { $0.identifier == "Copy" })
let define = menuItems.first(where: { $0.identifier == "Define" })
let highlight = MenuItem(title: "Highlight", block: {
_ = coordinator.highlightSelection(
pageView: pageView,
@ -101,7 +102,8 @@ import Utils
// shareLink = ShareLink(id: UUID(), url: shareURL)
// }
// })
return [copy, highlight /* , share */ ].compactMap { $0 }
define?.title = "Lookup"
return [copy, highlight, define].compactMap { $0 }
})
.onShouldShowMenuItemsForSelectedAnnotations(perform: { _, menuItems, annotations in
var result = [MenuItem]()