breautek commented on PR #298: URL: https://github.com/apache/cordova-plugin-geolocation/pull/298#issuecomment-4826303713
Two immediate comments: ## W3C Specification Deviation This PR extends the api with non standard features not defined in the W3C specification. All of Apache plugins where applicable implements the W3C standard so that (hopefully) one day the plugin won't be necessary and the user can use the webview implementation. While some plugins have deviate from the W3C for one reason or another, this plugin I believe still follows the specification accurately. Most plugins that have deviated do so because the specification became deprecated or obsolete without replacement. The geolocation specification however is active and stable. So I think the plugin requires extra scrutiny for proposals that deviate from the W3C specification. So I think this change would require a vote on our dev mailing list. ## Background location I didn't make a test app but I have a hard time believing this works as iOS pauses JS executions while in the background -- which means the callback events won't be delivered at least not at the time of the event and instead might be delivered in one burst once the app resumes back into the foreground. At least that was my past experiences when working with background location events which I admit has been awhile. -- 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: [email protected] For queries about this service, please contact Infrastructure at: [email protected] --------------------------------------------------------------------- To unsubscribe, e-mail: [email protected] For additional commands, e-mail: [email protected]
