Code review comment for lp:~danielholm/music-app/search

Revision history for this message
Daniel Holm (danielholm) wrote :

There, I've shortned the search timer, added the expander, made it so that no results are visible when there are no search terms. Also I added a keyboard hint so that individual characters are mede possible to enter - no word help since that only make searching worse/harder. Also I moved the "Add to playlist" button to parent.bottom. Added a search icon to the field to more look like the other core apps' search fields.
Finally I cant seem to get the text field to auto focus, but neither does Weather or Clock app.

Approving.

review: Approve

« Back to merge proposal