Closed
Bug 1260043
Opened 9 years ago
Closed 8 years ago
"Toggle rulers for the page" is broken if toolbox is detached
Categories
(DevTools Graveyard :: Graphic Commandline and Toolbar, defect, P2)
DevTools Graveyard
Graphic Commandline and Toolbar
Tracking
(firefox48 affected, firefox49 fixed)
RESOLVED
FIXED
Firefox 49
People
(Reporter: arni2033, Unassigned)
References
Details
Attachments
(1 file)
>>> My Info: Win7_64, Nightly 48, 32bit, ID 20160326030430
STR:
1. Open http://example.org/ in a new tab
2. Open devtools, dock it to the bottom side of the window
3. Open https://ya.ru/ in a new tab (optional: open devtools in that tab)
4. Switch to the tab from Step 1, detach toolbox in separate window
5. Switch to the tab from Step 3
6. Switch to devtools window from Step 4
7. Click "Toggle rulers for the page" button
8. Switch to the tab from Step 1
AR:
No rulers
ER: Either X or Y or Z
X) After Step 8 rulers should be toggled for the tab from Step 1
Y) After Step 7 browser window should be placed below toolbox window and the tab from Step 1
should be selected
Z) After Step 7 an alert/popup panel/something else should say that tab from Step 1 is not visible
See also explanation of (Y) in bug 1251863 comment 6
Component: Developer Tools → Developer Tools: Graphic Commandline and Toolbar
Updated•9 years ago
|
Flags: needinfo?(zer0)
Updated•9 years ago
|
Priority: -- → P2
Updated•8 years ago
|
Comment 1•8 years ago
|
||
I filed bug 1267219, fixing that should fixes both this bug and bug 1260044.
Updated•8 years ago
|
Flags: needinfo?(zer0)
Comment 2•8 years ago
|
||
Review commit: https://reviewboard.mozilla.org/r/56916/diff/#index_header
See other reviews: https://reviewboard.mozilla.org/r/56916/
Attachment #8758703 -
Flags: review?(pbrosset)
Comment 3•8 years ago
|
||
Comment on attachment 8758703 [details]
MozReview Request: Bug 1260043 - removed `doc.hidden` check from markup/utils; r=pbro
I think Alex would be interested in reviewing this since he was the one who worked on this part of the code if I remember correctly.
Alex, you should read Bug 1267219 comment 11 before I think.
Attachment #8758703 -
Flags: review?(pbrosset) → review?(poirot.alex)
Updated•8 years ago
|
Attachment #8758703 -
Flags: review?(poirot.alex) → review+
Comment 4•8 years ago
|
||
Comment on attachment 8758703 [details]
MozReview Request: Bug 1260043 - removed `doc.hidden` check from markup/utils; r=pbro
https://reviewboard.mozilla.org/r/56916/#review53906
I can't find any safe check that isn't document.hidden.
Nor can I find of any real case where similar thing would happen on desktop.
But that's great to keep this comment in case it starts to throw again.
The only case where it may fail is for documents from the bfcache, but I don't see why we would have an highlighter against such document as TabActor throws events when we switch into/off the bfcache...
So if try is green, let's just remove this.
Comment 5•8 years ago
|
||
The orange doesn't seems related to this patch:
https://treeherder.mozilla.org/#/jobs?repo=try&revision=1d65faa064c5&selectedJob=21935870
So I'm going to land it. Thanks for the review Alex!
Pushed by mferretti@mozilla.com:
https://hg.mozilla.org/integration/fx-team/rev/99dacc787a59
removed `doc.hidden` check from markup/utils; r=ochameau
Comment 8•8 years ago
|
||
bugherder |
Status: NEW → RESOLVED
Closed: 8 years ago
status-firefox49:
--- → fixed
Resolution: --- → FIXED
Target Milestone: --- → Firefox 49
Updated•6 years ago
|
Product: Firefox → DevTools
Updated•6 years ago
|
Product: DevTools → DevTools Graveyard
You need to log in
before you can comment on or make changes to this bug.
Description
•