alex/yomichan
1
Fork 0
Code Releases Activity

fix popup containsPoint offset

Browse Source
This commit is contained in:
siikamiika 2020-03-07 03:52:36 +02:00
parent 421b60db0f
commit 86be737508
1 changed files with 8 additions and 4 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

12
ext/fg/js/popup-proxy-host.js
View File

@ -116,7 +116,8 @@ class PopupProxyHost {
async _onApiContainsPoint({id, x, y}) {
const popup = this._getPopup(id);
return await popup.containsPoint(x, y);
const rootPagePoint = PopupProxyHost._convertPopupPointToRootPagePoint(popup, x, y);
return await popup.containsPoint(...rootPagePoint);
}
async _onApiShowContent({id, elementRect, writingMode, type, details}) {
@ -152,14 +153,17 @@ class PopupProxyHost {
}
static _convertJsonRectToDOMRect(popup, jsonRect) {
let x = jsonRect.x;
let y = jsonRect.y;
const [x, y] = PopupProxyHost._convertPopupPointToRootPagePoint(popup, jsonRect.x, jsonRect.y);
return new DOMRect(x, y, jsonRect.width, jsonRect.height);
}
static _convertPopupPointToRootPagePoint(popup, x, y) {
if (popup.parent !== null) {
const popupRect = popup.parent.getContainerRect();
x += popupRect.x;
y += popupRect.y;
}
return new DOMRect(x, y, jsonRect.width, jsonRect.height);
return [x, y];
}
static _popupCanShow(popup) {