1043284 - Trigger the global search when the app title bar is tapped

Status: RESOLVED FIXED

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

Comment 1

[Etienne Segonzac (:etienne)]

Attachment: Gaia PR

r? Vivien for the window manager par
r? Kevin for the rocketbar part

The PR also includes the commit from bug 1042083 so you can test it (if you have app.theme-color set to true in your settings db)

Comment 2

[Kevin Grandon :kgrandon]

Comment on attachment 8461438 [details] [review]
Gaia PR

Not exactly sure what parts I'm supposed to review, but the code generally looks fine. Few things I noticed:

1 - Homescreen statusbar is not transparent when at the top, fix this if you can before landing.
2 - Style of focused rocketbar state is wrong, the text looks too light, the colors off. We can fix this as a follow-up though.

Comment 3

[Etienne Segonzac (:etienne)]

(In reply to Kevin Grandon :kgrandon from comment #2)
> Comment on attachment 8461438 [details] [review]
> Gaia PR
> 
> Not exactly sure what parts I'm supposed to review, but the code generally
> looks fine. Few things I noticed:
> 
> 1 - Homescreen statusbar is not transparent when at the top, fix this if you
> can before landing.

Mmmhh... I tested all the commits for this.
How old is your gecko?
The metachange support landed 1-2 weeks ago.

> 2 - Style of focused rocketbar state is wrong, the text looks too light, the
> colors off. We can fix this as a follow-up though.

Fixed!

Thanks again for the help with the tests!

Comment 4

[Kevin Grandon :kgrandon]

(In reply to Etienne Segonzac (:etienne) from comment #3)
> (In reply to Kevin Grandon :kgrandon from comment #2)
> Mmmhh... I tested all the commits for this.
> How old is your gecko?
> The metachange support landed 1-2 weeks ago.

After testing your branch and updating gecko today everything seemed to work well :) Solid R+ from me :)

Comment 5

[Vivien Nicolas (:vingtetun) (:21) - (NOT reading bugmails, needinfo? please)]

Comment on attachment 8461438 [details] [review]
Gaia PR

r+ with nits and I would like a followup to remove the fade-overlay here and on the homescreen as I don't think they are needed.

Comment 6

[Etienne Segonzac (:etienne)]

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