View Issue Details

IDProjectCategoryView StatusLast Update
0009695MMAGeneralpublic2012-12-11 15:38
Reporterrusty Assigned To 
PriorityurgentSeverityminorReproducibilityalways
Status closedResolutionfixed 
Target Version1.0.1Fixed in Version1.0.1 
Summary0009695: Tweaks for Landscape mode
DescriptionOn switching to landscape mode, there are a couple of things that don't display properly:
1) The Action bar: half of it glows (up to the Search button), and the other half doesn't. I would get rid of the glow entirely (even in portrait mode).

2) NP > Art view's Player bar: The 'Toggle to NP>List' button should be left-align within the player toolbar so that the toggle button doesn't move on each toggle.

3) NP > Art view: the swipe to show lyrics doesn't really make sense within this view. Perhaps the Rating/Continuous/Shuffle functions should be moved onto the Artwork and the Lyrics should display on the right side? (this can be moved to a different bug)

4) Already mentioned at 0009596, but should probably be tracked here: the absence of any player controls in the miniplayer is quite annoying.
TagsNo tags attached.
Attached Files
album_landscape_qvga.png (7,244 bytes)   
album_landscape_qvga.png (7,244 bytes)   
Fixed in build74

Relationships

related to 0009596 closedmarek Player bar should show additional controls if space is available 
related to 0009813 closedrusty Overflow (Options) missing from Menu bar (regression) 

Activities

marek

2012-09-13 20:17

developer   ~0031978

1) I cannot see any glow. Could you please attach a screenshot ? And other specifications ? Api, phone, etc. Thanks

2) I completely changed a layout of landscape action bar. I put this button to the right.

3) Yes, the landscape layout isn't good. We should modify it.

4) I will add these controls to large and xlarge screens.

rusty

2012-09-13 21:59

administrator   ~0031979

1) Images attached. They were generated on a Nexus 7 running Jellybean. A phone running gingerbread doesn't exhibit this problem.

rusty

2012-09-16 06:49

administrator   ~0032013

5) The amount of space reserved for the Player/Context menu entries should be increased slightsly (i.e. move the Search icon to the left).

marek

2012-09-27 06:51

developer   ~0032218

4) fixed

5) well the sizes of miniplayer and context menu aren't dependent. Only NP screen controls and mini-player are dependent. I increased the space for large and xlarge screen according to modifications in 4). I don't think it is a good idea to move it more to the left. There isn't only search icon. There are "Play all shuffled" and "Sync progress icon". I think the current size is optimal. In some sizes, the count of icons should be even reduced (please look at the attached image) because of too big miniplayer.

marek

2012-09-27 07:02

developer   ~0032219

1) I've test it on Galaxy Tab and nothing. I've learned that Nexus 7 had some special DPI - tvdpi. I checked all Nine-path PNG files that are used for this border and I didn't found any fault.

I've found out that Nexus 7 is problematic in scaling of drawables (for example https://groups.google.com/forum/?fromgroups=#!topic/android-developers/eonYQJP6gpU). I can create some test drawables and we can test it.

jiri

2012-10-12 12:18

administrator   ~0032475

Assigning to Rusty for feedback.

rusty

2012-10-12 17:42

administrator   ~0032486

Tested build 43:

1) I've reviewed other apps on this device and none have this type of glow at the top. Note that early builds of mma did not have this glow. If you have no idea, I can start testing earlier builds and see where it was introduced...

2) This issue still isn't fixed. In landscape mode, the NP>Playlist toggle button for the miniplayer should be in the _exact_ same location as the Artwork in the miniplayer.

3) I'll open a new bug...

jiri

2012-10-23 14:57

administrator   ~0032749

Last edited: 2012-10-23 15:03

1) It seems to be caused by SVN Revision 547 where bitmap is used instead of vector graphics. Bitmap scaling explains the problem then. Marek please review how to be revert this change.

2) Fixed in build 50.

jiri

2012-10-24 09:05

administrator   ~0032763

1) As discussed over IM, it seems that we could try to provide the bitmap in 'tvdpi' resolution, so that Nexus 7 doesn't incorrectly scale it.

marek

2012-12-07 09:10

developer   ~0033715

Fixed in build 74

peke

2012-12-08 23:33

developer   ~0033775

Verified 75