Code review comment for lp://qastaging/~ubuntu-weather-dev/ubuntu-weather-app/reboot-finish-listitem-migration

Revision history for this message
Andrew Hayzen (ahayzen) wrote :

A few inline comments, visually ...
1) there appears to be a space between the detected location and the first selected location
2) if you multiselect, select an item, then select delete you are still in multiselect mode, this is weird if you remove the last item in the list as you have to hit back what feels like twice. I suggest when delete is pressed in the header that the multiselect mode is cancelled.

review: Needs Fixing

« Back to merge proposal