Make WordPress Core

Opened 21 years ago

Closed 19 years ago

#1585 closed defect (bug) (fixed)

Added `summary' field to table generated by get_calendar() to comply with WAI checkpoint 5.5

Reported by: littleram's profile littleram Owned by: rob1n's profile rob1n
Milestone: 2.2 Priority: normal
Severity: trivial Version: 2.1
Component: General Keywords: has-patch commit
Focuses: Cc:

Description

The W3C Web Accessibility Guidelines require <table> tags to have a summary field summarizing the table contents.

More info can be found here: http://www.w3.org/TR/1999/WAI-WEBCONTENT-19990505/#tech-table-summaries

Attachments (4)

get_calendar.txt (7.7 KB) - added by littleram 21 years ago.
contains fixed version of get_calendar();
template-functions-general.16.php.diff (884 bytes) - added by markjaquith 21 years ago.
Patch for 1.6 SVN
1585.diff (492 bytes) - added by rob1n 19 years ago.
1585-round2.diff (525 bytes) - added by rob1n 19 years ago.
Adds back in the ".

Download all attachments as: .zip

Change History (17)

@littleram
21 years ago

contains fixed version of get_calendar();

#1 @littleram
21 years ago

  • Priority changed from low to normal

@markjaquith
21 years ago

Patch for 1.6 SVN

#2 @markjaquith
21 years ago

  • Keywords bg|has-patch bg|dev-feedback added
  • Owner changed from anonymous to markjaquith
  • Status changed from new to assigned

#3 @markjaquith
21 years ago

  • Version changed from 1.5 to 1.5.2

#4 @matt
21 years ago

  • Milestone changed from 1.6 to 2.1

#5 @matt
19 years ago

  • Milestone changed from 2.1 to 2.2

@rob1n
19 years ago

#6 @rob1n
19 years ago

  • Keywords has-patch dev-feedback added; bg|has-patch bg|dev-feedback removed
  • Owner changed from markjaquith to rob1n
  • Status changed from assigned to new
  • Version changed from 1.5.2 to 2.1

Added patch for current SVN (r4904).

#7 @rob1n
19 years ago

  • Status changed from new to assigned

#8 @rob1n
19 years ago

  • Keywords commit added; dev-feedback removed

#9 @ryan
19 years ago

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

(In [4921]) Provide summary for calendar. Props littleram and rob1n. fixes #1585

#10 @Nazgul
19 years ago

  • Resolution fixed deleted
  • Status changed from closed to reopened

The patch introduced a bug. The closing " for the summary attribute is missing.

#11 @rob1n
19 years ago

OMG as they say. Patch coming right up.

Good eye, Nazgul.

@rob1n
19 years ago

Adds back in the ".

#12 @rob1n
19 years ago

Okay, new patch applies ON TOP of existing changes. Just to make sure :).

#13 @ryan
19 years ago

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

(In [4924]) Fix typo. Props rob1n. fixes #1585

Note: See TracTickets for help on using tickets.