1010739 - Toolbar refinement - Update curve shape to play better with Australis

Status: VERIFIED FIXED

(Firefox for Android Graveyard :: Awesomescreen, defect)

Comment 1

[Anthony Lam (:antlam) Find me, @antlam on Slack!]

Attachment: Screen Shot 2014-05-15 at 10.59.57 AM.png

Current tab curve on Android is shown in red, compared to the new adapted shape shown in blue.

Comment 2

[Anthony Lam (:antlam) Find me, @antlam on Slack!]

Attachment: Screen Shot 2014-05-15 at 11.11.42 AM.png

Brought the curve a bit closer to divide the space before the subsequent icon better and to hug the input field better

Comment 3

[Lucas Rocha (:lucasr)]

Attachment: Change curve shape to be more consistent with Australis (r=margaret)

Comment 4

[:Margaret Leibovic]

Comment on attachment 8423916 [details] [diff] [review]
Change curve shape to be more consistent with Australis (r=margaret)

Review of attachment 8423916 [details] [diff] [review]:
-----------------------------------------------------------------

I didn't build this to test, but I'll trust your math :)

Comment 5

[Lucas Rocha (:lucasr)]

Attachment: Change curve shape to be more consistent with Australis (r=margaret)

Comment 6

[Lucas Rocha (:lucasr)]

Comment on attachment 8471535 [details] [diff] [review]
Change curve shape to be more consistent with Australis (r=margaret)

Updated with the new tab shape we're planning to use on both tablet and phone UIs.

Comment 7

[Wesley Johnston (:wesj)]

Attachment: Comparison

I'm being an old man, but are we sure this is right? Overlapping this with Desktop's the two don't match. There's definitely more vertical stretch in the middle here than on desktop (i.e. if you zoom you can see a straight strip in the middle of this that isn't present in Desktop tabs).

Comment 8

[Lucas Rocha (:lucasr)]

https://hg.mozilla.org/integration/fx-team/rev/8474627f88d5

Comment 9

[Anthony Lam (:antlam) Find me, @antlam on Slack!]

(In reply to Wesley Johnston (:wesj) from comment #7)
> Created attachment 8473767 [details]
> Comparison
> 
> I'm being an old man, but are we sure this is right? Overlapping this with
> Desktop's the two don't match. There's definitely more vertical stretch in
> the middle here than on desktop (i.e. if you zoom you can see a straight
> strip in the middle of this that isn't present in Desktop tabs).

Good eye wesj! :D Maybe the title of this bug was a bit misleading but it should've been more about refining our current curve. It started off as an effort to map out a curve identical to Australis but in practice on a phone the curve was still awkward so this was about finding that happy medium.

Something we also looked at was how the curve hugged around the input box, how the inner shadow and highlight provided the visual affordance and worked with the aforementioned curve. Since we were trying out less corner radius and actually making it closer to modern conventions of the input box (like on Australis), the straighter edge actually helped.

All this was also in consideration of how it might look on tablets with our efforts to bring "tabs on top" to that side of the experience as well. These curves lent themselves better to both saving space and dividing content. :)

I should probably rename this bug... :P

Comment 10

[Ryan VanderMeulen [:RyanVM]]

Backed out along with the rest of the patch stack for robocop failures.
https://hg.mozilla.org/integration/fx-team/rev/d18d3e14ac3e

Comment 11

[Lucas Rocha (:lucasr)]

The string changes in bug 1010741 caused the failures. Forgot to update the tests accordingly. Try build looks nice and green now, pushed:

https://hg.mozilla.org/integration/fx-team/rev/03b97b1ea39b

Comment 12

[Ed Morley [:emorley]]

https://hg.mozilla.org/mozilla-central/rev/03b97b1ea39b

Comment 13

[Aaron Train [:aaronmt]]

The curve hype is real