From 8e9dc84ea949bfb7825401a69029c433fdf2faa3 Mon Sep 17 00:00:00 2001 From: Alexander Schmitz Date: Wed, 28 Sep 2016 21:42:50 -0400 Subject: [PATCH 1/2] Theme: Switch icon background to use bgColorContent It's more semanticly correct then fcActive and looks the same or better on most themes Fixes jquery/download.jqueryui.com#335 --- themes/base/theme.css | 5 ++--- 1 file changed, 2 insertions(+), 3 deletions(-) diff --git a/themes/base/theme.css b/themes/base/theme.css index bfdf9b7c0f4..c802a090003 100644 --- a/themes/base/theme.css +++ b/themes/base/theme.css @@ -119,7 +119,7 @@ a.ui-button:active, .ui-icon-background, .ui-state-active .ui-icon-background { border: #003eff/*{borderColorActive}*/; - background-color: #ffffff/*{fcActive}*/; + background-color: #ffffff/*{bgColorContent}*/; } .ui-state-active a, .ui-state-active a:link, @@ -204,8 +204,7 @@ a.ui-button:active, .ui-state-hover .ui-icon, .ui-state-focus .ui-icon, .ui-button:hover .ui-icon, -.ui-button:focus .ui-icon, -.ui-state-default .ui-icon { +.ui-button:focus .ui-icon { background-image: url("images/ui-icons_555555_256x240.png")/*{iconsHover}*/; } .ui-state-active .ui-icon, From eeae88afdb483bfb1454a8809822af0013617832 Mon Sep 17 00:00:00 2001 From: Alexander Schmitz Date: Wed, 28 Sep 2016 21:56:53 -0400 Subject: [PATCH 2/2] Theme: Replace missing definition for default icons Fixes jquery/download.jqueryui.com#335 --- themes/base/theme.css | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/themes/base/theme.css b/themes/base/theme.css index c802a090003..a1c3c3cd66d 100644 --- a/themes/base/theme.css +++ b/themes/base/theme.css @@ -219,7 +219,8 @@ a.ui-button:active, .ui-state-error-text .ui-icon { background-image: url("images/ui-icons_cc0000_256x240.png")/*{iconsError}*/; } -.ui-button .ui-icon { +.ui-button .ui-icon, +.ui-state-default .ui-icon { background-image: url("images/ui-icons_777777_256x240.png")/*{iconsDefault}*/; }