1168961 - [User Story] View Site Name

Status: RESOLVED FIXED

(Firefox OS Graveyard :: Gaia::System::Browser Chrome, defect)

Description

[Ben Francis [:benfrancis]]

As a user I want to view the name/domain name of the current site, so I know what site I'm on.

Should use app name from manifest, application-name meta tag or hostname. Not document title.

Comment 1

[ffos-browser]

A Pivotal Tracker story has been created for this Bug: https://www.pivotaltracker.com/story/show/95560448

Comment 2

[Ben Francis [:benfrancis]]

Attachment: WIP Patch

Comment 3

[Ben Francis [:benfrancis]]

Attachment: Patch

Hi Alive, would you be able to take a look at this?

This is the first part of the Pinning the Web [1] work to merge app windows and browser windows. All windows show either the app name or hostname in the Rocketbar title, the document title of individual pages will be used elsewhere (e.g. in the task manager).

This patch moves a lot of the state management out of AppChrome into AppWindow and simplifies the name/title handling to two separate properties.

I've fixed the unit tests but I still need to fix integration and UI tests before this can land (I can't currently get them to run locally).

Please let me know what you think!

1. https://wiki.mozilla.org/FirefoxOS/Pinning_the_Web

Comment 4

[(inactive after 6/18) Alive Kuo [:alive]]

Gimme some time to read or let's see if Etienne has time to review.

Comment 5

[(inactive after 6/18) Alive Kuo [:alive]]

Comment on attachment 8621156 [details] [review]
Patch

Will read tomorrow but let's see if Etienne could pick up before I finished

Comment 6

[(inactive after 6/18) Alive Kuo [:alive]]

Comment on attachment 8621156 [details] [review]
Patch

Basically looks fine, but I think this will change the behavior when the page location is changed. Is this defined somewhere?

Comment 7

[Etienne Segonzac (:etienne)]

Comment on attachment 8621156 [details] [review]
Patch

Made a few comments on github, would like to take one last look after that.
Do we have any UX support to go though the behavior changes? (At least the "Search the web" removal)

Comment 8

[Ben Francis [:benfrancis]]

Comment on attachment 8621156 [details] [review]
Patch

Thanks Alive and Etienne, I'm currently blocked on getting integration tests running locally but will address the review comments as soon as I can.

Jacqueline, in the mean time can you confirm that it's OK to go ahead with the UI changes as per the Pin the Web presentation:
* Unfocused Rocketbar always shows app name (or hostname if no app name provided), for both app windows and browser windows
* The page title is never used to represent a site, but will be used to represent individual pages in the task manager (and in future edge swipe overlays and pinned pages)

Comment 9

[Ben Francis [:benfrancis]]

I've addressed the review comments, just waiting to rebase on bug 1174813 (which is going to conflict a little) before finishing off the tests. (I can run integration tests now!)

Comment 10

[Ben Francis [:benfrancis]]

Rebased after bug 1174813 landed and fixed unit tests including review comments. Now just integration tests to go...

Comment 11

[Ben Francis [:benfrancis]]

Comment on attachment 8621156 [details] [review]
Patch

Etienne, can you give this another check over?

I've can't actually run integration tests locally at the moment but I think I may have managed to fix the failing browser chrome tests on Treeherder...

Comment 12

[Ben Francis [:benfrancis]]

Comment on attachment 8621156 [details] [review]
Patch

Francis, are you taking over the UX on this?

I'm just looking for a quick UI review on a change to the way site names are displayed.

The designs we're currently working from display an application name in the title bar from a page's meta tags or manifest where provided, but will fall back to showing the hostname for web sites with no app metadata. This is instead of showing the document title.

The thinking is that this system wide title bar always shows the site/app name alongside the site/app icon (in both expanded and collapsed mode), the document title is used elsewhere to identify individual pages of a site/app, like in the task manager or on pinned cards.

This helps with consistently separating site metadata from page metadata across the OS. The hostname or app name will usually fit in the title bar, whereas document titles currently usually have to be truncated due to a lack of space. Showing the hostname instead of the document title in browser chrome is similar to what Safari does and I'm sure I've seen this in one of our UX specs at some point, I just wanted you to sanity check it before we land.

Thanks

Comment 13

[Etienne Segonzac (:etienne)]

Comment on attachment 8621156 [details] [review]
Patch

Sorry it took so long. But spending time on the initial pin-the-web patches will probably help me do subsequent reviews faster :)

Happy with this patch and the integration test failure looks trivial to fix (another title value to change), so this is officially in the hands for Francis :)

Comment 14

[Ben Francis [:benfrancis]]

OK, this patch has been hanging around for a month now and waiting for UI review for over 3 weeks. It matches the UX spec we're currently working from and Treeherder is green so in the interests of moving forward I'm going to go ahead and land it before it bitrots again.

Let's get feedback and then iterate, thanks.

Comment 15

[Ben Francis [:benfrancis]]

https://github.com/mozilla-b2g/gaia/commit/dda7775cea807343b69ef233a41bbddaba48620e

Comment 16

[Ben Francis [:benfrancis]]

Attachment: View Site Name - Specification