Closed
Bug 1160678
Opened 9 years ago
Closed 9 years ago
Pocket door hangers arent automatically closed
Categories
(Firefox :: Pocket, defect)
Firefox
Pocket
Tracking
()
VERIFIED
FIXED
Firefox 40
People
(Reporter: pdehaan, Assigned: jaws)
References
(Depends on 1 open bug)
Details
Attachments
(1 file)
1.40 KB,
patch
|
Dolske
:
review+
Dolske
:
approval-mozilla-aurora+
Sylvestre
:
approval-mozilla-release+
|
Details | Diff | Splinter Review |
(originally filed as https://github.com/Pocket/Firefox/issues/14) ### Steps to reproduce: 1. Sign in to pocket dev w/ verified FxA account 2. Click the Pocket button in the toolbar to add a link to your reading list. 3. Click the Pocket button in the toolbar AGAIN and click the "Remove Page" link. ### Actual results: The door hanger says "Page Removed" with a Pocket logo, but the door hanger never auto-closes. (see https://cloud.githubusercontent.com/assets/557895/7438443/395f1a4e-f019-11e4-9cf2-6d1515127a90.png) ### Expected results: The door hanger should auto close after 4-6 seconds (and ideally be a lot shorter since it only says "Page Removed"). ### Workaround: Click outside of the door hanger to dismiss it.
Comment 1•9 years ago
|
||
In diff'n the code from the patches and the add-on in the repo, it appears that the auto-hide code was commented out: https://github.com/Pocket/Firefox/compare/develop...compare/patch-update-1?expand=1#diff-73b7fce17bd3a7cb741602a692692902L109 Jared can we get this back in? Was there an issue with the code that we can work on resolving?
Assignee | ||
Comment 2•9 years ago
|
||
We can uncomment the code now. As part of bug 1159744 I had to change how getPanel() worked (the panel is a temporary panel when the button exists in the toolbar). We can now use getPanel() cleanly, and as such this code path now works.
Updated•9 years ago
|
Attachment #8601681 -
Flags: review?(dolske) → review+
Comment 4•9 years ago
|
||
https://hg.mozilla.org/mozilla-central/rev/799f4c06bbc8
Status: ASSIGNED → RESOLVED
Closed: 9 years ago
status-firefox40:
--- → fixed
Resolution: --- → FIXED
Target Milestone: --- → Firefox 40
Comment 6•9 years ago
|
||
Comment on attachment 8601681 [details] [diff] [review] Patch a+ for uplift in service of 38.0.5 spring launch campaign.
Attachment #8601681 -
Flags: approval-mozilla-beta+
Attachment #8601681 -
Flags: approval-mozilla-aurora+
Comment 7•9 years ago
|
||
Comment on attachment 8601681 [details] [diff] [review] Patch [Triage Comment] Fix the branch
Attachment #8601681 -
Flags: approval-mozilla-beta+ → approval-mozilla-release+
Updated•9 years ago
|
status-firefox38.0.5:
--- → affected
status-firefox39:
--- → affected
Updated•9 years ago
|
Flags: qe-verify+
QA Contact: andrei.vaida
Assignee | ||
Comment 8•9 years ago
|
||
https://hg.mozilla.org/releases/mozilla-aurora/rev/0cc41289449d
Assignee | ||
Comment 9•9 years ago
|
||
https://hg.mozilla.org/releases/mozilla-release/rev/53b766c68811
Comment hidden (typo) |
Comment 11•9 years ago
|
||
Confirmed fixed on Nightly 40.0a1 (2015-05-07), using Ubuntu 14.04 (x64), Windows 7 (x64) and Mac OS X 10.9.5, with Bug 1162033 encountered and being treated separately.
Comment 12•9 years ago
|
||
Verified as fixed using Firefox 38.0.5 beta 1 build 2 under Ubuntu 14.04 LTS 32-bit, Windows 7 64-bit and Mac OS X 10.9.5.
Comment 13•9 years ago
|
||
Verified fixed on Firefox 39 Beta 1 (20150523155636) under Windows 7 64-bit, Ubuntu 14.04 32-bit and Mac OS X 10.9.5. The confirmation/warning doorhangers are automatically closed after about 5 seconds.
You need to log in
before you can comment on or make changes to this bug.
Description
•