Closed
Bug 976176
Opened 11 years ago
Closed 11 years ago
On Android 2.3 home banner will remain visible even when visibility set to GONE
Categories
(Firefox for Android Graveyard :: General, defect)
Tracking
(firefox29 fixed, firefox30 fixed, fennec29+)
RESOLVED
FIXED
Firefox 30
People
(Reporter: Margaret, Assigned: Margaret)
References
Details
Attachments
(2 files, 1 obsolete file)
1.34 KB,
patch
|
lucasr
:
review+
|
Details | Diff | Splinter Review |
2.08 KB,
patch
|
lucasr
:
review+
Sylvestre
:
approval-mozilla-aurora+
|
Details | Diff | Splinter Review |
I can reproduce on my Nexus S.
Assignee | ||
Updated•11 years ago
|
Assignee: nobody → margaret.leibovic
Assignee | ||
Updated•11 years ago
|
Summary: Home banner close button does not work on 2.3 → On Android 2.3 home banner will remain visible even when visibility set to GONE
Assignee | ||
Comment 2•11 years ago
|
||
Lucas, you probably have more experience with animation/visibility bugs on 2.3, so you may know a better solution, but I found this patch resolves this issue.
Attachment #8380814 -
Flags: review?(lucasr.at.mozilla)
Comment 3•11 years ago
|
||
Comment on attachment 8380814 [details] [diff] [review] Clear animations before trying to hide home banner on dismiss Review of attachment 8380814 [details] [diff] [review]: ----------------------------------------------------------------- ::: mobile/android/base/home/HomeBanner.java @@ +83,5 @@ > @Override > public void onClick(View view) { > + // On Android 2.3, setting the visibility to GONE won't actually hide the view > + // unless we clear animations first. > + HomeBanner.this.clearAnimation(); Seems like a known issue in pre-Honeycomb: http://stackoverflow.com/questions/16087913/unable-to-make-view-completely-gone-after-using-translateanimation I'd prefer something more 'future-proof' though. So, override HomeBanner's setVisibility() and do something like: @Override public void setVisibility(int visibility) { // On Android 2.3, setting the visibility to GONE won't actually // hide the view unless we clear animations first. if (visibility == View.GONE && Build.Version.VERSION_INT < 11) { clearAnimation(); } super.setVisibility(visibility); }
Attachment #8380814 -
Flags: review?(lucasr.at.mozilla) → feedback+
Assignee | ||
Comment 4•11 years ago
|
||
Attachment #8380814 -
Attachment is obsolete: true
Attachment #8381454 -
Flags: review?(lucasr.at.mozilla)
Assignee | ||
Comment 5•11 years ago
|
||
Attachment #8381455 -
Flags: review?(lucasr.at.mozilla)
Comment 6•11 years ago
|
||
Comment on attachment 8381455 [details] [diff] [review] (patch) Clear animations before trying to hide home banner on pre-honeycomb devices Review of attachment 8381455 [details] [diff] [review]: ----------------------------------------------------------------- Cool.
Attachment #8381455 -
Flags: review?(lucasr.at.mozilla) → review+
Updated•11 years ago
|
Attachment #8381454 -
Flags: review?(lucasr.at.mozilla) → review+
Assignee | ||
Comment 7•11 years ago
|
||
Landed a folded patch: https://hg.mozilla.org/integration/fx-team/rev/7ab155a7aa67
Comment 8•11 years ago
|
||
https://hg.mozilla.org/mozilla-central/rev/7ab155a7aa67
Status: NEW → RESOLVED
Closed: 11 years ago
Resolution: --- → FIXED
Target Milestone: --- → Firefox 30
Assignee | ||
Updated•11 years ago
|
tracking-fennec: ? → 29+
Assignee | ||
Updated•11 years ago
|
status-firefox29:
--- → affected
status-firefox30:
--- → fixed
Assignee | ||
Comment 9•11 years ago
|
||
Comment on attachment 8381455 [details] [diff] [review] (patch) Clear animations before trying to hide home banner on pre-honeycomb devices [Approval Request Comment] Bug caused by (feature/regressing bug #): snippets and promo banner User impact if declined: banner will not hide when it should on pre-honeycomb devices Testing completed (on m-c, etc.): landed on m-c 2/25, tested locally with gingerbread device Risk to taking this patch (and alternatives if risky): low-risk, tweaks some visibility code String or IDL/UUID changes made by this patch: none
Attachment #8381455 -
Flags: approval-mozilla-aurora?
Updated•11 years ago
|
Attachment #8381455 -
Flags: approval-mozilla-aurora? → approval-mozilla-aurora+
Assignee | ||
Comment 10•11 years ago
|
||
https://hg.mozilla.org/releases/mozilla-aurora/rev/d7744f9ef7ce
Updated•4 years ago
|
Product: Firefox for Android → Firefox for Android Graveyard
You need to log in
before you can comment on or make changes to this bug.
Description
•