Opened 3 months ago
Closed 3 months ago
#23571 closed defect (bug) (fixed)
Twenty Thirteen: improve styling for RSS widget title
| Reported by: |
|
Owned by: |
|
|---|---|---|---|
| Priority: | normal | Milestone: | 3.6 |
| Component: | Bundled Theme | Version: | |
| Severity: | normal | Keywords: | has-patch |
| Cc: |
Description
The RSS widget links it's title, so it doesn't look like it fits with the other widgets.
Attachments (3)
Change History (12)
aaroncampbell — 3 months ago
comment:1
aaroncampbell — 3 months ago

comment:2
lancewillett — 3 months ago
- Keywords needs-patch added
- Summary changed from Twenty Thirteen: RSS Widget Title to Twenty Thirteen: improve styling for RSS widget title
- Keywords has-patch added; needs-patch removed
23571.diff styles linked widget titles to match the other widget titles. Links are still underlined on hover, so they're still discoverable as links.
comment:4
lancewillett — 3 months ago
@sabreuse Tiny thing with your patch -- can you put new selectors on a new line? We're following the WP CSS guidelines, see "Structure" here: http://make.wordpress.org/core/handbook/coding-standards/css/#structure.
Otherwise, the technique should work well and I can push the fix in.
23571.2.diff Updates the patch to follow WP CSS guidelines, also uses better CSS selectors for widget titles as suggested in #23580.
Do we really want to have them in the same color? It's a link after all, the others aren't.
comment:7
in reply to:
↑ 6
aaroncampbell — 3 months ago
Replying to obenland:
Do we really want to have them in the same color? It's a link after all, the others aren't.
I prefer it to be the same color. I think it looks best. However, I'm open to another option but I think the RSS Widget header should be further differentiated from the links in the widget content.
Any suggestions?
comment:8
lancewillett — 3 months ago
I think it's fine if it's the same color. On hover it changes to denote a link.
comment:9
lancewillett — 3 months ago
- Owner set to lancewillett
- Resolution set to fixed
- Status changed from new to closed
In 23491:
