The default bug view has changed. See this FAQ.

Remove the resizer button in the Inspector Toolbar

RESOLVED FIXED in Firefox 12

Status

()

Firefox
Developer Tools: Inspector
RESOLVED FIXED
5 years ago
5 years ago

People

(Reporter: paul, Unassigned)

Tracking

Trunk
Firefox 12
x86
All
Points:
---

Firefox Tracking Flags

(Not tracked)

Details

(Whiteboard: [fixed-in-fx-team])

Attachments

(1 attachment, 2 obsolete attachments)

(Reporter)

Description

5 years ago
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

5 years ago
Created attachment 588714 [details] [diff] [review]
patch v1
(Reporter)

Comment 2

5 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

5 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 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

5 years ago
Whiteboard: [land-in-fx-team]
(Reporter)

Comment 5

5 years ago
Created attachment 589156 [details] [diff] [review]
patch v1 - rebased
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

5 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

5 years ago
Created attachment 589746 [details] [diff] [review]
patch v1.1
(Reporter)

Updated

5 years ago
Attachment #588714 - Attachment is obsolete: true
(Reporter)

Comment 9

5 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)
setting to ASSIGNED.
Status: NEW → ASSIGNED

Updated

5 years ago
Attachment #589746 - Flags: review?(dao) → review+

Updated

5 years ago
Attachment #589156 - Attachment is obsolete: true
(Reporter)

Updated

5 years ago
Whiteboard: [land-in-fx-team]
https://hg.mozilla.org/integration/fx-team/rev/96036564910e
Whiteboard: [land-in-fx-team] → [fixed-in-fx-team]
(Reporter)

Updated

5 years ago
Blocks: 717924
https://hg.mozilla.org/mozilla-central/rev/96036564910e
Status: ASSIGNED → RESOLVED
Last Resolved: 5 years ago
Resolution: --- → FIXED
Target Milestone: --- → Firefox 12
You need to log in before you can comment on or make changes to this bug.