Make WordPress Core

Opened 3 months ago

Closed 3 months 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.


3 months 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
3 months ago

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

@khokansardar commented on PR #6920:


3 months ago
#3

@sirreal changes LGTM.

@jonsurrell commented on PR #6920:


3 months ago
#4

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

@jonsurrell commented on PR #6920:


3 months 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
3 months 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.