Closed
Bug 1401943
Opened 7 years ago
Closed 7 years ago
reload button in overflow menu becomes inactive after tab detached from window
Categories
(Firefox :: Toolbars and Customization, defect, P1)
Tracking
()
VERIFIED
FIXED
Firefox 58
People
(Reporter: zalatik, Assigned: Gijs)
References
Details
(Whiteboard: [reserve-photon-structure])
Attachments
(1 file)
59 bytes,
text/x-review-board-request
|
jaws
:
review+
Sylvestre
:
approval-mozilla-beta+
|
Details |
User Agent: Mozilla/5.0 (X11; Linux x86_64; rv:57.0) Gecko/20100101 Firefox/57.0 Build ID: 20170920220431 Steps to reproduce: 1. Open firefox with several tabs open. 2. Pin reload button to overflow menu. 3. Detach a tab to separate window using mouse. Actual results: Reload button becomes inactive. It also remains inactive after unpinning from overflow menu. Expected results: Reload button remains active.
Status: UNCONFIRMED → NEW
Has STR: --- → yes
status-firefox57:
--- → affected
tracking-firefox57:
--- → ?
Component: Untriaged → Toolbars and Customization
Ever confirmed: true
OS: Unspecified → All
Hardware: Unspecified → All
Assignee | ||
Comment 1•7 years ago
|
||
(In reply to Maxim Zalata from comment #0) > Reload button becomes inactive. It also remains inactive after unpinning > from overflow menu. In the original window, the new window, or both? Generally, I suspect this is effectively a dupe of bug 1378819, even though the STR are different.
Reporter | ||
Comment 2•7 years ago
|
||
Only the new window. Also the button doesn't show 'stop' on page loading as in bug 1378819.
Flags: needinfo?(zalatik)
Updated•7 years ago
|
Flags: qe-verify+
Priority: -- → P4
QA Contact: gwimberly
Whiteboard: [photon-structure][triage] → [reserve-photon-structure]
Based on the fact that this was triaged as a P4, I am assuming this won't get fixed in 57. Please correct me if I am wrong. Moved tracking noms to 58.
Assignee | ||
Comment 4•7 years ago
|
||
(In reply to Ritu Kothari (:ritu) from comment #3) > Based on the fact that this was triaged as a P4, I am assuming this won't > get fixed in 57. Please correct me if I am wrong. It's unlikely, but we'll take a patch. It's just not a high priority given that: - customizing the button into the overflow panel is uncommon - tab detaching is uncommon - the button now (after bug 1378819) in my testing, at least, the button does start working in the new window/tab as soon as another page load happens.
Comment hidden (mozreview-request) |
Assignee | ||
Comment 6•7 years ago
|
||
Yoink. I tested if bug 1402493 fixed this (it didn't) and then realized fairly quickly what the problem was. X-ref bug 1388990.
Assignee: nobody → gijskruitbosch+bugs
Status: NEW → ASSIGNED
Priority: P4 → P1
Comment 7•7 years ago
|
||
mozreview-review |
Comment on attachment 8913628 [details] Bug 1401943 - make reload button work immediately after a tab detach, https://reviewboard.mozilla.org/r/185024/#review190128
Attachment #8913628 -
Flags: review?(jaws) → review+
Pushed by gijskruitbosch@gmail.com: https://hg.mozilla.org/integration/autoland/rev/c8bcfa26b552 make reload button work immediately after a tab detach, r=jaws
Comment 10•7 years ago
|
||
bugherder |
https://hg.mozilla.org/mozilla-central/rev/c8bcfa26b552
Status: ASSIGNED → RESOLVED
Closed: 7 years ago
Resolution: --- → FIXED
Target Milestone: --- → Firefox 58
Assignee | ||
Comment 11•7 years ago
|
||
Comment on attachment 8913628 [details] Bug 1401943 - make reload button work immediately after a tab detach, Approval Request Comment [Feature/Bug causing the regression]: we made the reload button movable for photon [User impact if declined]: in some edgecases it won't be enabled/disabled correctly [Is this code covered by automated tests?]: there are some basic tests for the behaviour of the button, but nothing specific to this particular bug [Has the fix been verified in Nightly?]: not yet [Needs manual test from QE? If yes, steps to reproduce]: see comment #0 [List of other uplifts needed for the feature/fix]: bug 1402493 (and bug 1378819, but that's already been uplifted) [Is the change risky?]: no [Why is the change risky/not risky?]: this is a very small change that only affects the initialization of the reload/stop buttons and ensuring they're in sync with their respective commands (ie whether refresh/stop are actually available at that point in time) when the window opens. [String changes made/needed]: nope
Attachment #8913628 -
Flags: approval-mozilla-beta?
Comment 12•7 years ago
|
||
Comment on attachment 8913628 [details] Bug 1401943 - make reload button work immediately after a tab detach, polish photon, taking it. Should be in 57b5
Attachment #8913628 -
Flags: approval-mozilla-beta? → approval-mozilla-beta+
Updated•7 years ago
|
Comment 13•7 years ago
|
||
bugherder uplift |
https://hg.mozilla.org/releases/mozilla-beta/rev/307f151ecf04
Comment 14•7 years ago
|
||
Verified with Windows 10 64bit, Mac OS X 10.11, Ubuntu 16.04 64bit using Nightly 58.0a1 (64-bit) as of this date.
Comment 15•7 years ago
|
||
Reproduced this issue using STR from comment 0, on 57.0a1 (20170921100141). Verified fixed on 57.0b6 (20171005195903) running Windows 10 x64, Mac OS X 10.11 and Ubuntu 16.04 x64.
You need to log in
before you can comment on or make changes to this bug.
Description
•