Opened 13 years ago
Closed 12 years ago
#26520 closed defect (bug) (fixed)
WordPress News Dashboard Widget Videos shows empty box on hover
| Reported by: | NikV | Owned by: | nacin |
|---|---|---|---|
| Priority: | normal | Milestone: | 3.9 |
| Component: | Widgets | Version: | 2.7 |
| Severity: | normal | Keywords: | has-patch 3.9-early |
| Cc: | Focuses: |
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
@
13 years ago
- Summary WordPress News Dashboard Widget shows empty box on hover → WordPress News Dashboard Widget Videos shows empty box on hover
#2
@
13 years ago
- Component General → Widgets
- Keywords has-patch 3.9-early added
- Milestone Awaiting Review → Future Release
- Version trunk → 2.7
#6
@
13 years ago
- Milestone Future Release → 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
@
12 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.
![(please configure the [header_logo] section in trac.ini)](/chrome/site/your_project_logo.png)

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.