Gregor Wagner [:gwagner] Reporter
	Description • 10 years ago

Seen on 2.1 with yelp

	Guillaume Marty [:gmarty]
	Comment 1 • 10 years ago

When the menu button is hidden, the width must be adapted in the maximised state of the status bar only. The minimised state width should remain unchanged as the menu button is never visible.

Ben can you please have a look?

	Guillaume Marty [:gmarty]
	Comment 2 • 10 years ago

Comment on attachment 8497473 [details]
Github PR

Redirecting the review to Alive after talking to Dale.

	Guillaume Marty [:gmarty]
	Comment 3 • 10 years ago

Merged in https://github.com/mozilla-b2g/gaia/commit/db4125ff56efbe3e8951a6e41c1e53a47a46aa5b

	Michael Henretty [:mikehenrty][:mhenretty]
	Comment 4 • 10 years ago

Please request uplift.

	Michael Henretty [:mikehenrty][:mhenretty]
	Comment 5 • 10 years ago

Reverted for causing bug 1075826.

master: https://github.com/mozilla-b2g/gaia/commit/a7599bb1923dec439dd5e89c7dc9952050bfb132

	Guillaume Marty [:gmarty]
	Comment 6 • 10 years ago

This new patch works both with the browser and search apps and doesn't cause the side effects described in Bug 1075826.
Alive, could you please review it?

	Guillaume Marty [:gmarty]
	Comment 7 • 10 years ago

Landed in master https://github.com/mozilla-b2g/gaia/commit/ed7deb66e3b329ac0187d379b0c6070813d1d5de

	Guillaume Marty [:gmarty]
	Comment 8 • 10 years ago

Comment on attachment 8498803 [details]
Github PR

[Approval Request Comment]
[Bug caused by] (feature/regressing bug #): Rocket bar
[User impact] if declined: The rocket bar may overlap some of the icons in the status bar on 3rd party apps
[Testing completed]: Manual testing
[Risk to taking this patch] (and alternatives if risky): None as it's a minor CSS change
[String changes made]: None

	Ryan VanderMeulen [:RyanVM]
	Comment 9 • 10 years ago

v2.1: https://github.com/mozilla-b2g/gaia/commit/3cc2b505981b588b9811975302015e61537069f5

	Kevin Grandon :kgrandon Assignee
	Comment 10 • 10 years ago

Reverted for causing bug 1081138.

Master revert: https://github.com/mozilla-b2g/gaia/commit/ac3768b899f2f1d6f31a046f4b28d3f75bcad517
v2.1 revert: https://github.com/mozilla-b2g/gaia/commit/e52f99593e13c0c4cd6f11268af0cabd5643fecf

	Kevin Grandon :kgrandon Assignee
	Comment 11 • 10 years ago

Going to try a fix for this, stealing.

	Kevin Grandon :kgrandon Assignee
	Comment 12 • 10 years ago

Work in progress, looking at tests now.

	Kevin Grandon :kgrandon Assignee
	Comment 13 • 10 years ago

Comment on attachment 8503461 [details]
Github pull request

Hey Mike - do you want to take a look at this? I tried writing a test, but getting everything to be mocked in the right way for this was extremely difficult, and would probably take a few days. Being that we're coming up on FC, I'd like to get this landed then open a follow-up bug for a test.

	Michael Henretty [:mikehenrty][:mhenretty]
	Comment 14 • 10 years ago

Comment on attachment 8503461 [details]
Github pull request

FWIW, I'm not seeing bug 1081138 on master anymore even without this patch. But we are still seeing it on v2.1, and any case this patch fixes it on 2.1. Also measure vs hard coding seems like a good idea in it's own right, so r+.

The only thing that worries me somewhat about this change is that we now measure the rocketbar width every time we update icon visibility. With the frequency of network activity icon show/hide this might be a performance hit. It would be nice if we could cache this, but I won't block on it.

	Kevin Grandon :kgrandon Assignee
	Comment 15 • 10 years ago

Thanks. I'm still seeing this on master, but the patch is not quite right - it introduces some other problems. I might need to add a few small adjustments to it, and will re-open for review if needed.

	Kevin Grandon :kgrandon Assignee
	Comment 16 • 10 years ago

Ok - managed to get this into better shape and streamlined some caching into this on the appchrome collapse. Will land once gaia-try is green.

	Kevin Grandon :kgrandon Assignee
	Comment 17 • 10 years ago

In master: https://github.com/mozilla-b2g/gaia/commit/711daa290264a1ad9a7708fccca05f41d5482a49

	Ryan VanderMeulen [:RyanVM]
	Comment 18 • 10 years ago

v2.1: https://github.com/mozilla-b2g/gaia/commit/92e47f0978542aee75174f7fd45fcccadc666a55

	Chris Kreinbring [:CKreinbring]
	Comment 19 • 10 years ago

The bug still repros on Flame 2.2 and Flame 2.1 nightly.
Actual result: If there are enough status bar icons, they will overlap with the rocketbar when certain apps are open (mainly third party apps).

Flame 2.2
BuildID: 20141015040201
Gaia: 5f1f0960ae9d22acf2a324ad37a48174d6df87f6
Gecko: 62f0b771583c
Gonk: 52c909e821d107d414f851e267dedcd7aae2cebf
Platform Version: 36.0a1
Firmware Version: V180
User Agent: Mozilla/5.0 (Mobile; rv:36.0) Gecko/36.0 Firefox/36.0

Flame 2.1
BuildID: 20141015001201
Gaia: 379ea4c9dd6d3f8ca2f79ce59c15f6afe6e557c3
Gecko: 4853208cb48a
Gonk: 52c909e821d107d414f851e267dedcd7aae2cebf
Platform Version: 34.0
Firmware Version: V180
User Agent: Mozilla/5.0 (Mobile; rv:34.0) Gecko/34.0 Firefox/34.0

	Kevin Grandon :kgrandon Assignee
	Comment 20 • 10 years ago

I need to look into this and verify that the patch was right for v2.1.

	Kevin Grandon :kgrandon Assignee
	Comment 21 • 10 years ago

There was a lot of shifting code introduced by bug 1049439. It seems that no system patch is safe to uplift any more. Here is a branch patch which should fix the issue.

	Kevin Grandon :kgrandon Assignee
	Comment 22 • 10 years ago

Hi Chris,

We're about to land a branch-patch that will fix the issue on 2.1, but I'm unable to reproduce the issue on 2.2. Could you provide us a screenshot of what you are seeing on 2.2? Thanks!

	Chris Kreinbring [:CKreinbring]
	Comment 23 • 10 years ago

I was unable to repro the bug on today's 2.2 nightly, so I went back to the build where I previously reproed it and got the screenshot from there.
Actual result from today's build: When the rocketbar is present, icons that would overlap it do not appear.

BuildID: 20141020040204
Gaia: dc496d04907dd314f9736ff78bab3bd27156f79a
Gecko: ca6b46cbca3b
Gonk: 05aa7b98d3f891b334031dc710d48d0d6b82ec1d
Platform Version: 36.0a1
Firmware Version: V180
User Agent: Mozilla/5.0 (Mobile; rv:36.0) Gecko/36.0 Firefox/36.0

	Kevin Grandon :kgrandon Assignee
	Comment 24 • 10 years ago

Branch-patch landed in v2.1: https://github.com/mozilla-b2g/gaia/commit/44aa3a4c71f54dc32df59a640f299e4be342a351

Chris - can you try to re-verify this? Thanks!

	Chris Kreinbring [:CKreinbring]
	Comment 25 • 10 years ago

The bug still repros on today's 2.1 engineering build with shallow flash.
Actual result: If there are enough status bar icons, they will overlap with the rocketbar when certain apps are open (mainly third party apps).

BuildID: 20141020134135
Gaia: 44aa3a4c71f54dc32df59a640f299e4be342a351
Gecko: 45b98fde47a3
Platform Version: 34.0
Firmware Version: V180
User Agent: Mozilla/5.0 (Mobile; rv:34.0) Gecko/34.0 Firefox/34.0

	Yeojin Chung [:YeojinC]
	Comment 26 • 10 years ago

This issue does not reproduce on Flame 2.1 and 2.2 anymore.

Result: No overlapping occurs when the rocketbar is minimized. Only 4 icons are shown on the status bar.

Device: Flame 2.1 (319mb, KK, Shallow Flash)
BuildID: 20141111001201
Gaia: 7154f9aa0a69af56c8bd89be0c98d9791449527b
Gecko: 452db1a0c9a0
Version: 34.0 (2.1) 
Firmware Version: v188-1
User Agent: Mozilla/5.0 (Mobile; rv:34.0) Gecko/34.0 Firefox/34.0

Device: Flame 2.2 (319mb, KK, Shallow Flash)
BuildID: 20141111040202
Gaia: 6af3a8a833eb8bb651e8b188cb3f3c3a43bb4184
Gecko: 76b85b90a8cb
Version: 36.0a1 (2.2) 
Firmware Version: v188-1
User Agent: Mozilla/5.0 (Mobile; rv:36.0) Gecko/36.0 Firefox/36.0