CSS class repeated needlessly

[johnp]

I downloaded a build of 1.8.15 with the UI lightness theme. The resulting jquery-ui-1.8.15.custom.css contains many instances where a class appears multiple times in a given rule. One example appears under "Interaction Cues". .ui-state-error is listed 3 times for this single rule.

.ui-state-error, .ui-widget-content .ui-state-error, .ui-widget-header .ui-state-error {border: 1px solid #cd0a0a; background: #b81900 url(images/ui-bg_diagonals-thick_18_b81900_40x40.png) 50% 50% repeat; color: #ffffff; }

I know that it's valid CSS, but it adds unnecessary weight to the CSS file.

[M.Gaetan89]

The change has been submitted on GitHub : ​https://github.com/jquery/jquery-ui/pull/518

[rdworth]

The proposed change will cut the specificity in half, down from two class names to one class name. This means a simple rule like

.main div { border: none; }

later in a stylesheet will override this style where before this change it would not. If the classes were listed needlessly I could understand removing them, but they're listed in this manner by design and removing them will have a negative effect.

Recommend closing "as designed"/invalid