Closed
Bug 718259
Opened 12 years ago
Closed 12 years ago
Remove the resizer button in the Inspector Toolbar
Categories
(DevTools :: Inspector, defect)
Tracking
(Not tracked)
RESOLVED
FIXED
Firefox 12
People
(Reporter: paul, Unassigned)
References
Details
(Whiteboard: [fixed-in-fx-team])
Attachments
(1 file, 2 obsolete files)
7.44 KB,
patch
|
dao
:
review+
|
Details | Diff | Splinter Review |
1) I am not sure this button is that useful. We don't need such a resizer anywhere else in Firefox. 2) Introducing this resizer force us to get it in the other tools (webconsole, debugger, ...) for consistency. That makes the UI quite heavy (especially when we will get the dock buttons) 3) The resizer-bar is certainly enough
Reporter | ||
Comment 1•12 years ago
|
||
Reporter | ||
Comment 2•12 years ago
|
||
Also, another problem with this resizer: opening the HTML panel move (a little) all the right buttons to the left.
Reporter | ||
Comment 3•12 years ago
|
||
Comment on attachment 588714 [details] [diff] [review] patch v1 Rob and Stephen, let me know what you think.
Attachment #588714 -
Flags: ui-review?(shorlander)
Attachment #588714 -
Flags: review?(rcampbell)
Comment 4•12 years ago
|
||
Comment on attachment 588714 [details] [diff] [review] patch v1 yeah, I'm fine with this change. Gets rid of a bunch of extra CSS and a control that is only of limited value. Plus, when we move the HTML panel, we'll be doing away with all of these anyway.
Attachment #588714 -
Flags: review?(rcampbell) → review+
Reporter | ||
Updated•12 years ago
|
Whiteboard: [land-in-fx-team]
Reporter | ||
Comment 5•12 years ago
|
||
Comment 6•12 years ago
|
||
Comment on attachment 588714 [details] [diff] [review] patch v1 Review of attachment 588714 [details] [diff] [review]: ----------------------------------------------------------------- I think removing it is the right way to go. I did notice that it removes the padding when the HTML panel is open: http://cl.ly/2A3l2s220r1s0y1S2O3S
Attachment #588714 -
Flags: ui-review?(shorlander) → ui-review+
Reporter | ||
Comment 7•12 years ago
|
||
(In reply to Stephen Horlander from comment #6) > Comment on attachment 588714 [details] [diff] [review] > patch v1 > > Review of attachment 588714 [details] [diff] [review]: > ----------------------------------------------------------------- > > I think removing it is the right way to go. > > I did notice that it removes the padding when the HTML panel is open: > http://cl.ly/2A3l2s220r1s0y1S2O3S Good catch! I'll update the patch.
Whiteboard: [land-in-fx-team]
Reporter | ||
Comment 8•12 years ago
|
||
Reporter | ||
Updated•12 years ago
|
Attachment #588714 -
Attachment is obsolete: true
Reporter | ||
Comment 9•12 years ago
|
||
Comment on attachment 589746 [details] [diff] [review] patch v1.1 A little explanation about this part of the patch (pinstripe only): diff --git a/browser/themes/pinstripe/browser.css b/browser/themes/pinstripe/browser.css #inspector-toolbar { border-top: 1px solid hsla(210, 8%, 5%, .65); - padding: 4px 16px 4px 0; /* use -moz-padding-end: 16px when/if bug 631729 gets fixed */ That was wrong in the first place. 0px of padding on the left was wrong on RTL. + padding-top: 4px; + padding-bottom: 4px; +} + +#inspector-toolbar:-moz-locale-dir(ltr) { + padding-left: 2px; + padding-right: 16px; /* use -moz-padding-end when/if bug 631729 gets fixed */ +} + +#inspector-toolbar:-moz-locale-dir(rtl) { + padding-left: 4px; + padding-right: 18px; /* use -moz-padding-end when/if bug 631729 gets fixed */ } These values are needed to make sure that the close button is aligned with the close button of the status bar. #inspector-toolbar[treepanel-open] { - padding: 0 0 4px; + padding-top: 0; + padding-right: 0; + -moz-padding-end: 4px; } Once the treepanel is open, we don't need the 16px padding anymore. In LTR, we need 2px on the left (to align with the status bar close-button), and 4px on the right (like any devtools toolbar). In RTL, we need 4px (like any devtools toobar) on the left, and 0px on the left (to align with the status bar close-button).
Attachment #589746 -
Flags: review?(dao)
Updated•12 years ago
|
Attachment #589746 -
Flags: review?(dao) → review+
Updated•12 years ago
|
Attachment #589156 -
Attachment is obsolete: true
Reporter | ||
Updated•12 years ago
|
Whiteboard: [land-in-fx-team]
Comment 11•12 years ago
|
||
https://hg.mozilla.org/integration/fx-team/rev/96036564910e
Whiteboard: [land-in-fx-team] → [fixed-in-fx-team]
Comment 12•12 years ago
|
||
https://hg.mozilla.org/mozilla-central/rev/96036564910e
Status: ASSIGNED → RESOLVED
Closed: 12 years ago
Resolution: --- → FIXED
Target Milestone: --- → Firefox 12
Updated•6 years ago
|
Product: Firefox → DevTools
You need to log in
before you can comment on or make changes to this bug.
Description
•