Make WordPress Core

Opened 2 years ago

Closed 2 years ago

#61520 closed defect (bug) (maybelater)

HTML API: class_list yields lower case class names needs documentation

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

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.


2 years 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
2 years ago

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

@khokansardar commented on PR #6920:


2 years ago
#3

@sirreal changes LGTM.

@jonsurrell commented on PR #6920:


2 years ago
#4

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

@jonsurrell commented on PR #6920:


2 years 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
2 years ago

  • Milestone 6.6
  • Resolutionmaybelater
  • Status assignedclosed

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.