fix: fix issue with safari 12

nolanlawson · Dec 17, 2023 · 0efcfc4 · 0efcfc4
1 parent ed57e94
commit 0efcfc4
Show file tree

Hide file tree

Showing 2 changed files with 5 additions and 3 deletions.
diff --git a/src/picker/components/Picker/Picker.js b/src/picker/components/Picker/Picker.js
@@ -182,8 +182,10 @@ export function createRoot (target, props) {
     calculateEmojiGridStyle
   }
 
+  let firstRender = true
   createEffect(() => {
-    render(target, state, helpers, events, actions, refs, abortSignal)
+    render(target, state, helpers, events, actions, refs, abortSignal, firstRender)
+    firstRender = false
   })
 
   //

diff --git a/src/picker/components/Picker/PickerTemplate.js b/src/picker/components/Picker/PickerTemplate.js
@@ -1,6 +1,6 @@
 import { createFramework } from './framework.js'
 
-export function render (container, state, helpers, events, actions, refs, abortSignal) {
+export function render (container, state, helpers, events, actions, refs, abortSignal, firstRender) {
   const { labelWithSkin, titleForEmoji, unicodeWithSkin } = helpers
   const { html, map } = createFramework(state)
 
@@ -221,7 +221,7 @@ export function render (container, state, helpers, events, actions, refs, abortS
 
   const rootDom = section()
 
-  if (rootDom.parentNode !== container) { // not a re-render
+  if (firstRender) { // not a re-render
     container.appendChild(rootDom)
 
     // we only bind events/refs/actions once - there is no need to find them again given this component structure