Closed Bug 718259 Opened 8 years ago Closed 8 years ago

Remove the resizer button in the Inspector Toolbar

Categories

(DevTools :: Inspector, defect)

x86
All
defect
Not set

Tracking

(Not tracked)

RESOLVED FIXED
Firefox 12

People

(Reporter: paul, Unassigned)

References

Details

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

Attachments

(1 file, 2 obsolete files)

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
Attached patch patch v1 (obsolete) — Splinter Review
Also, another problem with this resizer: opening the HTML panel move (a little) all the right buttons to the left.
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+
Whiteboard: [land-in-fx-team]
Attached patch patch v1 - rebased (obsolete) — Splinter Review
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+
(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]
Attached patch patch v1.1Splinter Review
Attachment #588714 - Attachment is obsolete: true
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
Attachment #589746 - Flags: review?(dao) → review+
Attachment #589156 - Attachment is obsolete: true
Whiteboard: [land-in-fx-team]
https://hg.mozilla.org/integration/fx-team/rev/96036564910e
Whiteboard: [land-in-fx-team] → [fixed-in-fx-team]
Blocks: 717924
https://hg.mozilla.org/mozilla-central/rev/96036564910e
Status: ASSIGNED → RESOLVED
Closed: 8 years ago
Resolution: --- → FIXED
Target Milestone: --- → Firefox 12
Product: Firefox → DevTools
You need to log in before you can comment on or make changes to this bug.