Project

General

Profile

Actions
Copy link

Defect #43508

closed

legacy-icons-compat.css misses PNG icon styles removed from responsive.css, rtl.css, and scm.css

Added by Katsuya HIDAKA 1 day ago. Updated 1 day ago.

Status:
Closed
Priority:
Normal
Assignee:
Go MAEDA
Category:
UI
Target version:
-
Start date:
Due date:
% Done:

0%

Estimated time:
Resolution:
Fixed
Affected version:

Description

Legacy PNG icon styles are now provided in legacy-icons-compat.css since r24103, but the styles removed from responsive.css, rtl.css, and scm.css are not included.

As a result, for example, when responsive layout rules apply, the admin menu icons may appear broken. It seems not to match the expected behavior when loading legacy-icons-compat.css.

The attached patch (fix.patch) adds the missing styles from responsive.css, rtl.css, and scm.css to legacy-icons-compat.css.

Files

Download all files
issue-menu-icon.png (19 KB) issue-menu-icon.png Katsuya HIDAKA, 2025-11-21 03:51
issue-menu-icon-after-fix.png (19 KB) issue-menu-icon-after-fix.png Katsuya HIDAKA, 2025-11-21 03:53
issue-contextual-icon.png (14.1 KB) issue-contextual-icon.png Katsuya HIDAKA, 2025-11-21 03:54
issue-contextual-icon-after-fix.png (14.4 KB) issue-contextual-icon-after-fix.png Katsuya HIDAKA, 2025-11-21 03:54
fix.patch (3.21 KB) fix.patch Katsuya HIDAKA, 2025-11-21 03:55

Related issues

Related to Redmine - Patch #43206: Remove deprecated icon-* classes from stylesheetsClosedMarius BĂLTEANU

Actions
Actions
Copy link
 #1

Updated by Go MAEDA 1 day ago

  • Related to Patch #43206: Remove deprecated icon-* classes from stylesheets added
Actions
Copy link
 #2

Updated by Go MAEDA 1 day ago

  • Assignee set to Go MAEDA
Actions
Copy link
 #3

Updated by Go MAEDA 1 day ago

  • Resolution set to Fixed

Committed the fix in r24153, as part of #43206. Thank you.

Actions
Copy link
 #4

Updated by Go MAEDA 1 day ago

  • Status changed from New to Closed
Actions
Copy link

Also available in: Atom PDF