Right-click does not work in text-based netmonitor sidepanels
Categories
(DevTools :: Netmonitor, defect, P3)
Tracking
(firefox65 unaffected, firefox66 wontfix, firefox67 wontfix, firefox81 fixed)
Tracking | Status | |
---|---|---|
firefox65 | --- | unaffected |
firefox66 | --- | wontfix |
firefox67 | --- | wontfix |
firefox81 | --- | fixed |
People
(Reporter: uskolor, Assigned: bomsy, Mentored)
References
(Blocks 1 open bug)
Details
Attachments
(2 files)
User Agent: Mozilla/5.0 (Windows NT 6.2; Win64; x64; rv:67.0) Gecko/20100101 Firefox/67.0
Steps to reproduce:
Hello I would like ask about right mouse button.I can't copy website links at Web developer tools.
Actual results:
I should copy links at Web developer tools but I can't right button doesn't works
Picture show this issue .
Expected results:
In stable Firefox I can copy but at beta doesn't
Comment 1•6 years ago
|
||
Mozilla/5.0 (Windows NT 10.0; Win64; x64; rv:67.0) Gecko/20100101 Firefox/67.0 (20190204092937)
I've tested this report on Windows 10 and Ubuntu 18.10 using the latest Nightly, Beta and Fx release builds. I was able to reproduce the mentioned behavior. On the Firefox release build, I am able to successfully copy from DevTools. On the Beta and Nightly, I`m not able to do so. The right mouse click does not trigger the context menu.
Comment 2•6 years ago
|
||
I can confirm too, on macOS, with Firefox 66 and 67.
It does work with Firefox 65 though. So it seems like a regression.
Comment 3•6 years ago
|
||
Thanks for the report!
This looks like a regression from bug 1495797
@Lenka, can you confirm it?
Honza
Comment 4•6 years ago
|
||
(In reply to Jan Honza Odvarko [:Honza] (always need-info? me) from comment #3)
Thanks for the report!
This looks like a regression from bug 1495797
@Lenka, can you confirm it?Honza
I checked and it comes from this bug . Where <input> was replaced by <span> for the values.
I can try to have a look.
Lenka
Comment 5•6 years ago
|
||
Hey,
I'm and outreachy applicant and would like to work on this project.
Can you please guide me about the issue?
Thanks
Comment 6•6 years ago
|
||
(In reply to paarmita1998 from comment #5)
Hey,
I'm and outreachy applicant and would like to work on this project.
Can you please guide me about the issue?
Thanks for the help!
-
Assuming you have downloaded Firefox source already...
-
The Headers panel is implemented in this file:
https://searchfox.org/mozilla-central/source/devtools/client/netmonitor/src/components/HeadersPanel.js -
The URL is rendered here:
https://searchfox.org/mozilla-central/rev/dc0adc07db3df9431a0876156f50c65d580010cb/devtools/client/netmonitor/src/components/HeadersPanel.js#380 -
Here is an example of attaching a context menu to the "Header bar" (displayed at the top of the list of requests in the panel content)
https://searchfox.org/mozilla-central/rev/dc0adc07db3df9431a0876156f50c65d580010cb/devtools/client/netmonitor/src/components/RequestListHeader.js#173
Search for onContextMenu
in this file to see all interesting places.
-
The menu for the Header is implemented here:
https://searchfox.org/mozilla-central/source/devtools/client/netmonitor/src/widgets/RequestListHeaderContextMenu.js -
You should implement a new menu in widgets/HeadersPanelContextMenu.js (create the file) and attach it to the URL element
summaryUrl
(mentioned in step #2). Actually, you could pass new argumentonContextMenu
into therenderSummary
method and attach it to all values rendered usingrenderSummary
(i.e. Request URL, Request Method, address, etc.) -
The menu should have just one item: "Copy".
Updated•6 years ago
|
Comment 7•6 years ago
|
||
Hey,
Does this bug happen in MacOS?
Comment 8•6 years ago
|
||
Okay, I got the issue. Working on it.
Comment 9•6 years ago
|
||
Hi :Paarmita,
How's it going with this bug? Let me know if you need help or anything. I'd like to work on this bug - it would help me get more familiar with the code base.
Thanks!
Comment 10•6 years ago
|
||
Hey,
I'm working on this bug and almost done with it, needs to fix some bugs.
Comment 11•6 years ago
|
||
Hi Paarmita. Do you still intend to work on this bug? If so, no problem at all if you need more time. But I prefer to ask in case you do not have time anymore, in which case we can make this bug unassigned again so other people can pick it up.
Comment 12•6 years ago
|
||
Unassigning this bug.
Updated•5 years ago
|
Updated•5 years ago
|
Reporter | ||
Comment 13•5 years ago
|
||
After 1 year my submission this copy still doesn't works.In Poland we said Mozilla has pipe as in my Car .of course we have time .for next year.
Comment 14•5 years ago
|
||
We are working on refactoring this panel and should also get to this as a follow up.
Same applies to the Timings
panel.
Updated•5 years ago
|
Comment 15•5 years ago
|
||
Bugbug thinks this bug is a regression, but please revert this change in case of error.
Comment 16•5 years ago
|
||
As prior art, the Changes panel has a good copy menu that also allows copying the current selection. Text-based panels could hook into the same mechanism:
https://searchfox.org/mozilla-central/source/devtools/client/inspector/changes/ChangesContextMenu.js
Updated•5 years ago
|
Reporter | ||
Comment 17•5 years ago
|
||
Firefox 75 nothing change situation is the same .Doesn't works.
Assignee | ||
Comment 18•4 years ago
|
||
This should be fixed now by Bug 1631295.
Assignee | ||
Comment 19•4 years ago
|
||
iiiiiikolor@gmail.com this should be on nightly.
Please confirm that it works for you.
Thanks
Reporter | ||
Comment 20•4 years ago
|
||
Hubert in 76.0.1 this doesn't works .I do not use Nightly.
Comment 21•4 years ago
|
||
Bomsy, bug 1631295 partially fixes it for the URL section, but the other fields for Status, Address, etc don't have a context menu.
Assignee | ||
Comment 22•4 years ago
|
||
Add a new context menu for the headers sidepanel with , Copy and Copy all
Updated•4 years ago
|
Updated•4 years ago
|
Comment 24•4 years ago
|
||
Comment 25•4 years ago
|
||
bugherder |
Updated•4 years ago
|
Description
•