feat: toto: selection translation on mobile support

This commit is contained in:
Gabe Yuan
2023-12-25 17:21:59 +08:00
parent d9d97bf14c
commit 16cf9ee1ed
2 changed files with 14 additions and 5 deletions

View File

@@ -1,4 +1,5 @@
import { limitNumber } from "../../libs/utils"; import { limitNumber } from "../../libs/utils";
import { isMobile } from "../../libs/mobile";
export default function TranBtn({ onClick, position, tranboxSetting }) { export default function TranBtn({ onClick, position, tranboxSetting }) {
const left = limitNumber( const left = limitNumber(
@@ -12,6 +13,14 @@ export default function TranBtn({ onClick, position, tranboxSetting }) {
document.body.scrollHeight - 20 document.body.scrollHeight - 20
); );
const touchProps = isMobile
? {
onTouchEnd: onClick,
}
: {
onMouseUp: onClick,
};
return ( return (
<div <div
style={{ style={{
@@ -21,10 +30,7 @@ export default function TranBtn({ onClick, position, tranboxSetting }) {
top, top,
zIndex: 2147483647, zIndex: 2147483647,
}} }}
onClick={onClick} {...touchProps}
onMouseUp={(e) => {
e.stopPropagation();
}}
> >
<svg <svg
xmlns="http://www.w3.org/2000/svg" xmlns="http://www.w3.org/2000/svg"

View File

@@ -49,6 +49,7 @@ export default function Slection({ contextMenus, tranboxSetting, transApis }) {
useEffect(() => { useEffect(() => {
async function handleMouseup(e) { async function handleMouseup(e) {
e.stopPropagation();
await sleep(10); await sleep(10);
const selectedText = window.getSelection()?.toString()?.trim() || ""; const selectedText = window.getSelection()?.toString()?.trim() || "";
@@ -64,7 +65,9 @@ export default function Slection({ contextMenus, tranboxSetting, transApis }) {
setPosition({ x: pageX, y: pageY }); setPosition({ x: pageX, y: pageY });
} }
window.addEventListener(isMobile ? "touchend" : "mouseup", handleMouseup); // TODO: mobile support
window.addEventListener("mouseup", handleMouseup);
// window.addEventListener(isMobile ? "touchend" : "mouseup", handleMouseup);
return () => { return () => {
window.removeEventListener( window.removeEventListener(
isMobile ? "touchend" : "mouseup", isMobile ? "touchend" : "mouseup",