Skip to content

Commit 9c87584

Browse files
ezio84Genkzsz11
authored andcommitted
Restore one-click to add/remove tiles after r20 merge
Google now added the possibility to double click to move tiles for accessibility purposes but i actually prefer the oneclick way Change-Id: Ia44fc4e2ddc77c9cf918c2b8fea38a4e9a3c8252 Signed-off-by: Joey Huab <joey@evolution-x.org>
1 parent 0b8bd60 commit 9c87584

1 file changed

Lines changed: 20 additions & 15 deletions

File tree

packages/SystemUI/src/com/android/systemui/qs/customize/TileAdapter.java

Lines changed: 20 additions & 15 deletions
Original file line numberDiff line numberDiff line change
@@ -89,7 +89,7 @@ public class TileAdapter extends RecyclerView.Adapter<Holder> implements TileSta
8989
private int mAccessibilityFromIndex;
9090
private QSTileHost mHost;
9191
private final UiEventLogger mUiEventLogger;
92-
private final AccessibilityDelegateCompat mAccessibilityDelegate;
92+
//private final AccessibilityDelegateCompat mAccessibilityDelegate;
9393
private RecyclerView mRecyclerView;
9494

9595
public TileAdapter(Context context, UiEventLogger uiEventLogger) {
@@ -98,7 +98,7 @@ public TileAdapter(Context context, UiEventLogger uiEventLogger) {
9898
mItemTouchHelper = new ItemTouchHelper(mCallbacks);
9999
mDecoration = new TileItemDecoration(context);
100100
mMinNumTiles = context.getResources().getInteger(R.integer.quick_settings_min_num_tiles);
101-
mAccessibilityDelegate = new TileAdapterDelegate();
101+
//mAccessibilityDelegate = new TileAdapterDelegate();
102102
}
103103

104104
@Override
@@ -317,22 +317,27 @@ public void onClick(View v) {
317317

318318
holder.mTileView.handleStateChanged(info.state);
319319
holder.mTileView.setShowAppLabel(position > mEditIndex && !info.isSystem);
320-
holder.mTileView.setImportantForAccessibility(View.IMPORTANT_FOR_ACCESSIBILITY_YES);
320+
/*holder.mTileView.setImportantForAccessibility(View.IMPORTANT_FOR_ACCESSIBILITY_YES);
321321
holder.mTileView.setClickable(true);
322322
holder.mTileView.setOnClickListener(null);
323323
holder.mTileView.setFocusable(true);
324-
holder.mTileView.setFocusableInTouchMode(true);
325-
326-
if (mAccessibilityAction == ACTION_NONE && selectable) {
327-
holder.mTileView.setOnClickListener(new OnClickListener() {
328-
@Override
329-
public void onClick(View v) {
330-
move(holder.getAdapterPosition(), mEditIndex /*, notify*/);
324+
holder.mTileView.setFocusableInTouchMode(true);*/
325+
326+
holder.mTileView.setOnClickListener(new OnClickListener() {
327+
@Override
328+
public void onClick(View v) {
329+
int position = holder.getLayoutPosition();
330+
if (position < mEditIndex) {
331+
if (canRemoveTiles()) {
332+
move(position, mEditIndex);
333+
}
334+
} else {
335+
move(position, mEditIndex);
331336
}
332-
});
333-
}
337+
}
338+
});
334339

335-
if (mAccessibilityAction != ACTION_NONE) {
340+
/*if (mAccessibilityAction != ACTION_NONE) {
336341
holder.mTileView.setClickable(selectable);
337342
holder.mTileView.setFocusable(selectable);
338343
holder.mTileView.setFocusableInTouchMode(selectable);
@@ -354,7 +359,7 @@ public void onClick(View v) {
354359
}
355360
if (position == mFocusIndex) {
356361
focusOnHolder(holder);
357-
}
362+
}*/
358363
}
359364

360365
private void focusOnHolder(Holder holder) {
@@ -511,7 +516,7 @@ public Holder(View itemView) {
511516
mTileView.setBackground(null);
512517
mTileView.getIcon().disableAnimation();
513518
mTileView.setTag(this);
514-
ViewCompat.setAccessibilityDelegate(mTileView, mAccessibilityDelegate);
519+
//ViewCompat.setAccessibilityDelegate(mTileView, mAccessibilityDelegate);
515520
}
516521
}
517522

0 commit comments

Comments
 (0)