696461 - Ux Designs For Primary UI

Status: RESOLVED FIXED

(Firefox for Android Graveyard :: General, defect, P1)

Description

[Erin Lancaster [:elan]]

Meta bug.

Comment 1

[Ian Barlow (:ibarlow)]

Attachment: wire-mock of primary ui

More polished mocks and assets are on the way, but this should be enough to start with.

Comment 3

[Sriram Ramasubramanian [:sriram]]

Attachment: Patch 1: Reskinning the primary ui

This reskins the primary UI.

The values have been moved out of styles for 3 reasons:
1. There are issues with the order each attribute is applied in a relative layout-- and styles are confusing it a bit.
2. The amount of reusability has reduced, and it makes sense to cut down the styles.
3. The relative layout uses id's for placing the view items, and it better not placed in styles.

Todos:
1. The "Go" button in awesomebar
2. A back button in awesomebar -- in the place of "tabs" button
3. A lock button in browser-toolbar -- same as "Go" button.
--> This will be added as a separate patch once the resources are available for the same.

Comment 4

[Sriram Ramasubramanian [:sriram]]

Attachment: Patch 2: Adding affordance to the browser bar

I felt the address bar not having enough affordance to tap on it. And also a visual glitch when a text box with orange border is shown when we tap on it.
I tried adding back the original design of having text box background for the button.

Pros: 
  Has good affordance and removes the visual glitch while transitioning from address bar to editing it.

Cons:
  Loss of the simple, smooth and elegant design without the "bounding" box for the button.

Ian can decide on having this. Will be adding a screenshot of the same in a bit.

Comment 5

[Sriram Ramasubramanian [:sriram]]

Attachment: Screenshot 1: Adding affordance to the url bar

This is a screenshot after applying Patch 2.

Comment 6

[Mark Finkle (:mfinkle) (use needinfo?)]

Comment on attachment 570296 [details] [diff] [review]
Patch 2: Adding affordance to the browser bar

Waiting for ibarlow to ui+

Comment 7

[Mark Finkle (:mfinkle) (use needinfo?)]

pushed the first patch:
https://hg.mozilla.org/projects/birch/rev/878512b3be8a

I think we should close this bug and move any other patches/ideas to new bugs

Comment 8

[Aaron Train [:aaronmt]]

^ build failed 

https://tbpl.mozilla.org/php/getParsedLog.php?id=7084126&tree=Birch

Comment 9

[Sriram Ramasubramanian [:sriram]]

Are we having a clean build? I reused a name with a different resource. That's causing the failure. If the obj directory is removed and compiled, the build is fine.

Comment 10

[Mark Finkle (:mfinkle) (use needinfo?)]

(In reply to Aaron Train [:aaronmt] from comment #8)
> ^ build failed 
> 
> https://tbpl.mozilla.org/php/getParsedLog.php?id=7084126&tree=Birch

I tried using the Mozilla Clobberer, but it doesn't seem to have worked.