dasilva333 / TowerGhostForDestiny

Tower Ghost for Destiny - This app allows you to transfer items between profiles, with various filters.
112 stars 42 forks source link

Auto Pad Height creates unnecessarily large buckets #449

Open lightmaster opened 9 years ago

lightmaster commented 9 years ago

When using 2 columns on Android, and the Vault or a character have a large number of items in a bucket, the bucket of the other characters that aren't in that "row" are unnecessarily expanded, creating large gaps that look bad and takes up a good bit of extra space on the screen. Not sure I explained that too great, so here are 2 screenshots to shows what I'm trying to explain.

http://i.imgur.com/mGzw5Ke.png http://i.imgur.com/8StpAVg.png

First one shows the Vault has large primary and special weapon buckets, and the first character does as well, which is exactly what the Auto Pad Height is supposed to do. The second screenshot shows the other 2 characters that are found if you scroll down a good bit. Their primary and secondary buckets are also expanded a lot, which if it was 4 columns would look right, but being 2 columns and 2 rows, I think their buckets should only be as big as the one next to it.

Hope you can make sense of what I'm trying to explain.

dasilva333 commented 9 years ago

Looks fine in Chrome, I'll have to debug it under Android only

Screenshot of same view in Chrome: http://i.imgur.com/T0Ba1Ae.png

lightmaster commented 9 years ago

What do the other 2 characters look like on Chrome?

dasilva333 commented 9 years ago

@lightmaster they also look fine, this is a mobile specific bug so I'll have to test it in Android specifically to get this fixed. Thanks