Closed
Bug 991168
Opened 10 years ago
Closed 10 years ago
Increase the default width of the request details pane
Categories
(DevTools :: Netmonitor, defect)
DevTools
Netmonitor
Tracking
(Not tracked)
RESOLVED
FIXED
Firefox 31
People
(Reporter: vporof, Assigned: speaker)
Details
(Whiteboard: [good first bug][lang=js][mentor=vporof])
Attachments
(1 file)
588 bytes,
patch
|
vporof
:
review+
|
Details | Diff | Splinter Review |
Since bug 956357 landed, the default width of the right pane isn't quite enough for all the buttons, thus the HTML button ends up being bit squished. Increasing the default width will fix this.
Reporter | ||
Comment 1•10 years ago
|
||
This involves changing the devtools.netmonitor.panes-network-details-width pref in firefox.js to something like 550. Super easy good first bug.
Whiteboard: [good first bug][lang=js][mentor=vporof]
Assignee | ||
Comment 2•10 years ago
|
||
Hi i would like to take up this bug i have seen the file(mozilla-central/browser/app/profile) and made the change to 550 but unable to see any size change on the width of network monitor ( i am opening network monitor using ctrl + shift + Q )
Reporter | ||
Comment 3•10 years ago
|
||
Hi! Thanks for the interest. That is the correct change to make. To see the difference, navigate to a page that requests an html (which is probably easy to find :P) and inspect that request. The pane on the right should be slightly wider, and leave room for all the top buttons. Please attach a patch and ask for review.
Assignee: nobody → spkr322
Status: NEW → ASSIGNED
Assignee | ||
Comment 4•10 years ago
|
||
I have submitted the patch for the change i have done
Attachment #8401740 -
Flags: review+
Comment 5•10 years ago
|
||
Comment on attachment 8401740 [details] [diff] [review] patch for the change r?somebody@somewhere means "I'd like you to review this" r+ means "this was reviewed and was deemed ready to land in the codebase"
Attachment #8401740 -
Flags: review+ → review?(vporof)
Reporter | ||
Comment 6•10 years ago
|
||
Comment on attachment 8401740 [details] [diff] [review] patch for the change Review of attachment 8401740 [details] [diff] [review]: ----------------------------------------------------------------- Thanks! LGTM.
Attachment #8401740 -
Flags: review?(vporof) → review+
Reporter | ||
Updated•10 years ago
|
Keywords: checkin-needed
Comment 7•10 years ago
|
||
landed on fx-team. Welcome and thanks for contributing to Mozilla! https://hg.mozilla.org/integration/fx-team/rev/b619705ba480
Keywords: checkin-needed
Whiteboard: [good first bug][lang=js][mentor=vporof] → [good first bug][lang=js][mentor=vporof][fixed-in-fx-team]
Comment 8•10 years ago
|
||
https://hg.mozilla.org/mozilla-central/rev/b619705ba480
Status: ASSIGNED → RESOLVED
Closed: 10 years ago
Resolution: --- → FIXED
Whiteboard: [good first bug][lang=js][mentor=vporof][fixed-in-fx-team] → [good first bug][lang=js][mentor=vporof]
Target Milestone: --- → Firefox 31
Updated•6 years ago
|
Product: Firefox → DevTools
You need to log in
before you can comment on or make changes to this bug.
Description
•