[Merge] lp:~ahayzen/webbrowser-app/browser-colour-update-001 into lp:webbrowser-app/staging
Olivier Tilloy
olivier.tilloy at canonical.com
Tue Nov 8 15:08:58 UTC 2016
Disclaimer: only tested on desktop so far, up-to-date xenial+overlay.
The pressed state of items in the drawer menu looks very dark to me, is that intended?
The divider between tabs in the tabs bar looks plain black, it looks too dark to me, is that intended?
When hovering over a tab in the tabs bar, it looks awkward that the divider between it and the next/prev tab remains visible, is that intended?
The progress bar and the horizontal line at the bottom of the chrome are stacked, and this makes the progress bar look taller, which I find disturbing to the eye. Maybe anchor the bottom of the progress bar to the bottom of the line?
There’s no hovered and pressed states for the + button at the right of the top bar to open a new tab. Should we add them?
The reference to pad.lv/1638337 in Suggestion.qml looks like it’s the wrong bug number?
--
https://code.launchpad.net/~ahayzen/webbrowser-app/browser-colour-update-001/+merge/310173
Your team Ubuntu Phablet Team is requested to review the proposed merge of lp:~ahayzen/webbrowser-app/browser-colour-update-001 into lp:webbrowser-app/staging.
More information about the Ubuntu-reviews
mailing list