Fixed

Bug

Status Update

No update yet.

Description

xz...@gmail.com

created issue #1

Aug 20, 2020 07:27AM

Description of the issue

When using a VerticalGridView with a ConcatAdapter, the focus jumps when passing from an element from one sub-adapter to an element from the other sub-adapter.

E.g.:

Adapter 1 = 2 elements
Adapter 2 = 15 elements

Focus is on element 0 (Element 0 of Adapter 1)
I press the DPAD down
Focus is on element 1 (Element 1 of Adapter 1)
I press the DPAD down
Focus is on element 4 (Element 2 of Adapter 2)
I press the DPAD up
Focus is on element 3 (Element 1 of Adapter 2)
I press the DPAD up
Focus is on element 2 (Element 0 of Adapter 2)
I press the DPAD up
Focus is on element 0 (Element 0 of Adapter 1)

A recording of this can be found here: ConcatAdapter_bug_on_Android_TV.mp4.

Component used

LeanBack: 1.1.0-alpha04 (I had the same behavior with 1.0.0 though)
RecyclerView: 1.2.0-alpha05

Devices/Android versions reproduced on

Any Android TV

Sample project

https://gist.github.com/bmonjoie/6ceb2965834bc22ef78be8ca8ddae969

Comments

da...@google.com <da...@google.com> #2Aug 25, 2020 04:53PM

Hi Ed, Thank you so much for these suggestions. I've been reviewing them and merging them in. Hopefully it should be live. I've included a thank you note too in the article.

xz...@gmail.com <xz...@gmail.com> #3Aug 26, 2020 07:00AM

Great! Thanks a lot, I'll look for the live updates soon!

xz...@gmail.com <xz...@gmail.com> #4Aug 26, 2020 07:16AM

Our team investigated that the issue is caused by a change in the RecyclerView library.

The now deprecated methods RecyclerView.ViewHolder.getAdapterPosition() and RecyclerView.LayoutParams.getViewAdapterPosition() fall back to calling RecyclerView.ViewHolder.getBindingAdapterPosition() instead of RecyclerView.ViewHolder.getAbsoluteAdapterPosition().

Leanback depends on the fact that getAdapterPosition() returns a unique position for the RecyclerView, which is no longer the case if you use a ConcatAdapter.

xz...@gmail.com <xz...@gmail.com> #5Sep 21, 2020 02:44PM

Hi, any news regarding this issue?

da...@google.com <da...@google.com> Sep 21, 2020 06:01PM

Accepted by da...@google.com.

ap...@google.com <ap...@google.com> #6Sep 21, 2020 10:04PM

Project: platform/frameworks/support
Branch: androidx-master-dev

commit 2b6777f6f3d8d8015150dfb17b04ade6f1932b0e
Author: Dake Gu <dake@google.com>
Date: Mon Sep 21 12:33:16 2020

leanback: fix ConcatAdapter

The widget should only care about item's getAbosoluteAdapterPosition.

Also fixed flaky GridWidgetTest.scroll(), sometimes it didn't scroll to
begin/end of the screen.

Test: GridWidgetTest.testConcat
Bug: 165694295
Change-Id: Idb70ec00aae74170425a8628290cd604934259c4

M leanback/leanback/src/androidTest/java/androidx/leanback/widget/GridActivity.java
M leanback/leanback/src/androidTest/java/androidx/leanback/widget/GridWidgetTest.java
M leanback/leanback/src/main/java/androidx/leanback/widget/GridLayoutManager.java

https://android-review.googlesource.com/1433120

da...@google.com <da...@google.com> #7Sep 25, 2020 08:47PM

Marked as fixed.

Expect to be release in upcoming release in about one week

IssueTracker

ConcatAdapter doesn't play well with VerticalGridView

Status Update

Description

Description of the issue

Component used

Devices/Android versions reproduced on

Sample project

Comments

da...@google.com <da...@google.com> #2Aug 25, 2020 04:53PM

xz...@gmail.com <xz...@gmail.com> #3Aug 26, 2020 07:00AM

xz...@gmail.com <xz...@gmail.com> #4Aug 26, 2020 07:16AM

xz...@gmail.com <xz...@gmail.com> #5Sep 21, 2020 02:44PM

da...@google.com <da...@google.com> Sep 21, 2020 06:01PM

ap...@google.com <ap...@google.com> #6Sep 21, 2020 10:04PM

da...@google.com <da...@google.com> #7Sep 25, 2020 08:47PM

Issue 165694295

Description

Description of the issue

Component used

Devices/Android versions reproduced on

Sample project

Issue summary

Comments

da...@google.com <da...@google.com> #2Aug 25, 2020 04:53PM

xz...@gmail.com <xz...@gmail.com> #3Aug 26, 2020 07:00AM

xz...@gmail.com <xz...@gmail.com> #4Aug 26, 2020 07:16AM

xz...@gmail.com <xz...@gmail.com> #5Sep 21, 2020 02:44PM

da...@google.com <da...@google.com> Sep 21, 2020 06:01PM

ap...@google.com <ap...@google.com> #6Sep 21, 2020 10:04PM

da...@google.com <da...@google.com> #7Sep 25, 2020 08:47PM

Add comment

Issue metadata