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

Replace the "About" WebView with a FragmentDialog #320

Merged
merged 1 commit into from
Feb 11, 2019
Merged

Replace the "About" WebView with a FragmentDialog #320

merged 1 commit into from
Feb 11, 2019

Conversation

splitowo
Copy link
Contributor

@splitowo splitowo commented Feb 3, 2019

This code relates to the #203 issue. WebView has been replaced with a regular TextView inside FragmentDialog. Hardcoded dialog body is now in strings.xml and can be translated.

This type of dialog is kept alive in case of screen orientation change (discussed in #274). Links are functional and open in an internet browser.

@hilmizaytouni
Copy link

jackzaty3@

@robinpaulson
Copy link
Contributor

Excellent. The v1.9 release is going to be pretty impressive!

@robinpaulson
Copy link
Contributor

Hi @splitowo I'm afraid another commit I merged today means your PR will no longer merge. I don't have the skills to fix it, perhaps you could take a look? I'm sure it's simple, at a guess only a few lines of code. Cheers, Robin

@splitowo
Copy link
Contributor Author

splitowo commented Feb 5, 2019

Thanks for the message. I rebased my commit on top of your master branch, it should now merge with no issues

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.

3 participants