Source/devtools/front_end/main/module.json - Issue 1176343002: DevTools: make toolbar button click targets square

Keyboard Shortcuts

	File
u :	up to issue
j / k :	jump to file after / before current file
J / K :	jump to next file with a comment after / before current file
	Side-by-side diff
i :	toggle intra-line diffs
e :	expand all comments
c :	collapse all comments
s :	toggle showing all comments
n / p :	next / previous diff chunk or comment
N / P :	next / previous comment
<Up> / <Down> :	next / previous line

	Issue
u :	up to list of issues
j / k :	jump to patch after / before current patch
o / <Enter> :	open current patch in side-by-side view
i :	open current patch in unified diff view

	Issue List
j / k :	jump to issue after / before current issue
o / <Enter> :	open current issue

Unified Diff: Source/devtools/front_end/main/module.json

Issue 1176343002: DevTools: make toolbar button click targets square (Closed) Base URL: svn://svn.chromium.org/blink/trunk

Patch Set: Created 5 years, 6 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

« Source/devtools/front_end/emulation/responsiveDesignView.css ('K') | « Source/devtools/front_end/inspectorStyle.css ('k') | Source/devtools/front_end/network/NetworkPanel.js » ('j') | Source/devtools/front_end/sidebarPane.css » ('J')
Expand Comments ('e') | Collapse Comments ('c') | Hide Comments ('s')

Index: Source/devtools/front_end/main/module.json

diff --git a/Source/devtools/front_end/main/module.json b/Source/devtools/front_end/main/module.json

index b78b575ba4830f950fd964012529df34ce394f3c..3a9818e3ca7c77f650b27f4e2490fff39c417830 100644

--- a/Source/devtools/front_end/main/module.json

+++ b/Source/devtools/front_end/main/module.json

@@ -158,20 +158,39 @@

{

"type": "@WebInspector.ToolbarItem.Provider",

- "className": "WebInspector.Main.WarningErrorCounter",

+ "separator": true,

+ "location": "main-toolbar-left",

+ "order": 100,

dgozman 2015/06/11 16:14:12 100 is for close button. 99?

pfeldman 2015/06/11 16:38:06 These are different toolbars.

+ "actionId": "main.toggle-element-search"

dgozman 2015/06/11 16:14:12 Remove actionId.

pfeldman 2015/06/11 16:38:06 Done.

+ },

+ {

+ "type": "@WebInspector.ToolbarItem.Provider",

+ "className": "WebInspector.InspectorView.ToggleDrawerButtonProvider",

"order": 1,

"location": "main-toolbar-right"

{

"type": "@WebInspector.ToolbarItem.Provider",

- "className": "WebInspector.InspectorView.ToggleDrawerButtonProvider",

+ "className": "WebInspector.Main.WarningErrorCounter",

dgozman 2015/06/11 16:14:11 Why swapped with toggle drawer?

pfeldman 2015/06/11 16:38:06 Done.

"order": 2,

"location": "main-toolbar-right"

{

"type": "@WebInspector.ToolbarItem.Provider",

+ "separator": true,

+ "order": 3,

+ "location": "main-toolbar-right"

+ },

+ {

+ "type": "@WebInspector.ToolbarItem.Provider",

"className": "WebInspector.DockController.ButtonProvider",

- "order": 4,

+ "order": 5,

+ "location": "main-toolbar-right"

+ },

+ {

+ "type": "@WebInspector.ToolbarItem.Provider",

+ "separator": true,

+ "order": 90,

"location": "main-toolbar-right"

{