Skip to content
This repository has been archived by the owner on Aug 31, 2021. It is now read-only.

[Bug 13180] Allow GPS access from Android Browser Control #6374

Merged
merged 5 commits into from Mar 13, 2018

Conversation

livecodepanos
Copy link
Contributor

Note: You still have to check the Location permissions in the Android Standalone Settings

Note: You still have to check the Location permissions in the Android Standalone Settings
@livecodepanos livecodepanos added this to the 8.1.10-rc-1 milestone Mar 5, 2018
@@ -321,10 +321,14 @@ public void onHideCustomView()
m_custom_view_callback = null;
}
}
public void onGeolocationPermissionsShowPrompt(String origin, GeolocationPermissions.Callback callback) {
callback.invoke(origin, true, false);
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@livecodepanos I'm thinking we should be presenting a dialog here

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Sure, I'll look into adding one

@livecodepanos livecodepanos removed the WIP label Mar 12, 2018
@montegoulding
Copy link
Member

@livecode-vulcan review ok 5e91c0f

@livecode-vulcan
Copy link
Contributor

💙 review by @montegoulding ok 5e91c0f

livecode-vulcan added a commit that referenced this pull request Mar 12, 2018
[Bug 13180]  Allow GPS access from Android Browser Control

Note: You still have to check the Location permissions in the Android Standalone Settings
@livecode-vulcan
Copy link
Contributor

😎 test success 5e91c0f

  • try-community-armv6-android-api8: success
  • try-community-armv6-android-api9: success
  • try-community-js-emscripten-sdk1.35: success
  • try-community-universal-ios-iphoneos11.2: success
  • try-community-universal-ios-iphonesimulator11.2: success
  • try-community-universal-mac-macosx10.6: success
  • try-community-universal-mac-macosx10.9: success
  • try-community-x86-linux-debian7: success
  • try-community-x86-linux-debian8: success
  • try-community-x86_64-linux-debian7: success
  • try-community-x86_64-linux-debian8: success
  • try-community-x86-win32: success
  • try-community-x86_64-win32: success

@livecodepanos livecodepanos merged commit 4a41492 into livecode:develop-8.1 Mar 13, 2018
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
3 participants