-
-
Notifications
You must be signed in to change notification settings - Fork 1.3k
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 a switch to control suggested apps showing #3803
Conversation
Seems recent apps were shown when Recents are enabled, should we check if Recents is available? |
There’s already an option for this, at least it was in 12.1. |
I know why, I can't see this
|
Oh I see, maybe try updating ASI and Google app |
In this case, can we keep this flag to control the suggestions' visibility, maybe I should merge this option into |
Maybe add some check, only show if this occur? |
This check is enough. lawnchair/lawnchair/src/app/lawnchair/ui/preferences/components/SuggestionsPreference.kt Lines 19 to 21 in e968788
|
quickstep/src/com/android/launcher3/appprediction/PredictionRowView.java
Show resolved
Hide resolved
quickstep/src/com/android/launcher3/appprediction/PredictionRowView.java
Show resolved
Hide resolved
yeah look great |
@@ -23,5 +25,14 @@ fun SuggestionsPreference() { | |||
context.startActivity(intent) | |||
}, | |||
) | |||
} else { | |||
// On some devices, the Suggestions activity could not be found or PACKAGE_USAGE_STATS is not granted. |
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.
This would be confusing if the user's device doesn't have a suggestions service and this toggle appears. A better way would be to investigate why your device has a suggestion service but not the activity.
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.
I found this issue when I set Lawnchair as the QS provider, even if I fallback to the default QS provider, suggested apps are still showing.
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.
We probably merge this first, for the ones who need this. Look back on this later.
Description
Enabled:
Disabled:
Type of change
❌ General change (non-breaking change that doesn't fit the below categories like copyediting)
❌ Bug fix (non-breaking change which fixes an issue)
✅ New feature (non-breaking change which adds functionality)
❌ Breaking change (fix or feature that would cause existing functionality to not work as expected)