View Issue Details

IDProjectCategoryView StatusLast Update
0009186MMANavigationpublic2014-11-02 03:06
Reporterjiri Assigned To 
PriorityurgentSeverityfeatureReproducibilityN/A
Status closedResolutionfixed 
Target Version1.1.0Fixed in Version1.1.0 
Summary0009186: Album grid
DescriptionWe will need to present Albums (and other views, like Videos) in a 2D grid, particularly for the tablet version, but it might be useful even for phones. In order to choose different sizes of images, Pinch Zoom In/Out could be used. Note that Swipe could be used as well to choose between the current (1D) view and new (2D grid view), but maybe it would be fine to leave this choice only for the Pinch Zoom action.
TagsNo tags attached.
Fixed in build327

Relationships

related to 0012281 closedmarek Grid view for more than just Albums 
related to 0012282 closedmarek Scrolling in grid view stutters 

Activities

rusty

2014-08-27 19:48

administrator   ~0040430

Last edited: 2014-09-19 05:29

Tested build 321. Per discussion over email:
1) The grid should be supported in other views besides Albums (It could be an improvement to change the aspect ratio of the grid entries for faster scrolling and increased focus on the text in some views--e.g. in Genres view the image could be shorter and wider to display more albums, and so that there's more room to display the text which is of greater importance than the images).

2) Scrolling is 'jumpy'. This is especially noticeable when scrolling on a Nexus 7 in portrait mode. Google Play avoids this by displaying blank cards with the relevant text, and loading the images after the card has displayed.

Other 'nice to have' changes (i.e. for a future version of MMA):
A) Grid doesn't show letters when using the scroll handle
B) The use of a text overlay on the albums appears slightly cluttered in comparison to the card-based view used in Google Play. That said, it might be preferable to delay changes in this regard until MMA is migrated to a Material Design look and feel.
C) Support for multiple image sizes via Toolbar icon (or alternatively Pinch to zoom (to larger images, to list)). Note that we'll _need_ the 'view configuration' dropdown anyhow in order to support choice of which fields to display + sort order).

marek

2014-09-25 10:05

developer   ~0040575

1) Yes, but it cannot be now implemented in Playlist and Tracklist view because drag and drop has to be implmeneted for grid view.

But do you want to show grid view for tracks generally ? Or only for collections like genres, artists, composers, albums, ...

2) Sure, the loading is always async. There are some bottlenecks somewhere. Big bottleneck was fixed in 322 but it is still not as fast as it should be.

A) Implemented in build 322
B) I agree, working on it.
C) we have discussed with Jiri simple implementation of P2Z. I will try to implement it in next build.

marek

2014-09-27 09:25

developer   ~0040601

I suggest to implement following small tweaks to MultiImageView (component that shows artworks)
D) A lot of artworks are almost square. When we display more these almost square images next to each other, it doesn't look good because they are not completely aligned. It is more visible in grid view. Google play shows all artwork as squares and it looks good for most artworks but it doesn't look good for artowrks with much bigger or much lower aspect ratio than 1.

So I think that I can add small tweak that images with aspect ratio e.g. 0.87 - 1.15 can be stretched to square. Btw. this is only related to list views, not to NP view.

E) I will also change the alignement of images. Image on front layer (the one that is whole visible) will be centered. Images on lower layers will be top left aligned.

marek

2014-09-30 18:17

developer   ~0040653

3) MultiImageView is not working correctly for more then 1 artwork (i.e. artists with more albums).

marek

2014-09-30 18:24

developer   ~0040654

Build 323:

2) I think that now is finally removed the biggest bottleneck.

3) Fixed resizing of bitmap

B) Added text under the artwork. Now I use micro font. Maybe small font might be used. I am not sure. But I don't recommend same size as in list (i.e. medium). It doesn't look good.

D) Added stretching of almost square albums. It looks much better now.

E) Aligment improved. I have also changed order. First album is in the front.

marek

2014-10-07 07:03

developer   ~0040723

Last edited: 2014-10-07 07:06

C) I was trying to implement it. But simple creating of Fragments with different layout during p2z gesture is not possible. It has to be done in more sophisticated way. It cannot block main thread and Fragments has to be well managed.

I think that it might be possible to somehow change ViewPager. We use it in NP screen as slider for next/previous track. ViewPager can be used with Fragments too. Two things has to be adapted:

 a) Animation - change from sliding to some crossfade. This might be possible as described here: http://developer.android.com/training/animation/screen-slide.html#pagetransformer
 
 b) Gesture - change somehow that it should not react on dragging but only to pinch to zoom. Drag gesture will still be needed by underlaying listviews. Hopefully it is somehow possible.

But still it will not be probably so easy to implement because ViewPager is quite complicated view. Should I work on it now?

jiri

2014-10-07 08:02

administrator   ~0040726

C) Let's defer this...

rusty

2014-10-07 15:20

administrator   ~0040729

Agreed. And even if we decide we want to add support for different size images, it can be done via a dropdown.

marek

2014-10-23 11:27

developer   ~0040809

Added some fixes to be compatible with API < 11

Fixed in build 327

rusty

2014-11-02 03:01

administrator   ~0040864

Verified 327.