The containers project needs UX elements that signify which container the user is currently in. As a first pass, we expect to add highlighting on tabs and address bar indicator icons/titles.

When you file bugs, can you either try to link them to other bugs that explain more of the project they're for using the blocks/"depends on" field, or ensure that the description/summary are understandable for people active in the area where you're filing the bug? (Right now I wonder what containers are, and what kind of highlighting this is, and what project this is for, and if it's related to UITour or not, and... well, I have a lot of questions, but "containers" is a very general term so it's not helpful if I have no context. :-) )

Gjis: I may have filed this in the

Gjis: I likely filed this bug in the wrong place. We are using this as a place holder for some UX mockups for a new content security project. I've changed it to Security.

Colour values (used to set container name on location bar): * Personal: #00a7e0 * Work: #f89c24 * Banking: #7dc14c * Shopping: #ee5195

Work and shopping icons may get confusing if people take the orange briefcase to be a shopping bag. Making the briefcase grey or black would solve this problem, except that black isn't all that pretty and grey won't work given the tab itself is grey. Maybe since the icon always shows up next to the name of the container, this is okay.

Looks like Bram and Francois created a wiki for this project! https://wiki.mozilla.org/Security/Contextual_Identity_Project/Containers Steve/Bram, once we have some more of this worked out, it would be helpful to update the wiki.

(In reply to Tanvi Vyas [:tanvi] from comment #10) > Steve/Bram, once we have some more of this worked out, it would be helpful > to update the wiki. I’ve started updating the wiki with the latest mockup and information. Feel free to continue building on top of it.

Bram: I just want to confirm that it is fine to land patches with these assets in Nightly. We'll hopefully have a few shortly (Bug 1191494). Thanks!

Yep. These icons should be fine for a first pass (ie. test to a limited audience, like Nightly and Dev Edition).

Updated all container icons to follow formatting outlined by Paolo in bug 1191455 comment 4. Steven, do they look good?

(In reply to Bram Pitoyo [:bram] from comment #19) > Updated all container icons to follow formatting outlined by Paolo in bug > 1191455 comment 4. > > Steven, do they look good? Awesome, thanks Bram! The `xmlns:xlink="http://www.w3.org/1999/xlink` namespace declaration is unnecessary since we aren't using xlinks. It looks like you stripped out all other unnecessary information. Also, I believe the viewBox should be `viewBox="0 0 32 32"` rather than `viewBox="0 0 16 16"` (If I understand the viewBox property correctly -- https://developer.mozilla.org/en-US/docs/Web/SVG/Attribute/viewBox).

* Strip out `xmlns:xlink` * Changed `viewBox` to `0 0 32 32` * Path declaration values changed to reflect the new dimension I think this should fulfil our requirements.

(In reply to Bram Pitoyo [:bram] from comment #25) > * Strip out `xmlns:xlink` > * Changed `viewBox` to `0 0 32 32` > * Path declaration values changed to reflect the new dimension > > I think this should fulfil our requirements. The updates look good for Personal, Banking, and Shopping. However the path is incorrect for work (briefcase handle not showing).

Whoops. Forgot to add `fill-rule="evenodd"`. Fixed.

This bug is complete. Only remaining item is the hamburger menu option which I will file a new bug for: https://bug1181953.bmoattachments.org/attachment.cgi?id=8640888

(In reply to Tanvi Vyas [:tanvi] from comment #28) > This bug is complete. Only remaining item is the hamburger menu option > which I will file a new bug for: > https://bug1181953.bmoattachments.org/attachment.cgi?id=8640888 Created https://bugzilla.mozilla.org/show_bug.cgi?id=1269029