Open Bug 974434 Opened 6 years ago Updated 5 years ago

Add support for items with variable height in PanelGridView

Categories

(Firefox for Android :: Awesomescreen, defect, P2)

All
Android
defect

Tracking

()

People

(Reporter: lucasr, Unassigned)

References

(Blocks 3 open bugs)

Details

Stock GridView doesn't deal with them very well, the layout gets all broken as you scroll. We'll have to customize GridView somehow.
Priority: -- → P1
Priority: P1 → P2
Just a quick update here:

Turns out GridView is pretty dumb about rows containing items with variable heights. It simply takes the last (or the first, if it's filling from bottom) child in the row as a reference view to define where to place the next (or previous) row. See:

https://github.com/android/platform_frameworks_base/blob/kitkat-release/core/java/android/widget/GridView.java#L357
https://github.com/android/platform_frameworks_base/blob/kitkat-release/core/java/android/widget/GridView.java#L294
https://github.com/android/platform_frameworks_base/blob/kitkat-release/core/java/android/widget/GridView.java#L391
Blocks: 1006081
Setting P2 hub bugs to block hub v2 EPIC bug (targeting fx31 release).

Filter on epic-hub-bugs.
Blocks: 1014030
Assignee: lucasr.at.mozilla → nobody
You need to log in before you can comment on or make changes to this bug.