Bug: unable to edit date on editing a highlight (Safari Only)

[0-vortex]

Describe the bug

Screen.Recording.2023-05-04.at.22.14.25.mov

Steps to reproduce

1. Add a new highlight without date
2. Edit highlight and try to set date

Affected services

insights.opensauced.pizza

Platforms

No response

Browsers

No response

Environment

Production, Development, Testing

Additional context

No response

Code of Conduct

• I agree to follow this project's Code of Conduct

Contributing Docs

• I agree to follow this project's Contribution Docs

[0-vortex]

@OgDev-01 Knowing you have worked on implementing the date picker, do you happen to know if this bug is a duplicate of any existing issue? 😅

[OgDev-01]

So I tried to replicate this issue from my end...
https://www.loom.com/share/2f39f0bd7d15456ea64f3bbc0898c4c9

It works as expected on other browsers except Safari... I'll look into this for a possible fix 👍

[diivi]

I'm on ubuntu and almost all alternatives I tried to safari ended up being either too old or ruining other elements of the website too:

If someone knows how to run safari on Linux, I can work on this.

[0-vortex]

If someone knows how to run safari on Linux, I can work on this.

Most online tutorials will use wine to install safari version 5 (2010) on linux - in my humble opinion, that is a waste of time and would not recommend trying 🍕

[nickytonline]

I did a quick test in Safari and if you click quickly multiple times, you'll see it open very briefly and close. I tried recording it, but for some reason it didn't get picked up.

There's also no errors in the console on Safari. The fact that it opens and closes so quickly leads me to believe the state of a hook is somehow triggering in invalid ways.

[OgDev-01]

After our pair session yestersday @nickytonline , all solutions to resolve this had hit a dead end, and i'll open an issue on the radix primitive repo for this if i don't see a related one...

I'll move on with the solution we found (manually handling the popover state with react useState) as a fix for now

Update:
found a related issue in radix-ui/primitives#2348. seems they haven’t been a fix for safari yet

[open-sauced]

🎉 This issue has been resolved in version 1.72.0-beta.4 🎉

The release is available on GitHub release

Your semantic-release bot 📦🚀

[open-sauced]

🎉 This issue has been resolved in version 1.72.0 🎉

The release is available on GitHub release

Your semantic-release bot 📦🚀