Skip to content

Commit 9a21e0a

Browse files
Merge pull request #17 from digitalsamba/fix/cursors
fix cursor for safari browser
2 parents 1801c4c + 670379b commit 9a21e0a

2 files changed

Lines changed: 4 additions & 12 deletions

File tree

packages/editor/editor.css

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -85,7 +85,7 @@
8585
overflow: clip;
8686
}
8787

88-
.tl-container.tl-old-safari {
88+
.tl-container.tl-safari {
8989
/* Cursor for old safari versions older than 17.0.0 */
9090
--tl-cursor-none: none;
9191
--tl-cursor-default: default;

packages/editor/src/lib/TldrawEditor.tsx

Lines changed: 3 additions & 11 deletions
Original file line numberDiff line numberDiff line change
@@ -158,28 +158,20 @@ export const TldrawEditor = memo(function TldrawEditor({
158158
tools: rest.tools ?? EMPTY_TOOLS_ARRAY,
159159
}
160160

161-
const isOldSafari = useMemo(() => {
161+
const isSafari = useMemo(() => {
162162
if (typeof navigator === 'undefined') {
163163
return false
164164
}
165165

166-
const isSafari = /^((?!chrome|android).)*safari/i.test(navigator.userAgent)
167-
168-
if (!isSafari) {
169-
return false
170-
}
171-
172-
const match = navigator.userAgent.match(/version\/(\d+)/i)
173-
174-
return match ? parseInt(match[1], 10) <= 18 : false
166+
return /^((?!chrome|android).)*safari/i.test(navigator.userAgent)
175167
}, [])
176168

177169
return (
178170
<div
179171
ref={rContainer}
180172
draggable={false}
181173
className={classNames('tl-container tl-theme__light', className, {
182-
'tl-old-safari': isOldSafari,
174+
'tl-safari': isSafari,
183175
})}
184176
onPointerDown={stopEventPropagation}
185177
tabIndex={-1}

0 commit comments

Comments
 (0)