Opened 5 years ago
Closed 5 years ago
#51063 closed defect (bug) (wontfix)
Comment description update on class.wp-scripts file
Reported by: |
|
Owned by: | |
---|---|---|---|
Milestone: | Priority: | normal | |
Severity: | normal | Version: | 5.5 |
Component: | Script Loader | Keywords: | has-patch |
Focuses: | docs | Cc: |
Description
Some comments description updates.
Attachments (1)
Change History (5)
#2
@
5 years ago
- Component changed from General to Script Loader
- Keywords close added; good-first-bug removed
Hi there, welcome back to WordPress Trac! Thanks for the patch.
The existing descriptions look fine as is to me:
- "Name of the script to add the inline script to" is a description of the parameter. "Must be lowercase" is a clarification on which values are accepted. I don't think these should be combined in a single sentence.
- Per the documentation standards, DocBlocks should be wrapped to the next line after around 80 characters for better readability. There are some functions with complex DocBlocks in core where this is not always feasible, in which case they can take up to 120 characters. But the ones in this file seem fine as is.
For reference, these DocBlocks were previously adjusted per the documentation standards in [47170].
Note: See
TracTickets for help on using
tickets.
Created patch.