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

Kotlin version of app demo #656

Open
wants to merge 1 commit into
base: master
Choose a base branch
from
Open

Conversation

Pertence
Copy link

@Pertence Pertence commented Feb 17, 2021

Checklist

  • I read the Contribution Guidelines
  • I signed the CLA and WG Agreements
  • I ran, updated and added unit tests as necessary.
  • I verified the contribution matches existing coding style.
  • I updated the documentation if necessary.

Motivation and Context

Kotlin is an officially supported language for Android app development but, there is only a Java version of the demo. With the Android Studio, it is possible to automatically convert the Java code, but the resulting code needs additional work to compile and run correctly. This commit cuts the step to convert Java to Kotlin every time someone wants to have a Kotlin version.

Description

This commit adds a Kotlin version of the demo app based on the Java version. It also uses view binding. However, it does not have the browser selection feature because replacing AsyncTask with Kotlin Coroutines is nontrivial.

@Pertence Pertence changed the title Kotlin app sample Kotlin version of app demo Feb 17, 2021
@Pertence Pertence marked this pull request as ready for review February 18, 2021 18:31
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.

None yet

3 participants