#44520 closed defect (bug) (fixed)
Fix default parameters in controls.js JSDoc.
| Reported by: | herregroen | Owned by: | atimmer |
|---|---|---|---|
| Priority: | normal | Milestone: | 5.1 |
| Component: | Customize | Version: | 5.1 |
| Severity: | normal | Keywords: | has-patch |
| Cc: | Focuses: | javascript, docs |
Description
While JSDoc technically supports the @param {string=} optional or @param {string=} [optional] style of setting optional parameters.
It does unfortunately not support @param {string=default} optional or @param {string=default} [optional] style of default parameters.
Instead, as the JSDoc standards recommend, the following should be used:
@param {string} [optional] or @param {string} [optional=default].
src/js/_enqueues/wp/customize/controls.js still contains several notations of the incorrect style.
Attachments (1)
Change History (4)
Note:
See TracTickets
for help on using tickets.
![(please configure the [header_logo] section in trac.ini)](/chrome/site/your_project_logo.png)
In 43391: