Opened 3 years ago

Closed 3 years ago

#13719 closed enhancement (fixed)

Remove background color for default RSS widget icon

Reported by: lancewillett Owned by: azaozz
Priority: low Milestone: 3.0
Component: Widgets Version: 3.0
Severity: minor Keywords: has-patch
Cc:

Description

See wp-includes/default-widgets.php line 740.

The RSS icon in the default RSS widget currently uses a style of background: orange; which shows up poorly on dark backgrounds.

Patch changes the rule to background: transparent; along with removing unneeded color rule.

Attachments (2)

rsswidget-13719.diff​ (870 bytes) - added by lancewillett 3 years ago.
black_rss.png​ (4.4 KB) - added by ocean90 3 years ago.

Download all attachments as: .zip

Change History (7)

lancewillett — 3 years ago

comment:1   nacin — 3 years ago

Screenshot I imagine would be quite useful here.

ocean90 — 3 years ago

comment:2   ocean90 — 3 years ago

  • Milestone changed from Unassigned to 3.0

Added a screenshot, should be harmless.

comment:3   nacin — 3 years ago

Does it even need background:transparent at that point? That is the default.

comment:4   nacin — 3 years ago

Also, it can become esc_attr, but not esc_attr_e, we can't echo there.

comment:5   nacin — 3 years ago

  • Resolution set to fixed
  • Status changed from new to closed

(In [15192]) Don't specify a background for the default RSS widget's icon. props lancewillet, fixes #13719.

Note: See TracTickets for help on using tickets.