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

Introduce About Fragment #128

Merged
merged 7 commits into from
Feb 15, 2022
Merged

Introduce About Fragment #128

merged 7 commits into from
Feb 15, 2022

Conversation

theimpulson
Copy link
Contributor

This MR:

  • Upstreams the Gradle and dependencies to the latest stable release
  • Adds a new About Fragment which displays app logo, description, and privacy policy URL (not hosted on the server yet)

This should close #111, however, keep in mind that the URL seems not hosted or public yet and should be handled as well.

The URL can be changed by maintainers by editing the variable value in app/src/main/java/org/eu/exodus_privacy/exodusprivacy/fragments/AboutFragment.java

@theimpulson
Copy link
Contributor Author

The ic_information vector drawable has been taken from materialdesignicons.

@pnu-s pnu-s requested review from jfoucry, Schoumi and stom79 February 8, 2022 18:45
@Jean-BaptisteC
Copy link
Contributor

Request changes:

  • Change privacy policy string like others links on app (in exodus app on purple, we don't see difference between purple and black)
  • Hide search button and filter button on about ihm like trackers ihm

@Jean-BaptisteC
Copy link
Contributor

On smartphone in mode landscape, we don't see privacy policy string

- Also migrate to new lint block

Signed-off-by: Aayush Gupta <aayushgupta219@gmail.com>
Signed-off-by: Aayush Gupta <aayushgupta219@gmail.com>
Signed-off-by: Aayush Gupta <aayushgupta219@gmail.com>
Signed-off-by: Aayush Gupta <aayushgupta219@gmail.com>
Signed-off-by: Aayush Gupta <aayushgupta219@gmail.com>
Signed-off-by: Aayush Gupta <aayushgupta219@gmail.com>
@Jean-BaptisteC
Copy link
Contributor

Ok for me, i have tested changes:

  • No regression
  • No new bugs

@jfoucry
Copy link
Contributor

jfoucry commented Feb 15, 2022

Ok for me, tested on OnePlus 5T+ Lineageos 18.1 (Android 11)

  • No regression
  • No new bugs

@Jean-BaptisteC Jean-BaptisteC merged commit 7c64507 into Exodus-Privacy:master Feb 15, 2022
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.

Google Play requirements to republish Exodus application
4 participants