bug symfony#17330 Limit the max height/width of icons in the profiler menu (javiereguiluz)

fabpot · fabpot · commit ebacf55919ea · 2016-01-25T12:41:28.000+01:00
This PR was squashed before being merged into the 2.8 branch (closes symfony#17330). Discussion ---------- Limit the max height/width of icons in the profiler menu | Q | A | ------------- | --- | Bug fix? | yes | New feature? | no | BC breaks? | no | Deprecations? | no | Tests pass? | yes | Fixed tickets | symfony#17329 | License | MIT | Doc PR | - Commits ------- 1f5f81c Limit the max height/width of icons in the profiler menu
diff --git a/Resources/views/Profiler/profiler.css.twig b/Resources/views/Profiler/profiler.css.twig
@@ -619,6 +619,11 @@ tr.status-warning td {
     width: 50px;
     text-align: center;
 }
+#menu-profiler .label .icon img,
+#menu-profiler .label .icon svg {
+    height: 24px;
+    max-width: 24px;
+}
 #menu-profiler li a .label .icon svg path {
     fill: #DDD;
 }
diff --git a/Resources/views/Profiler/toolbar.css.twig b/Resources/views/Profiler/toolbar.css.twig
@@ -53,8 +53,7 @@
 }
 .sf-toolbarreset svg,
 .sf-toolbarreset img {
-    max-height: 20px;
-    max-width: 20px;
+    height: 20px;
 }
 
 .sf-toolbarreset .hide-button {

Original file line number	Diff line number	Diff line change
`@@ -619,6 +619,11 @@ tr.status-warning td {`
`619`	`619`	`width: 50px;`
`620`	`620`	`text-align: center;`
`621`	`621`	`}`
	`622`	`+#menu-profiler .label .icon img,`
	`623`	`+#menu-profiler .label .icon svg {`
	`624`	`+ height: 24px;`
	`625`	`+ max-width: 24px;`
	`626`	`+}`
`622`	`627`	`#menu-profiler li a .label .icon svg path {`
`623`	`628`	`fill: #DDD;`
`624`	`629`	`}`
Original file line number	Diff line number	Diff line change
`@@ -53,8 +53,7 @@`
`53`	`53`	`}`
`54`	`54`	`.sf-toolbarreset svg,`
`55`	`55`	`.sf-toolbarreset img {`
`56`		`- max-height: 20px;`
`57`		`- max-width: 20px;`
	`56`	`+ height: 20px;`
`58`	`57`	`}`
`59`	`58`
`60`	`59`	`.sf-toolbarreset .hide-button {`