rmorawski/standardnotes-app-web
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

chore: dont add toolbar position update listeners on mobile as its unnecessary [skip e2e]

Browse Source
This commit is contained in:
Aman Harwara
2023-10-27 20:03:43 +05:30
parent 0b9116934b
commit 64d3b01b6b
1 changed files with 5 additions and 1 deletions
Download Patch File Download Diff File Expand all files Collapse all files

6
packages/web/src/javascripts/Components/SuperEditor/Plugins/ToolbarPlugin/ToolbarPlugin.tsx
View File

@@ -366,6 +366,10 @@ const ToolbarPlugin = () => {
}, []) }, [])
useEffect(() => { useEffect(() => {
if (isMobile) {
return
}
const scrollerElem = activeEditor.getRootElement() const scrollerElem = activeEditor.getRootElement()
const update = () => { const update = () => {
@@ -386,7 +390,7 @@ const ToolbarPlugin = () => {
scrollerElem.removeEventListener('scroll', debouncedUpdate) scrollerElem.removeEventListener('scroll', debouncedUpdate)
} }
} }
}, [activeEditor, updateToolbarFloatingPosition]) }, [activeEditor, isMobile, updateToolbarFloatingPosition])
useEffect(() => { useEffect(() => {
return mergeRegister( return mergeRegister(