Make WordPress Core

Opened 4 weeks ago

Closed 4 weeks ago

#61520 closed defect (bug) (maybelater)

HTML API: class_list yields lower case class names needs documentation

Reported by: jonsurrell's profile jonsurrell Owned by: jonsurrell's profile jonsurrell
Milestone: Priority: normal
Severity: normal Version: 6.5
Component: HTML API Keywords: has-patch has-unit-tests
Focuses: docs Cc:

Description

That Tag Processor and HTML Processor yield lower case class names. This is by design, but it is undocumented.

Add documentation about this behavior.

Change History (6)

This ticket was mentioned in PR #6920 on WordPress/wordpress-develop by @jonsurrell.


4 weeks ago
#1

  • Keywords has-patch has-unit-tests added

The class_list method yields class name strings in lower case. This is
intended but undocumented. Document and add tests for this behavior.

Trac ticket: https://core.trac.wordpress.org/ticket/61520

#2 @jonsurrell
4 weeks ago

This is a documentation and test change that would be good to backport to 6.6 RC.

@khokansardar commented on PR #6920:


4 weeks ago
#3

@sirreal changes LGTM.

@jonsurrell commented on PR #6920:


4 weeks ago
#4

@dmsnell Will you review these documentation changes? These would be good to backport to 6.6.

@jonsurrell commented on PR #6920:


4 weeks ago
#5

This may not be the best approach. Rather than documenting the specifics of the behavior here, it seems best to audit the class-related methods.

#6 @jonsurrell
4 weeks ago

  • Milestone 6.6 deleted
  • Resolution set to maybelater
  • Status changed from assigned to closed

Let's audit the behavior of class related behaviors instead. This current behavior may not be ideal. Superseded by #61531.

Note: See TracTickets for help on using tickets.