Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Adding modules does not work in Safari #23

Open
valtism opened this issue May 25, 2022 · 15 comments
Open

Adding modules does not work in Safari #23

valtism opened this issue May 25, 2022 · 15 comments

Comments

@valtism
Copy link
Contributor

valtism commented May 25, 2022

Just found out about this project. Looks awesome, especially because Bundlephobia has been pretty flaky as of late.

In Safari, when selecting a package from the search select box, clicking add does nothing. Looks like the search results box is closing before the selection is made. No console output unfortunately.

Screen.Recording.2022-05-25.at.11.35.24.am.mov
@okikio
Copy link
Owner

okikio commented May 25, 2022

I'm having trouble recreating this issue (I don't have a mac so that might be why 😅). Right now I'm using Gnomes Epiphany browser on WSL to test WebKit, but as I don't have a Mac I can't test on Safari directly. Could you please clear the cache and try clicking the add button again.

For the search results I am using CSS :focused-within to keep the results shown, so the problem is most likely related to :focused-within, not working properly for some reason. 🤔

@valtism
Copy link
Contributor Author

valtism commented May 25, 2022

Yeah, cleared cache and is still persisting. I would say that Safari is handling focus states differently so focus is either being dropped or is not considered within the input so :focused-within turn off before the click goes through.

I think it would be better to handle the open state as actual state in a useSignal because :focused-within does feel a little hacky.

I can have a look at fixing this myself later if you are struggling with replication, but I may take a while to get around to it.

@okikio
Copy link
Owner

okikio commented May 25, 2022

I'll try to get on it as soon as possible, I am currently working on porting the entire project over to Astro

@valtism
Copy link
Contributor Author

valtism commented May 25, 2022

Yeah no rush. Still works with manually typing in the package name. Will be keeping an eye on this project because it looks super nice 👍

@MauriceArikoglu
Copy link

MauriceArikoglu commented Nov 24, 2022

Can confirm it NOT working on any iPhone browser either @valtism @okikio

@okikio
Copy link
Owner

okikio commented Nov 24, 2022

@MauriceArikoglu Thanks for notifiying me, I'm working on it

@MauriceArikoglu
Copy link

@okikio thanks for the app. It's honestly great. Just a bummer I can't use it on iPhone while on the go.. 👍 if you need help let me know, I'm down for having a look or call etc.

@okikio
Copy link
Owner

okikio commented Nov 24, 2022

Yeah, I'm having trouble debugging this. Please try the beta version https://bundlejs.netlify.app does this have the same error from your side

@okikio
Copy link
Owner

okikio commented Nov 24, 2022

@MauriceArikoglu After a bit of debugging it seems this issue only occurs on iOS 16 but on iOS 16.1 everything seems fine

@okikio
Copy link
Owner

okikio commented Mar 6, 2023

I think this is resolved

@okikio okikio closed this as completed Mar 6, 2023
@valtism
Copy link
Contributor Author

valtism commented Mar 6, 2023

Confirming this works on Safari on the beta version, but it does not work on the current prod version. Tried with private browsing too, and unregistering service workers.

@okikio okikio reopened this Mar 6, 2023
@okikio
Copy link
Owner

okikio commented Mar 6, 2023

Interesting...ok I'll try to backport a fix

@valtism
Copy link
Contributor Author

valtism commented Mar 6, 2023

Also, should mention this is Safari 16.3 on macOS.

@valtism
Copy link
Contributor Author

valtism commented Mar 6, 2023

Although testing on iOS also doesn't work. I think it works on beta because the popover doesn't close

@pketh
Copy link

pketh commented Mar 25, 2024

just ran into this issue on macOS safari 17.2

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

4 participants