Closed Bug 972448 Opened 11 years ago Closed 4 years ago

Add support for StaggeredGridView layout in dynamic panels

Categories

(Firefox for Android Graveyard :: Awesomescreen, defect)

ARM
Android
defect
Not set
normal

Tracking

(Not tracked)

RESOLVED INCOMPLETE

People

(Reporter: Margaret, Assigned: alexandru.chiriac, Mentored)

References

Details

(Whiteboard: [lang=java][bad first bug])

Filing this bug, so we don't lose track of it. As mentioned on IRC, it would be cool to play around with using this StaggeredGridView library in dynamic panels:

http://codeascraft.com/2014/01/13/android-staggered-grid/
This could be a fun project for someone to play around with. It would be interesting to see how easy/hard it is to plug a new view type into our home panels system.

To start, I would look at how PanelGridView is implemented:
http://mxr.mozilla.org/mozilla-central/source/mobile/android/base/home/PanelGridView.java
http://mxr.mozilla.org/mozilla-central/source/mobile/android/base/home/PanelLayout.java#269
http://mxr.mozilla.org/mozilla-central/source/mobile/android/base/home/HomeConfig.java#453
http://mxr.mozilla.org/mozilla-central/source/mobile/android/modules/Home.jsm#244
Whiteboard: [mentor=margaret][lang=java][bad first bug]
Hello Margaret, 

I would like to work on this bug, if that is ok with you.

Thanks, 
Alex
(In reply to Alexandru Chiriac from comment #2)
> Hello Margaret, 
> 
> I would like to work on this bug, if that is ok with you.
> 
> Thanks, 
> Alex

Awesome! This bug involves some experimentation and exploration figuring out exactly what we would want this view to look like. It's definitely a more complicated bug than the others you've worked on, but you can handle it :)

As I mentioned in my last comment, I would start by just trying to create a new view type for StaggeredGridView, and then share some screenshots and/or a test build with some folks from our UX team to see what they think. I cc'd ibarlow and antlam to this bug, they're both on our UX team and would love to provide feedback about how things look :)
Assignee: nobody → alexandru.chiriac
Blocks: 942293
No longer blocks: lists
Mentor: margaret.leibovic
Whiteboard: [mentor=margaret][lang=java][bad first bug] → [lang=java][bad first bug]
We have completed our launch of our new Firefox on Android. The development of the new versions use GitHub for issue tracking. If the bug report still reproduces in a current version of [Firefox on Android nightly](https://play.google.com/store/apps/details?id=org.mozilla.fenix) an issue can be reported at the [Fenix GitHub project](https://github.com/mozilla-mobile/fenix/). If you want to discuss your report please use [Mozilla's chat](https://wiki.mozilla.org/Matrix#Connect_to_Matrix) server https://chat.mozilla.org and join the [#fenix](https://chat.mozilla.org/#/room/#fenix:mozilla.org) channel.
Status: NEW → RESOLVED
Closed: 4 years ago
Resolution: --- → INCOMPLETE
Product: Firefox for Android → Firefox for Android Graveyard
You need to log in before you can comment on or make changes to this bug.