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

Side by Side Diff: third_party/WebKit/Source/devtools/front_end/ui/SplitWidget.js

Issue 2207283002: DevTools: make sidebar section headers wider. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Created 4 years, 4 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 unified diff | Download patch
OLDNEW
1 /* 1 /*
2 * Copyright (C) 2012 Google Inc. All rights reserved. 2 * Copyright (C) 2012 Google Inc. All rights reserved.
3 * 3 *
4 * Redistribution and use in source and binary forms, with or without 4 * Redistribution and use in source and binary forms, with or without
5 * modification, are permitted provided that the following conditions are 5 * modification, are permitted provided that the following conditions are
6 * met: 6 * met:
7 * 7 *
8 * 1. Redistributions of source code must retain the above copyright 8 * 1. Redistributions of source code must retain the above copyright
9 * notice, this list of conditions and the following disclaimer. 9 * notice, this list of conditions and the following disclaimer.
10 * 10 *
(...skipping 459 matching lines...) Expand 10 before | Expand all | Expand 10 after
470 return; 470 return;
471 471
472 if (!this._resizerElementSize) 472 if (!this._resizerElementSize)
473 this._resizerElementSize = this._isVertical ? this._resizerElement.o ffsetWidth : this._resizerElement.offsetHeight; 473 this._resizerElementSize = this._isVertical ? this._resizerElement.o ffsetWidth : this._resizerElement.offsetHeight;
474 474
475 // Invalidate layout below. 475 // Invalidate layout below.
476 476
477 this._removeAllLayoutProperties(); 477 this._removeAllLayoutProperties();
478 478
479 // this._totalSizeDIP is available below since we successfully applied c onstraints. 479 // this._totalSizeDIP is available below since we successfully applied c onstraints.
480 var sidebarSizeValue = WebInspector.zoomManager.dipToCSS(sizeDIP) + "px" ; 480 var roundSizeCSS = Math.round(WebInspector.zoomManager.dipToCSS(sizeDIP) );
481 var mainSizeValue = (this._totalSizeCSS - WebInspector.zoomManager.dipTo CSS(sizeDIP)) + "px"; 481 var sidebarSizeValue = roundSizeCSS + "px";
482 var mainSizeValue = (this._totalSizeCSS - roundSizeCSS) + "px";
482 this._sidebarElement.style.flexBasis = sidebarSizeValue; 483 this._sidebarElement.style.flexBasis = sidebarSizeValue;
483 484
484 // Make both sides relayout boundaries. 485 // Make both sides relayout boundaries.
485 if (this._isVertical) { 486 if (this._isVertical) {
486 this._sidebarElement.style.width = sidebarSizeValue; 487 this._sidebarElement.style.width = sidebarSizeValue;
487 this._mainElement.style.width = mainSizeValue; 488 this._mainElement.style.width = mainSizeValue;
488 this._sidebarElement.style.height = this._totalSizeOtherDimensionCSS + "px"; 489 this._sidebarElement.style.height = this._totalSizeOtherDimensionCSS + "px";
489 this._mainElement.style.height = this._totalSizeOtherDimensionCSS + "px"; 490 this._mainElement.style.height = this._totalSizeOtherDimensionCSS + "px";
490 } else { 491 } else {
491 this._sidebarElement.style.height = sidebarSizeValue; 492 this._sidebarElement.style.height = sidebarSizeValue;
(...skipping 424 matching lines...) Expand 10 before | Expand all | Expand 10 after
916 this._showHideSidebarButton.classList.toggle("toggled-hide", !sidebarHid den); 917 this._showHideSidebarButton.classList.toggle("toggled-hide", !sidebarHid den);
917 this._showHideSidebarButton.classList.toggle("top-sidebar-show-hide-butt on", !this.isVertical() && !this.isSidebarSecond()); 918 this._showHideSidebarButton.classList.toggle("top-sidebar-show-hide-butt on", !this.isVertical() && !this.isSidebarSecond());
918 this._showHideSidebarButton.classList.toggle("right-sidebar-show-hide-bu tton", this.isVertical() && this.isSidebarSecond()); 919 this._showHideSidebarButton.classList.toggle("right-sidebar-show-hide-bu tton", this.isVertical() && this.isSidebarSecond());
919 this._showHideSidebarButton.classList.toggle("bottom-sidebar-show-hide-b utton", !this.isVertical() && this.isSidebarSecond()); 920 this._showHideSidebarButton.classList.toggle("bottom-sidebar-show-hide-b utton", !this.isVertical() && this.isSidebarSecond());
920 this._showHideSidebarButton.classList.toggle("left-sidebar-show-hide-but ton", this.isVertical() && !this.isSidebarSecond()); 921 this._showHideSidebarButton.classList.toggle("left-sidebar-show-hide-but ton", this.isVertical() && !this.isSidebarSecond());
921 this._showHideSidebarButton.title = sidebarHidden ? WebInspector.UIStrin g("Show %s", this._showHideSidebarButtonTitle) : WebInspector.UIString("Hide %s" , this._showHideSidebarButtonTitle); 922 this._showHideSidebarButton.title = sidebarHidden ? WebInspector.UIStrin g("Show %s", this._showHideSidebarButtonTitle) : WebInspector.UIString("Hide %s" , this._showHideSidebarButtonTitle);
922 }, 923 },
923 924
924 __proto__: WebInspector.Widget.prototype 925 __proto__: WebInspector.Widget.prototype
925 } 926 }
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698