-
-
Notifications
You must be signed in to change notification settings - Fork 1.8k
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
Add Miscellaneous setting and Hide app icon option #2462
base: master
Are you sure you want to change the base?
Add Miscellaneous setting and Hide app icon option #2462
Conversation
Hide icon from launcher
optionHide app icon
option
85b2d24
to
39c349b
Compare
The icon should be enabled by default. These two should seems to do the opposite: |
Also in my opinion the variable name seems a bit strange: Personally I would choose |
The option in microG settings is disabled so the icon should be enabled by default, but you disable the icon in reality here:
Edit: Now that I read better, here you don't disable the icon but the MainSettingsActivity so you actually break the microG settings. This shouldn't be touched => org.microg.gms.ui.MainSettingsActivity |
play-services-base/core/src/main/kotlin/org/microg/gms/settings/SettingsProvider.kt
Outdated
Show resolved
Hide resolved
This is just personal opinion but in this case: instead with this: So actually it is the same in practice but I think the first is just more confusing but it is still personal preference. @mar-v-in What do you think? |
75e6627
to
6f4e97f
Compare
@ale5000-git, I believe I have addressed all the requested changes in this pull request. Please let me know if there are any additional modifications or suggestions. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
See my comments on the code. Also, please add a way to hide this setting on the Huawei build flavor, as devices with Huawei OS that support microG, the microG settings is automatically hidden from launcher.
play-services-core/src/main/java/org/microg/gms/auth/login/LoginActivity.java
Outdated
Show resolved
Hide resolved
play-services-base/core/src/main/kotlin/org/microg/gms/settings/SettingsContract.kt
Outdated
Show resolved
Hide resolved
@mar-v-in |
This reverts commit 3094aff.
I have made changes according to the requests, not sure if it is what was intended. Do let me know if further changes are needed. |
Is this within the scope of this PR or just a suggestion for the future of microg? |
It can also be done later in a different PR. |
@ILoveOpenSourceApplications |
Also, although I haven't tried, the changes to AndroidManifest.xml seems totally unnecessary. |
Can you point me as to where I need to make the changes? |
Hide app icon
optionMiscellaneous
setting and Hide app icon
option
Miscellaneous
setting and Hide app icon
option
Adds an option to hide icon from launcher.
Resolves #2263, resolves #2266 as well as resolves #2361.
Credits: @rufusin