Ubuntu Music App

Code review comment for lp:~mcintire-evan/music-app/songs-fastscroll

  1. songs-fastscroll
  2. Merge into trunk
Revision history for this message
Victor Thompson (vthompson) wrote :

The code for this looks good! I'm not sure however, that we'll want to merge this into the app. The Ubuntu SDK will be introducing scrollbars sometime soon and I think it'd be better to use that feature/component as we'll also want to use it in the Queue and other listviews.

Codewise, I do have 1 inline comment.

review: Needs Fixing
Reply

« Back to merge proposal