Opened 11 years ago
Closed 11 years ago
#26520 closed defect (bug) (fixed)
WordPress News Dashboard Widget Videos shows empty box on hover
Reported by: | NikV | Owned by: | nacin |
---|---|---|---|
Milestone: | 3.9 | Priority: | normal |
Severity: | normal | Version: | 2.7 |
Component: | Widgets | Keywords: | has-patch 3.9-early |
Focuses: | Cc: |
Description
For the WordPress News Widget, the regular articles come with a preview text box on hover, but for videos, that box shows up empty. This may be really trivial, but just something to look at for a future release.
Image: https://i.cloudup.com/w4hsYa5OuD.jpg
I am on Chrome running WordPress 3.8 RC2.
Attachments (2)
Change History (10)
#1
@
11 years ago
- Summary changed from WordPress News Dashboard Widget shows empty box on hover to WordPress News Dashboard Widget Videos shows empty box on hover
#2
@
11 years ago
- Component changed from General to Widgets
- Keywords has-patch 3.9-early added
- Milestone changed from Awaiting Review to Future Release
- Version changed from trunk to 2.7
#6
@
11 years ago
- Milestone changed from Future Release to 3.9
26520.2.patch removes the title attribute from wp_widget_rss_output()
and makes $summary
assignment more consistent with $date
and $author
.
#7
@
11 years ago
Keep in mind that wp_widget_rss_output() is used on the frontend. We *could* outright remove it, but rather than making such a decision late in a release, let's start by using the wp_html_excerpt()'d version as the title attribute, too, and omitting the title attribute if the summary is being shown.
Introduced in [9219].
Video links have a space as the title attribute:
title=" "
, and Chrome displays it.Adding
trim()
towp_widget_rss_output()
fixes that.