-
Notifications
You must be signed in to change notification settings - Fork 46
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
Updating to Modern iOS Support #13
base: gh-pages
Are you sure you want to change the base?
Conversation
Does this fix the iOS orientation bug?
Undo last change and remove adjustment as a new test
Heading works now, time to work on offset as browser rotates.
Switching orientation source to window.orientation
On Android there is still an offset by 180deg like on your latest readme example page https://compass.wooleys.us/ My friend with an Iphone 11pro reported the compass website also as not working. Any ideas why? Regards, Hans |
Hmm, Is the iPhone 180° offset as well or is it just not working at all? If it is the second have your friend touch the screen. That triggers the second permission prompt that Apple requires to access compass data. As for your Android I would like to fix that but I cannot replicate it currently. Any chance you are in the Southern (or Eastern) Hemisphere? Maybe this code has an assumption of being in the Northern Hemisphere somewhere? Or Perhaps the Western Hemisphere? |
I don't have an Iphone to test, but the wooleys.us was reported completely not working. For the android, I'm in Belgium (51°N 4°E), could this explain the offset? |
Update from the Iphone-side : all is working fine after tapping the screen : Great job ! |
Glad to hear that the iPhone is working right! Now to figure out the Android issue. I didn't modify the Android code, so I assume your issue exists on the main repo too? I'll look around and see if I can find anything to test. Will report back here. |
I think you need for Android: So, in total (can be you need to add 90deg)
|
Ok I updated https://compass.wooleys.us/ to include your suggested changes. Does it work correctly for you now? (See antgiant@3b8c0b1 and antgiant@055b249 for code changes.) |
That's working on my side. Also the orientation from portrait and landscape is fine. Great Job ! |
Thanks for catching the issue. I have added the fix to this pull request. I can confirm that these changes did not break iPhone support. So, hopefully this now works even better for all devices. :-) |
I was checking the compatibility on other browsers and found that the location is not working on Firefox for Android. |
Just to be clear https://lamplightdev.github.io/compass is the original. https://compass.wooleys.us is my updated version. (This pull request would make them both the same.) Which version is not working with Firefox? |
I downloaded the latest commit and placed it on my site : https://www.dehemen.be/antgiant On Google Chrome for Android both sites are working, orientation = OK, location = OK I testen on my phone : Firefox For Android 99.2.0 (2022-04-12T04:32:43.163454) Firefox For Windows 99.0.1 : |
Sorry I do not know. The two immediate possibilities I can think of are
Hmm I just tested Firefox on a iPhone and there neither your site nor mine shows the location but both show compass data. Maybe Firefox uses a different location API? I have not touched the location code at all. Guess I'll need to dig in there later. |
Hey,
I was checking the compatibility on other browsers and found that the location is not working on Firefox for Android.
In the original https://compass.wooleys.us/ it’s working fine.
Any idea why?
Kind Regards, Hans
From: antgiant ***@***.***>
Sent: vrijdag 29 april 2022 14:24
To: lamplightdev/compass ***@***.***>
Cc: Hans ***@***.***>; Comment ***@***.***>
Subject: Re: [lamplightdev/compass] Updating to Modern iOS Support (PR #13)
Thanks for catching the issue. I have added the fix to this pull request. I can confirm that these changes did not break iPhone support. So, hopefully this now works even better for all devices. :-)
—
Reply to this email directly, view it on GitHub<#13 (comment)>, or unsubscribe<https://github.com/notifications/unsubscribe-auth/AKIDAZBCNXLRECJEQPEPXZTVHPIFJANCNFSM5KYJMCJA>.
You are receiving this because you commented.Message ID: ***@***.******@***.***>>
|
Apple devices now require a new permission request to access compass (orientation) data. This adds the required permission request. I also went ahead and fixed the orientation and offset issues on Apple. So this pull request also
Closes #7
Closes #8
Closes #9
Closes #11
Closes #12