View Issue Details
| ID | Project | Category | View Status | Date Submitted | Last Update |
|---|---|---|---|---|---|
| 0009678 | MMA | Playlists | public | 2012-09-09 21:26 | 2012-10-25 19:15 |
| Reporter | rusty | Assigned To | |||
| Priority | urgent | Severity | minor | Reproducibility | always |
| Status | closed | Resolution | fixed | ||
| Target Version | 1.0.1 | Fixed in Version | 1.0.1 | ||
| Summary | 0009678: Playlist and NP list scrollbar conflicts with track handles | ||||
| Description | The scrollbar that appears when scrolling through the NP list and Playlists is different from the regular Android scrollbar. Instead of appearing at the edge of the list, it: a) is superimposed on top of the track handles (that are used to drag and drop tracks) b) has a handle that upon attempting to use it, causes existing tracks to get moved (i.e. even though the scrollbar is superimposed on top of the handles, it has no effect re. scrolling). The simple solution would be to use the standard android scrollbar. | ||||
| Tags | No tags attached. | ||||
| Fixed in build | 37 | ||||
| related to | 0009636 | resolved | marek | D&D problems |
| related to | 0009777 | closed | jiri | Better usage of space in playlists (empty space on the right side) |
| related to | 0009887 | closed | marek | Fast Scroll bar conflicts with selection checkboxes |
| related to | 0009392 | closed | marek | Using scrollbar for scrolling => weird results |
| related to | 0008629 | closed | marek | Larger draggable scrollbars |
| related to | 0009812 | closed | marek | Scroll bar doesn't display correctly on some devices |
| related to | 0009858 | resolved | rusty | Scroll bar is missing on NP/Playlist views on some devices |
|
|
We use the same scrollbars in all view. I think that we should rather hide the d&d handles when scrolling is in progress (or at least don't use them). For this purpose public void setOnScrollListener (AbsListView.OnScrollListener l) method could probably be used. Some more info about it: http://stackoverflow.com/questions/1768391/how-to-detect-android-listview-scrolling-stopped |
|
|
Note: the scrollbar with the 'handle' on it really only does appear sometimes--only when the tracklist is rather large. BUT even then, it doesn't appear consistently--it appears on a Nexus 7 (jellybean) device, but for a similarly-sized playlist on a Gingerbread phone, the standard scrollbar (without the handle) shows up. |
|
|
Note: a similar problem occurs in Multi-select mode. Any attempt to grab the scrollbar (which appears on top of the checkboxes) results in items being checked instead of the scrollbar being selected. |
|
|
Last time I spoke with Matej, he asked me why the handlers aren't on the left side. Well it is true that it is quite common (according to other apps). So I want just suggest to move the handlers to the left side and solve all issues with this. It has an advantage that users don't have to wait on scrollbar's disappearing. Re. scrollbars: Scrollbars are pure system thing. Only Android manages them. We are only able to override the styles. Standard scrollbar is the thick grey line. It can be styled in all versions of API. So it is styled according to our design. Fast scrollbar is an optional scrollbar mode that was added according to http://www.ventismedia.com/mantis/view.php?id=8629. This scrollbar is applied when list is too long so users can scroll faster. The appearing of this scrollbar is managed by system. This scrollbar is stylable only since version 3.0. So there is Android's default grey box in earlier versions. In newer versions, the scrollbar is styled according to our design. I tried to reproduce the problem with checkboxes in contextual mode but works fine for me. Every time I grab the scrollbar. |
|
|
Ok, let's move it to the left side. |
|
|
Fixed in build 37 |
|
|
I'm not that keen on the grab handles being on the left side(it's more usable for righties to have it on the right side), but we can worry about that in the future. I would have preferred a solution that retains the handles on the right, and allows the scroll handle to override the track handles. But in the meantime, I can't verify this issue because it seems that the scrollbars are completely removed in Gingerbread (in NP and Playlist views). |
|
|
Yes, handled to the right would probably be slightly more usable, however it seems that we are unable to control scrollbars to the level needed. And no, I don't think scrollbars were removed, I have them. It's just that Android doesn't show the 'fast' scrollbars until there's enough items in the list, I guess that something like >30 is needed. |
|
|
I have 1000+ tracks (Now Playing>List or Playlist views) and the scrollbar doesn't appear. Tested build 42 on GB. Edit: it seems that the scrollbar appears briefly and then disappears for my 1000+ list. When I use a smaller list of e.g. 50 items, the scrollbar does appear. |
|
|
Assigning to Rusty to review whether any app shows the fast scrollbar differently than MM on that device. |
|
|
Upon further examination, the problem with the missing scrollbars in GB phone device is specific to the Now Playing and Playlist > <Playlist> views. All other views use the standard scrollbar that's normally used on small format screens (thin line to the right with a transparent handle that appears for fast scrolling). Maybe marek fixed this in 0009777 ? |
|
|
I can't reproduce on my Nexus One 2.3.6. And no, 0009777 was a different fix. Assigning to Marek to review, we'd need to defer if he has no idea... |
|
|
Also cannot reproduce on CM7 2.3.7. Playlist has about 2500 tracks I tried build 35 and 45 |
|
|
I tested build 45 (with views that would display 1000 tracks), and the issue persists in those two views (Playlists and Now Playing lists do _not_ show a scroll handle) in contrast to all other list views which do. Is MMA handling those 2 views differently in some way? Is there any additional information that I can provide to help resolve this? |
|
|
Yes, these two options are processed differently than others. They have handlers and support drag and drop. So they are different. This issue might be API specific. Can you please write me version of your GB ? I can try to find it for my phone. I went through the code and it looks fine. And it looks it works well on most devices. |
|
|
I'm running Android 2.3.4. |
|
|
I tried it many times on Android 2.3.3 (there are no big differences). This is the nearest version of API I can test but it doesn't happen to me. Maybe it is happening only on this device / branded system. |
|
|
Tagging as resolved. I'll open a low priority issue that's sepecific to the NP/Playlist problem. |
|
|
Verified 48. |
