Closed Bug 862851 Opened 8 years ago Closed 3 years ago

Please swap the Domain and File columns in the network panel

Categories

(DevTools :: Netmonitor, defect, P3)

18 Branch
defect

Tracking

(firefox65 verified)

VERIFIED FIXED
Firefox 65
Tracking Status
firefox65 --- verified

People

(Reporter: st3fan, Assigned: aburone)

Details

(Whiteboard: [devtools-ux])

Attachments

(3 files, 2 obsolete files)

I find myself looking at the wrong place. Wouldn't it make sense to show domain first like it appears in URLs anyway?
Shorlander, what do you think about this? I tend to agree.
Flags: needinfo?(shorlander)
OS: Mac OS X → All
Hardware: x86 → All
It is more consistent with the way we typically read URLs. Although on the other hand the file is probably more relevant than the domain in a lot of cases.

I would flip them for now. I think we talked about potentially making them arrangeable/hiddeable in the future?
Flags: needinfo?(shorlander)
Priority: -- → P3
Whiteboard: [devtools-ux]
I think we should be consistent with Chrome DevTools in this case.
I.e. not swap and hide the domain column by default

Honza
Product: Firefox → DevTools
Attached patch bug862851.patch (obsolete) — Splinter Review
First patch hidding the Domain column by default
Sorry. Ignore the patch (can't delete it) at first looked like it worked but later not :(
Attached patch bug862851.patch (obsolete) — Splinter Review
Fixed.
The first patch worked but upon selecting "reset columns", domain would reappear.
ui.js stores the defaults.

I tried to reorder the columns to provide a 2nd patch emulating chrome distribution but couldn't do it. I suppose there's a 4th file setting the order that I can't find?
Attachment #9018807 - Attachment is obsolete: true
@Ariel, thanks for helping with this one!

Sorry for the confusion here. It looks like swapping the Domain and File columns is the right way to go in this report.

Can you please update your patch accordingly?

Honza
Flags: needinfo?(aburone)
The only file that really needs to be changed is constants.js. But
decided to change the order in the other files for consistency sake.
Comment on attachment 9018814 [details]
Summary: Hides Domain column from network panel by default

Done

The only file that really needed to be changed is constants.js. But
decided to change the order in the other files for consistency sake.
Flags: needinfo?(aburone)
Thanks for the patience.
I can't mark the older files as obsolete, I suppose I don't have permissions to do so.
Assignee: nobody → aburone
Status: NEW → ASSIGNED
Attached image image.png
The column-header-titles are swapped, but not the column content.
Honza
Flags: needinfo?(aburone)
(In reply to Jan Honza Odvarko [:Honza] need-info me from comment #12)
> Created attachment 9020732 [details]
> image.png
> 
> The column-header-titles are swapped, but not the column content.
> Honza

Sorry, I was distracted.

Here's the fix
https://phabricator.services.mozilla.com/D9989
Flags: needinfo?(aburone)
(In reply to Ariel Burone from comment #13)
> Here's the fix
> https://phabricator.services.mozilla.com/D9989

Great, thanks!

Couple of comments:

1) The patch looks good I am waiting for Try results
https://treeherder.mozilla.org/#/jobs?repo=try&revision=421c05334e0b7ac56e5307c569ddb5f78b234430
We can land if tests are green

2) Please mark all non-relevant or obsolete patches as obsolete (Attachment -> Details -> Edit Details -> Check Obsolete checkbox)

3) The patch in Phabricator doesn't have reviewer set. Also there is no need-info here. That means that you request for review is not in my queue and can be simply to be lost in the bug-mail.


Honza
(In reply to Jan Honza Odvarko [:Honza] (need-info? me) from comment #14)
> (In reply to Ariel Burone from comment #13)
> > Here's the fix
> > https://phabricator.services.mozilla.com/D9989
> 
> Great, thanks!
> 
> Couple of comments:
> 
> 1) The patch looks good I am waiting for Try results
> https://treeherder.mozilla.org/#/
> jobs?repo=try&revision=421c05334e0b7ac56e5307c569ddb5f78b234430
> We can land if tests are green
> 
> 2) Please mark all non-relevant or obsolete patches as obsolete (Attachment
> -> Details -> Edit Details -> Check Obsolete checkbox)

I can't :( don't have permissions to do so.
The only moment I cant mark as obsolete is when I upload am attachment.

> 
> 3) The patch in Phabricator doesn't have reviewer set. Also there is no
> need-info here. That means that you request for review is not in my queue
> and can be simply to be lost in the bug-mail.

Just found how to do so from phabricator. Thanks for the heads up!

> 
> 
> Honza

When I reply to a message like this. Do you see something different? or it's just another normal looking email?
Thanks again,
Ariel
(In reply to Ariel Burone from comment #15)
> I can't :( don't have permissions to do so.
> The only moment I cant mark as obsolete is when I upload am attachment.
I marked the attached patch as obsolete.
Can you remove the on Phabricator revision?

> > 3) The patch in Phabricator doesn't have reviewer set. Also there is no
> > need-info here. That means that you request for review is not in my queue
> > and can be simply to be lost in the bug-mail.
> 
> Just found how to do so from phabricator. Thanks for the heads up!
Excellent!


> When I reply to a message like this. Do you see something different? or it's
> just another normal looking email?
Not sure what you mean, bug my bug-mail (for this comment) looks the same as others...

New try push:
https://treeherder.mozilla.org/#/jobs?repo=try&revision=07b23dcd81ef67db553f5561866a1b4d5cfdbce9
(the previous was wrong)

Honza
> Not sure what you mean, bug my bug-mail (for this comment) looks the same as
> others...


Ok. Yes that's was I was asking. 
Thanks

About the obsoletes. The only moment I can mark as obsolete is if I attach a new file from Bugzilla. I couldn't find another way to do so. 


Odd thing:
Every time I used arc diff, it automatically associated the upload with the bug in bugzilla, but not this time, and I've got no errors at all.
> New try push:
> https://treeherder.mozilla.org/#/
> jobs?repo=try&revision=07b23dcd81ef67db553f5561866a1b4d5cfdbce9
Green

Honza

(In reply to Ariel Burone from comment #17)
> Every time I used arc diff, it automatically associated the upload with the
> bug in bugzilla, but not this time, and I've got no errors at all.

In order to update a patch in Phabricator you should use --update

Example:

# upload the last commit
arc diff .^ --update D3225

Honza
Pushed by jodvarko@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/c975dd62f9d3
Sumary: Bug 862851 - Swap the Domain and File columns in the network panel r=Honza
https://hg.mozilla.org/mozilla-central/rev/c975dd62f9d3
Status: ASSIGNED → RESOLVED
Closed: 3 years ago
Resolution: --- → FIXED
Target Milestone: --- → Firefox 65
Hello,

I reproduced this issue using Fx 65.0a1, build ID: 20181027100344, on MacOS 10.13.

I can confirm this issue is verified as fixed using Fx 65.0b5, build ID:  20181217180946, on Windows 10 x32, macOS X 10.13 and Ubuntu 18.04.
Status: RESOLVED → VERIFIED
You need to log in before you can comment on or make changes to this bug.