Closed
Bug 1403530
Opened 7 years ago
Closed 6 years ago
Export as HAR toolbar button
Categories
(DevTools :: Netmonitor, enhancement, P3)
DevTools
Netmonitor
Tracking
(relnote-firefox -, firefox61 fixed)
RESOLVED
FIXED
Firefox 61
People
(Reporter: Honza, Assigned: Honza)
References
(Blocks 1 open bug)
Details
(Keywords: dev-doc-complete)
Attachments
(3 files)
The HAR export feature is a bit hidden since it's only available in Net pane's context menu.
There are currently two actions:
1) Copy All as HAR (to the clipboard)
2) Save All as HAR (to a file)
The new toolbar button should duplicate the #2 'Save All as HAR' action. Using label 'Export' and tooltip 'Save All as HAR'?
There should be a drop down menu associated with the button - offering an option:
* Export with content
(off by default)
Here is how the UI looked in Firebug:
http://www.softwareishard.com/blog/netexport/
Honza
Comment 1•7 years ago
|
||
I like it, the dropdown button nicely hides the complexity.
Do we have prior art for such a button existing in devtools? The other idea would be that export triggers a second bar, like the filter option in Console.
Since we have the pref, maybe we could also expose `Auto Export` in the dropdown. `Copy to Clipboard` could also be in those prefs.
Assignee | ||
Updated•7 years ago
|
Severity: normal → enhancement
Priority: -- → P3
Assignee | ||
Updated•7 years ago
|
Blocks: netmonitor-har
Updated•7 years ago
|
Keywords: dev-doc-needed
Assignee | ||
Updated•6 years ago
|
Assignee: nobody → odvarko
Status: NEW → ASSIGNED
Assignee | ||
Comment 2•6 years ago
|
||
Victoria, the mockup for HAR toolbar button shows a 'double arrow' icon (within the button). Do we have that icon?
https://mozilla.invisionapp.com/share/TAGK6KIMFR3#/screens
Honza
Flags: needinfo?(victoria)
Comment hidden (mozreview-request) |
Comment 4•6 years ago
|
||
(In reply to Jan Honza Odvarko [:Honza] from comment #2)
> Victoria, the mockup for HAR toolbar button shows a 'double arrow' icon
> (within the button). Do we have that icon?
> https://mozilla.invisionapp.com/share/TAGK6KIMFR3#/screens
>
> Honza
They're used in the responsive design mode: https://dxr.mozilla.org/mozilla-central/source/devtools/client/responsive.html/images/select-arrow.svg
Comment 5•6 years ago
|
||
Ah yes, thanks :ntim - for my mockups I made a larger/more spaced out version of the icon; however, I just compared the two icons and realized that the existing icon looks fine once resized to 5px wide/9px tall and colored darker (Gray 90), so we can just continue using that icon.
Flags: needinfo?(victoria)
Comment hidden (mozreview-request) |
Assignee | ||
Comment 7•6 years ago
|
||
reply to Victoria Wang [:victoria] from comment #5)
> Ah yes, thanks :ntim - for my mockups I made a larger/more spaced out
> version of the icon; however, I just compared the two icons and realized
> that the existing icon looks fine once resized to 5px wide/9px tall and
> colored darker (Gray 90), so we can just continue using that icon.
@victoria, ntim: Thanks!
Two questions:
1) The HAR button is a drop-down button (it shows a menu when clicked, see the screenshot). In case of RDM throttle it's a select button. Is the icon correct? Just checking.
2) What do you mean by Gray 90 ? What should be the CSS value?
Honza
Flags: needinfo?(victoria)
Comment 8•6 years ago
|
||
1: Yes, it's fine for now - we're using it for the "refresh" dropdown in RDM as well. I see what you mean though, it's a good point - I'll file this as a UX polish task to think about later.
2: Gray 90 refers to one of the Firefox Photon tokens which can be found at the bottom of https://searchfox.org/mozilla-central/source/devtools/client/themes/variables.css
From the looks of your screenshot I realized Gray 90 will probably be a bit too dark, so instead I would suggest --grey-90-alpha-80. Gray-40 can be used in dark mode.
Flags: needinfo?(victoria)
Comment hidden (mozreview-request) |
Comment hidden (mozreview-request) |
Comment 11•6 years ago
|
||
mozreview-review |
Comment on attachment 8969641 [details]
Bug 1403530 - HAR export toolbar button;
https://reviewboard.mozilla.org/r/238442/#review245036
Works as intended. The only suggestion I have is adjusting the toolbar button text position, as it's slightly above the text to its left, but not a blocker.
Attachment #8969641 -
Flags: review?(dwalsh) → review+
Assignee | ||
Comment 12•6 years ago
|
||
(In reply to David Walsh :davidwalsh from comment #11)
> Comment on attachment 8969641 [details]
> Bug 1403530 - HAR export toolbar button;
>
> https://reviewboard.mozilla.org/r/238442/#review245036
>
Thanks for the review David!
> Works as intended. The only suggestion I have is adjusting the toolbar
> button text position, as it's slightly above the text to its left, but not a
> blocker.
Can you please attach a screenshot? I don't see this on my win machine.
Honza
Flags: needinfo?(dwalsh)
Comment hidden (mozreview-request) |
Assignee | ||
Comment 14•6 years ago
|
||
Flags: needinfo?(dwalsh)
Assignee | ||
Comment 15•6 years ago
|
||
(In reply to David Walsh :davidwalsh from comment #11)
> The only suggestion I have is adjusting the toolbar
> button text position, as it's slightly above the text to its left, but not a
> blocker.
Thanks for testing David, fixed!
Honza
Comment 16•6 years ago
|
||
Pushed by jodvarko@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/b5a1430d99b5
HAR export toolbar button; r=davidwalsh
Comment 17•6 years ago
|
||
Backed out for failing clipboard at devtools/client/netmonitor/src/har/test/browser_net_har_copy_all_as_har.js - note that this does not fail on Linux debug
Push that started the failures: https://treeherder.mozilla.org/#/jobs?repo=autoland&revision=b5a1430d99b50108bf7f1f44ec9c29f0a544fd7c
Failure log: https://treeherder.mozilla.org/logviewer.html#?job_id=175674762&repo=autoland&lineNumber=3625
Backout: https://hg.mozilla.org/integration/autoland/rev/56084ee1029acf0a608eb8122d5451354ddee7fd
Flags: needinfo?(odvarko)
Comment hidden (mozreview-request) |
Assignee | ||
Comment 19•6 years ago
|
||
Test failures fixed, trying to land again.
Honza
Flags: needinfo?(odvarko)
Comment 20•6 years ago
|
||
Pushed by jodvarko@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/d8bb9fa651c6
HAR export toolbar button; r=davidwalsh
Comment 21•6 years ago
|
||
bugherder |
Status: ASSIGNED → RESOLVED
Closed: 6 years ago
status-firefox61:
--- → fixed
Resolution: --- → FIXED
Target Milestone: --- → Firefox 61
Assignee | ||
Comment 22•6 years ago
|
||
Release Note Request (optional, but appreciated)
[Why is this notable]: New UI for the Network panel in DevTools toolbox.
[Affects Firefox for Android]: no
[Suggested wording]: DevTools Network panel introduces a new toolbar drop-down menu button that allows export as well as import of HAR files. These actions where a bit hidden behind a context menu and drag-and-drop, so this should increase discoverability.
[Links (documentation, blog post, etc)]: There is a follow up bug 1456129 that adds "Import" action into this menu
Honza
relnote-firefox:
--- → ?
Updated•6 years ago
|
Comment 23•6 years ago
|
||
The more I think about this, the more I think this is better suited for the developer notes on MDN and not the overall Fx61 release notes. Sorry for the confusion.
Updated•6 years ago
|
Product: Firefox → DevTools
Comment 24•6 years ago
|
||
I've documented this, first by covering it in the following section:
https://developer.mozilla.org/en-US/docs/Tools/Network_Monitor#CopySave_All_As_HAR
And second by adding a note to the Fx61 rel notes:
https://developer.mozilla.org/en-US/Firefox/Releases/61#Developer_tools
Let me know if you think that looks OK. Thanks!
Flags: needinfo?(odvarko)
Keywords: dev-doc-needed → dev-doc-complete
Comment 26•6 years ago
|
||
Current implementation is not good enough/flexible enough.
Actually I don't even understand real use case for this implementation.
Imagine, I need to save only a few records(usually some API calls) out of 1000(which will include not needed css/js/jps/etc) for further analysis.
There should be possibility to save/export only selected records
Comment 27•6 years ago
|
||
You should file a new bug, because the changes made here already shipped with Firefox 61.
Sebastian
You need to log in
before you can comment on or make changes to this bug.
Description
•