search

JayThomason / Tutti

Tutti is a shared music application for the car being made in conjunction with Audi.
3 stars 1 forks source link

Bugfix: Reduce likelihood of accidental song deletion #225

Closed hwray closed 10 years ago

hwray commented 10 years ago

Swiping left and right is our main method of moving between the tabs. However, swiping either direction on the Jam screen while touching a song title will remove that song from the Jam. I've already done this a couple times by accident myself, and I coded the damn thing.

We should either make the target for swiping much smaller (i.e., not cover the entire song title) or make it so only right-to-left swipes remove songs, leaving left-to-right swipes exclusively for navigating back to the Songs view.

JayThomason commented 10 years ago

I also agree that it is too easy to delete songs. I think it may be a little too hard to reorder songs, but that may be only tangentially related and perhaps another issue.

hwray commented 10 years ago

I agree with you. Unfortunately, it seems that our event trigger choices with the Drag-Sort list view are onTouch or onTouchHold. If we use the former, it's really difficult to scroll without randomly rearranging songs. If we use the latter (as we are now), it's difficult for users to discover the rearrangement functionality.

Perhaps we should have some kind of high-level "tutorial" that pops up when a user visits each page for the first time, showing them what they can do? Or add this to the "Help" menu which we aren't really using currently?

hwray commented 10 years ago

I've solved the swiping issue for now, using a one-line fix in the DragSortController similar to the fix suggested here:

https://github.com/bauerca/drag-sort-listview/issues/98