[Merge] lp:~artmello/webbrowser-app/webbrowser-app-bookmarks_view into lp:webbrowser-app
Ugo Riboni
ugo.riboni at canonical.com
Sun Sep 13 23:16:07 UTC 2015
Review: Needs Fixing
- I can not get any favicon to load, even for sites that I just visited and have a favicon. The "globe" icon is always shown instead
- Open two browsers windows side by side and in one open the bookmarks view and in the other the history view:
The styles of the two are not consistent with each other. If necessary speak with design about this, but I would change the following things:
- In widescreen mode:
- Use the same font as the history items for bookmark items
- Use the same font as the dates in the history view for folders
- The bookmarks view header should have a separator
- The folder list items should have a separator
- In non-widescreen mode:
- The bookmarks view has an header, the history doesn't. I would add one to the history.
- The folders in the bookmarks view should probably start all collapsed instead of all open, unless no folders exist.
--
https://code.launchpad.net/~artmello/webbrowser-app/webbrowser-app-bookmarks_view/+merge/270613
Your team Ubuntu Phablet Team is subscribed to branch lp:webbrowser-app.
More information about the Ubuntu-reviews
mailing list