[webapp-container] New top header takes too much space
Affects | Status | Importance | Assigned to | Milestone | |
---|---|---|---|---|---|
Ubuntu UX |
Fix Committed
|
Medium
|
James Mulholland | ||
webbrowser-app |
Invalid
|
Medium
|
Unassigned | ||
webbrowser-app (Ubuntu) |
Triaged
|
Medium
|
Alexandre Abreu | ||
webbrowser-app (Ubuntu RTM) |
Triaged
|
Medium
|
Alexandre Abreu |
Bug Description
The new top-header takes too much space by default. It doesn't hide until a long page is scrolled.
Suggestion: auto-hiding, like the previous bottom bar, would probably make that less of an issue.
** UX Solution **
HTML 5 apps
may occasionally need back button functionality
I also agree with Alex that the optional header could be either fixed or
hide-out on scroll. That's what we specified.
https:/
It should behave similarly as the current Browser top bar.
https:/
Related branches
- Olivier Tilloy: Disapprove
- PS Jenkins bot: Needs Fixing (continuous-integration)
- Alexandre Abreu (community): Needs Fixing
-
Diff: 187 lines (+151/-2)3 files modifiedsrc/app/webcontainer/ChromeStateTracker.qml (+82/-0)
src/app/webcontainer/ScrollTracker.qml (+66/-0)
src/app/webcontainer/WebApp.qml (+3/-2)
Changed in webbrowser-app: | |
assignee: | nobody → Alexandre Abreu (abreu-alexandre) |
status: | New → Triaged |
Changed in ubuntu-ux: | |
assignee: | nobody → Giorgio Venturi (giorgio-venturi) |
Changed in ubuntu-ux: | |
status: | New → Triaged |
Changed in webbrowser-app: | |
importance: | Undecided → Medium |
Changed in ubuntu-ux: | |
importance: | Undecided → Medium |
tags: |
added: ota-1 removed: touch-2014-09-18 |
tags: |
added: touch-2014-10-09 removed: rtm14 |
tags: | added: rtm14 |
tags: |
added: touch-2014-10-23 removed: touch-2014-10-09 |
tags: |
added: touch-2014-10-30 removed: touch-2014-10-23 |
tags: |
added: touch-2014-11-06 removed: touch-2014-10-30 |
description: | updated |
Changed in ubuntu-ux: | |
status: | Won't Fix → Fix Committed |
tags: |
added: touch-2014-11-13 removed: touch-2014-11-06 |
tags: |
added: touch-2014-11-20 removed: touch-2014-11-13 |
tags: | removed: touch-2014-11-20 |
tags: | removed: ota-1 |
Changed in ubuntu-ux: | |
assignee: | Giorgio Venturi (giorgio-venturi) → James Mulholland (jamesjosephmulholland) |
Changed in webbrowser-app (Ubuntu): | |
assignee: | nobody → Alexandre Abreu (abreu-alexandre) |
Changed in webbrowser-app (Ubuntu RTM): | |
assignee: | nobody → Alexandre Abreu (abreu-alexandre) |
Changed in webbrowser-app (Ubuntu): | |
importance: | Undecided → Medium |
Changed in webbrowser-app (Ubuntu RTM): | |
importance: | Undecided → Medium |
Changed in webbrowser-app (Ubuntu): | |
status: | New → Triaged |
Changed in webbrowser-app (Ubuntu RTM): | |
status: | New → Triaged |
Changed in webbrowser-app: | |
assignee: | Alexandre Abreu (abreu-alexandre) → nobody |
status: | Triaged → Invalid |
The real issue is with most webapps passing the --enable- back-forward and/or --enable-addressbar command line switches to the container. Most webapps probably don’t need that any longer, but for backward compatibility I kept the mapping of those switches to showing the chrome.
Note that the new chrome is not intended to behave like the old one: either it’s shown, and then it autohide upon scrolling, or it’s hidden, and it nevers appears (the latter is the default, and probably what most webapps want).
There can’t be a swipe to reveal/hide gesture for the new chrome, as it would conflict with interaction with the indicators. The new chrome has the same behaviour as the new header component from the SDK, its visibility is tied to the contents of the page, not triggered by explicit manual interaction.
Also note that with the new container, even when there’s no chrome at all, there will always be a thin progress bar shown at the top while loading, which gives some feedback to the user that something is happening (IIRC the --enable-addressbar switch was initially introduced to work around the lack of an external progress indicator, since the progress bar used to be embedded inside the address bar).
So I think the fix is to assess for each webapp whether they really need the legacy command line switches, and remove them accordingly.