Code review comment for lp:~artem-anufrij/audience/library-view
Revision history for this message
| Danielle Foré (danrabbit) wrote | # |
© 2004
Canonical Ltd.
•
Terms of use
•
Data privacy
•
Contact Launchpad Support
•
Blog
•
Careers
•
System status
•
0e1f616
(Get the code!)
* I don't think the words "Back to" are necessary. On other back buttons we just put the thing you're going back to. So "Library" and "Welcome Screen" are probably fine.
* The window title doesn't seem to update when you change screens. It should show "Videos" on the Library and Welcome pages
* I think these pages should be added to a Gtk.Stack so we can have a nice swipe animation when moving back like you see in Switchboard
* There should probably be some kind of placeholder for when Videos is unable to fetch a poster image
* The shortcut "alt + left" should activate the back button