WordPress.org
Get WordPress

Make WordPress Core

Opened 20 years ago

Closed 19 years ago

#3834 closed defect (bug) (fixed)

wp_rss() puts item description in title attribute

Reported by: foofy Owned by: rob1n
Priority: low Milestone: 2.3
Component: General Version: 2.1.1
Severity: normal Keywords: has-patch commit
Cc: Focuses:

Description

wp_rss() puts the full item description (including HTML tags) in the title. This causes validation errors, and adds a lot of bloat to the HTML that a lot of users won't see.

Attachments (2)

wp_rss_description_removal.diff​ (1.0 KB ) - added by markjaquith 20 years ago.
Removes description from the title
3834.diff​ (1.4 KB ) - added by rob1n 19 years ago.

Download all attachments as: .zip

Change History (14)

#1 @foofy
20 years ago

  • Component Administration → General

#2 @foolswisdom
20 years ago

  • Milestone 2.3 → 2.2

@markjaquith
20 years ago

Removes description from the title

#3 @markjaquith
20 years ago

  • Keywords has-patch added

Patch removes it for wp_rss() and get_rss()

#4 @Otto42
20 years ago

The title tag appears when you hover over a link in most user agents. Do we really want to eliminate this?

#5 @markjaquith
20 years ago

We don't want it to be the full description. And title is redundant.

#6 @foolswisdom
20 years ago

  • Keywords has-patch removed
  • Milestone 2.2 → 2.4

#7 @foolswisdom
20 years ago

  • Keywords has-patch added
  • Milestone 2.4 → 2.3

#8 @foolswisdom
20 years ago

  • Owner changed from anonymous to markjaquith

#9 @rob1n
19 years ago

  • Owner changed from markjaquith to rob1n
  • Status new → assigned

#10 @rob1n
19 years ago

We could do attribute_escape( strip_tags( $desc ) ).

@rob1n
19 years ago

#11 @rob1n
19 years ago

  • Keywords commit added

#12 @rob1n
19 years ago

  • Resolution → fixed
  • Status assigned → closed

(In [5698]) wp_rss() updates. fixes #3834

Note: See TracTickets for help on using tickets.