Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(972)

Unified Diff: Source/devtools/front_end/InspectorView.js

Issue 197823010: [DevTools] Add minimum size to WebInspector.View. (Closed) Base URL: https://chromium.googlesource.com/chromium/blink.git@splitdip2
Patch Set: Created 6 years, 9 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: Source/devtools/front_end/InspectorView.js
diff --git a/Source/devtools/front_end/InspectorView.js b/Source/devtools/front_end/InspectorView.js
index cb1ffaa19252b8d982c28526bdc8d947fe3c0c27..8824319fe01f0d0cf4974e363a06fcb8a4233d0d 100644
--- a/Source/devtools/front_end/InspectorView.js
+++ b/Source/devtools/front_end/InspectorView.js
@@ -36,13 +36,12 @@ WebInspector.InspectorView = function()
{
WebInspector.VBox.call(this);
WebInspector.Dialog.setModalHostView(this);
+ this.setMinimumSize(new Size(180, 50));
// DevTools sidebar is a vertical split of panels tabbed pane and a drawer.
this._drawerSplitView = new WebInspector.SplitView(false, true, "Inspector.drawerSplitViewState", 200, 200);
this._drawerSplitView.hideSidebar();
this._drawerSplitView.enableShowModeSaving();
- this._drawerSplitView.setSidebarElementConstraints(Preferences.minDrawerHeight, Preferences.minDrawerHeight);
- this._drawerSplitView.setMainElementConstraints(25, 25);
this._drawerSplitView.show(this.element);
this._tabbedPane = new WebInspector.TabbedPane();
@@ -440,11 +439,31 @@ WebInspector.RootView = function()
{
WebInspector.VBox.call(this);
this.markAsRoot();
- this.element.classList.add("fill", "root-view");
+ this.element.classList.add("root-view");
this.element.setAttribute("spellcheck", false);
window.addEventListener("resize", this.doResize.bind(this), true);
};
WebInspector.RootView.prototype = {
+ attachToBody: function()
+ {
+ var overflowDiv = document.body.createChild("div");
+ overflowDiv.className = "fill";
+ overflowDiv.style.overflow = "hidden";
+ this.show(overflowDiv);
+ this.doResize();
+ },
+
+ doResize: function()
+ {
+ var size = this.minimumSize();
+ var width = Math.max(size.width, window.innerWidth);
+ this.element.style.width = width + "px";
+ var height = Math.max(size.height, window.innerHeight);
+ this.element.style.height = height + "px";
+
+ WebInspector.VBox.prototype.doResize.call(this);
+ },
+
__proto__: WebInspector.VBox.prototype
};

Powered by Google App Engine
This is Rietveld 408576698