WordPress.org
Get WordPress

Make WordPress Core

Opened 11 months ago

Closed 11 months ago

#61431 closed defect (bug) (invalid)

Remove Extra spaces

Reported by: dhruval04's profile dhruval04 Owned by:
Milestone: Priority: normal
Severity: normal Version:
Component: General Keywords: has-patch
Focuses: docs Cc:

Description

Removed extra space in the inline doc comment wp-includes/class-wp-theme-json-schema.php

Attachments (1)

61431.patch​ (937 bytes) - added by dhruval04 11 months ago.

Download all attachments as: .zip

Change History (4)

@dhruval04
11 months ago

#1 @khokansardar
11 months ago

  • Keywords close added

@dhruval04 your patch is invalid as those line of changes is not needed. Extra spaces are there for the alignment with other @param.

#2 @khokansardar
11 months ago

  • Keywords dev-feedback added; close removed

#3 @sabernhardt
11 months ago

  • Keywords dev-feedback removed
  • Milestone Awaiting Review deleted
  • Resolution set to invalid
  • Status changed from new to closed

Thanks! Those docs are good as they are.

The ​handbook page mentions aligning multiple globals according to "type, variable, and description, with each other as a group." I did not find it saying the same about parameters, but that is how they should align.

Examples:

  • Changeset 57687 added spaces before the description for $mime so it would line up with the description for $preferred_ext.
  • Changesets such as [58220] and [58224] added spaces to align the variable names when one of the types became longer.
Note: See TracTickets for help on using tickets.