From 0fe05e52231502a31bbb1048e8ac7c8276b48a57 Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?Micha=C5=82=20Go=C5=82=C4=99biowski-Owczarek?= Date: Sun, 26 Sep 2021 00:03:44 +0200 Subject: [PATCH] Theme: Revert three theme commits that are causing styling issues (#1983) * Revert "Theme: Removes css for ui-state-checked its not used any more" This reverts commit 1b0e947f46bc1261b15816f2dcbd239d83a86335. * Revert "Theme: Replace missing definition for default icons" This reverts commit dde9b83df61d1d676e66cb2a2f7970dd44a05137. * Revert "Theme: Switch icon background to use bgColorContent" This reverts commit 265b8f5e533923b9b4c9cbd9f1dd7b7785423381. This reverts three commits: * Theme: Removes css for ui-state-checked its not used any more https://github.com/jquery/jquery-ui/commit/1b0e947f46bc1261b15816f2dcbd239d83a86335 (https://github.com/jquery/jquery-ui/pull/1753) * Theme: Replace missing definition for default icons https://github.com/jquery/jquery-ui/commit/dde9b83df61d1d676e66cb2a2f7970dd44a05137 * Theme: Switch icon background to use bgColorContent https://github.com/jquery/jquery-ui/commit/265b8f5e533923b9b4c9cbd9f1dd7b7785423381 which caused styling issues when compared to UI 1.12.1. This unfixes a few issues: * https://github.com/jquery/download.jqueryui.com/issues/335 * https://bugs.jqueryui.com/ticket/15059 * https://forum.jquery.com/topic/checkboxradio-widget-checkbox-click-doesn-t-work-with-ui-lightness-theme-22-9-2016 However, old & known issues are better than new & unknown ones, especially with our current very limited resources. Closes gh-1983 --- themes/base/theme.css | 14 +++++++++----- 1 file changed, 9 insertions(+), 5 deletions(-) diff --git a/themes/base/theme.css b/themes/base/theme.css index 192624a11ff..a37d2c97a85 100644 --- a/themes/base/theme.css +++ b/themes/base/theme.css @@ -116,9 +116,10 @@ a.ui-button:active, font-weight: normal/*{fwDefault}*/; color: #ffffff/*{fcActive}*/; } -.ui-icon-background { - border: #dddddd/*{borderColorContent}*/; - background-color: #ffffff/*{bgColorContent}*/; +.ui-icon-background, +.ui-state-active .ui-icon-background { + border: #003eff/*{borderColorActive}*/; + background-color: #ffffff/*{fcActive}*/; } .ui-state-active a, .ui-state-active a:link, @@ -136,6 +137,10 @@ a.ui-button:active, background: #fffa90/*{bgColorHighlight}*/ /*{bgImgUrlHighlight}*/ /*{bgHighlightXPos}*/ /*{bgHighlightYPos}*/ /*{bgHighlightRepeat}*/; color: #777620/*{fcHighlight}*/; } +.ui-state-checked { + border: 1px solid #dad55e/*{borderColorHighlight}*/; + background: #fffa90/*{bgColorHighlight}*/; +} .ui-state-highlight a, .ui-widget-content .ui-state-highlight a, .ui-widget-header .ui-state-highlight a { @@ -214,8 +219,7 @@ 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-state-default .ui-icon { +.ui-button .ui-icon { background-image: url("images/ui-icons_777777_256x240.png")/*{iconsDefault}*/; }