langfingaz/anki
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

Revert "Allow copying of images using context menu"

Browse Source 
This reverts commit d5b3927d0b.

This approach no longer appears to work in Qt6. There is a
QWebEngineView.lastContextMenuRequest() that may work, though whether
onCopy() qualifies as "during contextMenuEvent" is not clear.

https://doc.qt.io/qt-6/qwebengineview.html#lastContextMenuRequest
This commit is contained in:
Damien Elmes 2021-10-04 21:15:24 +10:00
parent aabf693bb8
commit aaceeec25a
1 changed files with 1 additions and 8 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

9
qt/aqt/webview.py
View File

@ -282,14 +282,7 @@ class AnkiWebView(QWebEngineView):
w = w.parent()
def onCopy(self) -> None:
if not self.selectedText():
if not isMac:
# crashes on a mac when clicking in the main window and pressing cmd+c
ctx = self._page.contextMenuData()
if ctx and ctx.mediaType() == QWebEngineContextMenuData.MediaTypeImage:
self.triggerPageAction(QWebEnginePage.CopyImageToClipboard)
else:
self.triggerPageAction(QWebEnginePage.Copy)
self.triggerPageAction(QWebEnginePage.Copy)
def onCut(self) -> None:
self.triggerPageAction(QWebEnginePage.Cut)