Merge "Sharesheet: Make max shortcut targets configurable" into qt-dev
This commit is contained in:
committed by
Android (Google) Code Review
commit
8f09cd3de1
@@ -2327,10 +2327,12 @@ public class ChooserActivity extends ResolverActivity {
|
||||
|
||||
private static final int MAX_SUGGESTED_APP_TARGETS = 4;
|
||||
private static final int MAX_CHOOSER_TARGETS_PER_APP = 2;
|
||||
private static final int MAX_SHORTCUT_TARGETS_PER_APP = 8;
|
||||
|
||||
private static final int MAX_SERVICE_TARGETS = 8;
|
||||
|
||||
private final int mMaxShortcutTargetsPerApp =
|
||||
getResources().getInteger(R.integer.config_maxShortcutTargetsPerApp);
|
||||
|
||||
private int mNumShortcutResults = 0;
|
||||
|
||||
// Reserve spots for incoming direct share targets by adding placeholders
|
||||
@@ -2648,7 +2650,7 @@ public class ChooserActivity extends ResolverActivity {
|
||||
final float baseScore = getBaseScore(origTarget, isShortcutResult);
|
||||
Collections.sort(targets, mBaseTargetComparator);
|
||||
|
||||
final int maxTargets = isShortcutResult ? MAX_SHORTCUT_TARGETS_PER_APP
|
||||
final int maxTargets = isShortcutResult ? mMaxShortcutTargetsPerApp
|
||||
: MAX_CHOOSER_TARGETS_PER_APP;
|
||||
float lastScore = 0;
|
||||
boolean shouldNotify = false;
|
||||
|
||||
@@ -4143,4 +4143,6 @@
|
||||
one bar higher than they actually are -->
|
||||
<bool name="config_inflateSignalStrength">false</bool>
|
||||
|
||||
<!-- Sharesheet: define a max number of targets per application for new shortcuts-based direct share introduced in Q -->
|
||||
<integer name="config_maxShortcutTargetsPerApp">3</integer>
|
||||
</resources>
|
||||
|
||||
@@ -2812,6 +2812,7 @@
|
||||
<java-symbol type="layout" name="chooser_grid_preview_file" />
|
||||
<java-symbol type="id" name="chooser_row_text_option" />
|
||||
<java-symbol type="dimen" name="chooser_row_text_option_translate" />
|
||||
<java-symbol type="integer" name="config_maxShortcutTargetsPerApp" />
|
||||
<java-symbol type="layout" name="resolve_grid_item" />
|
||||
<java-symbol type="id" name="day_picker_view_pager" />
|
||||
<java-symbol type="layout" name="day_picker_content_material" />
|
||||
|
||||
Reference in New Issue
Block a user