Closed
Bug 1619018
Opened 5 years ago
Closed 5 years ago
Copy in TreeView resets current row selection
Categories
(DevTools :: Netmonitor, defect, P3)
DevTools
Netmonitor
Tracking
(firefox75 verified)
VERIFIED
FIXED
Firefox 75
| Tracking | Status | |
|---|---|---|
| firefox75 | --- | verified |
People
(Reporter: Harald, Assigned: Harald)
References
(Blocks 1 open bug)
Details
Attachments
(1 file, 1 obsolete file)
|
(deleted),
text/x-phabricator-request
|
Details |
What were you doing?
- Open Network panel on https://tuner.cloud/manifest.json?1=2&3=4 and select
/manifest.json - In Security or Params panel right-click one of the rows, but not the first
What happened?
Not the current focused row but the first row gets selected. Copy works as expected though.
What should have happened?
Set the selection on the clicked row.
Anything else we should know?
Initial analysis is that TreeView.js' selectRow isn't meant to take an element, which PropertiesView.js is trying to use: https://searchfox.org/mozilla-central/source/devtools/client/netmonitor/src/components/request-details/PropertiesView.js#142
|
Assignee | |
Comment 1•5 years ago
|
||
|
||
Updated•5 years ago
|
Assignee: nobody → hkirschner
Status: NEW → ASSIGNED
|
|
Assignee | |
Comment 2•5 years ago
|
||
|
|
||
Updated•5 years ago
|
Attachment #9129901 -
Attachment is obsolete: true
Pushed by hkirschner@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/c9e6b5cf2987
Fix selection for right-click context menu in Security/Params/WS r=bomsy
|
||
Comment 4•5 years ago
|
||
| bugherder | ||
Status: ASSIGNED → RESOLVED
Closed: 5 years ago
status-firefox75:
--- → fixed
Resolution: --- → FIXED
Target Milestone: --- → Firefox 75
|
||
Updated•5 years ago
|
Flags: qe-verify+
|
||
Comment 5•5 years ago
|
||
Confirmed issue with 75.0b1 on macOS 10.15.
Fix verified with 75.0b5 on macOS 10.15, Windows 10, Ubuntu 19.04.
You need to log in
before you can comment on or make changes to this bug.
Description
•