kumo01GitHub commented on PR #279: URL: https://github.com/apache/cordova-plugin-geolocation/pull/279#issuecomment-2407740554
I'm sorry for insufficient explanation. Most of change is convert WebView implementation to `FusedLocationProviderClient` but it is not the point. So, I will close this pull request. Essentially, I wanted focus on below points: - In [W3C](https://w3c.github.io/geolocation/#introduction), it looks no distinction between fine and coarse location permissions. And according to introduction, user agent can ignore `enableHighAccuracy`. But, permission check doesn't allow application to use coarse location when request fine location with `enableHighAccuracy`. - Through whole processing, plugin doesn't check whether location service is available. So when location service is unavailable, plugin treats as timeout. But refer to `Note: Browser permission VS OS permission` in W3C, it seems location service availability is similar to OS permission and it should return [PERMISSION_DENIED](https://w3c.github.io/geolocation/#dom-geolocationpositionerror-permission_denied) As you can see from my poor English, I'm not good at English. So there may be some misunderstandings. But some part of my idea makes sense, I will make another PR from [77dff7d](https://github.com/apache/cordova-plugin-geolocation/pull/279/commits/77dff7dd41b39f2572bbe9ae56d20f3a5e22daf9) -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: issues-unsubscr...@cordova.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org --------------------------------------------------------------------- To unsubscribe, e-mail: issues-unsubscr...@cordova.apache.org For additional commands, e-mail: issues-h...@cordova.apache.org
