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

added android platform #30

Merged

Conversation

sebastianovide
Copy link
Contributor

for #24 added the android platform as descrived in https://capacitorjs.com/docs/android

tested in:

  • web chrome / linux
  • web chrome / android
  • replay_gpx.html web chrome / linux
  • replay_gpx.html web chrome / android
  • app android emulator
  • app android

still todo next (next PR): images, background (probably another ticket)

Copy link
Member

@steinbro steinbro left a comment

Choose a reason for hiding this comment

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

I haven't tested any of the Android parts of this, but the web stuff still works.

voiceSelect.innerHTML = ""; // Clear existing options
populateVoices();
};
// Android Webview doesn have speechSynthesis
Copy link
Member

Choose a reason for hiding this comment

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

The web page works in Chrome on Android, right? I think what you're saying here is that the Capacitor-based app uses an Android Webview which doesn't implement the Web Speech API, but presumably that's where the Capacitor plugin interfaces with the native OS services?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Yes, what Capacitor does is run the web app inside a WebView within a real app. It provides access to native components for JavaScript using different plugins, enabling interaction with native components from different platforms.

@steinbro steinbro merged commit a0dbc01 into soundscape-community:main Jul 4, 2024
2 checks passed
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

Successfully merging this pull request may close these issues.

2 participants