Ux Designs For Primary UI

RESOLVED FIXED

Status

()

Firefox for Android
General
P1
normal
RESOLVED FIXED
6 years ago
9 months ago

People

(Reporter: elan, Assigned: madhava)

Tracking

unspecified
ARM
Android
Points:
---

Firefox Tracking Flags

(firefox11 fixed, fennec11+)

Details

(Whiteboard: [QA+])

Attachments

(4 attachments)

(Reporter)

Description

6 years ago
Meta bug.
Whiteboard: [QA+]
(Reporter)

Updated

6 years ago
Assignee: nobody → madhava
Created attachment 569499 [details]
wire-mock of primary ui

More polished mocks and assets are on the way, but this should be enough to start with.
Duplicate of this bug: 696901
Created attachment 570120 [details] [diff] [review]
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.
Attachment #570120 - Flags: review?(mark.finkle)
Created attachment 570296 [details] [diff] [review]
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.
Attachment #570296 - Flags: review?(mark.finkle)
Created attachment 570298 [details]
Screenshot 1: Adding affordance to the url bar

This is a screenshot after applying Patch 2.
Attachment #570120 - Flags: review?(mark.finkle) → review+
Comment on attachment 570296 [details] [diff] [review]
Patch 2: Adding affordance to the browser bar

Waiting for ibarlow to ui+
Attachment #570296 - Flags: review?(mark.finkle)
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
Status: NEW → RESOLVED
Last Resolved: 6 years ago
Resolution: --- → FIXED
^ build failed 

https://tbpl.mozilla.org/php/getParsedLog.php?id=7084126&tree=Birch
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.
(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.
Depends on: 699302
tracking-fennec: --- → 11+
status-firefox11: --- → fixed
You need to log in before you can comment on or make changes to this bug.