-
Notifications
You must be signed in to change notification settings - Fork 3.7k
Improve Customizer JSDoc #10743
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Open
westonruter
wants to merge
48
commits into
WordPress:trunk
Choose a base branch
from
westonruter:trac-40831
base: trunk
Could not load branches
Branch not found: {{ refName }}
Loading
Could not load tags
Nothing to show
Loading
Are you sure you want to change the base?
Some commits from the old base branch may be removed from the timeline,
and old review comments may become outdated.
Open
Improve Customizer JSDoc #10743
Changes from all commits
Commits
Show all changes
48 commits
Select commit
Hold shift + click to select a range
a603e94
https://core.trac.wordpress.org/attachment/ticket/40831/40831.diff
westonruter a45b863
Fix additional jsdoc
westonruter 9817e3f
Further improve jsdoc
westonruter a2396e2
Add blank line in jsdoc
westonruter 9a5e1a7
Merge branch 'trunk' into trac-40831
westonruter c633c29
Improve Customizer JSDoc and replace `arguments` with rest parameters.
westonruter 5221d35
Consume the `wp` argument passed into the Customizer IIFEs.
westonruter 1835ade
Replace the last use of `arguments` in `wp.customize.Class`.
westonruter 162a1f1
Correct the direction described by `wp.customize.Value#link()`.
westonruter cef3122
Cover the variadic parts of the Customizer base classes.
westonruter 55be81e
Replace the `mixed` JSDoc type with `*` in the Customizer.
westonruter 6ddae5a
Use resolvable names for the Customizer JSDoc types.
westonruter c7f407f
Correct the types documented for the Customizer IIFE arguments.
westonruter d512ff3
Fix further JSDoc defects in the Customizer.
westonruter a5c5d5f
Drop the hyphen before Customizer JSDoc parameter descriptions.
westonruter 3d17619
Document the arguments of three more Customizer functions.
westonruter d944c5c
Check both directions when unsyncing two values.
westonruter 5ca507b
Describe optional parameters and member types accurately in the Custo…
westonruter ab9eb85
Mark the remaining optional Customizer parameters as optional.
westonruter 31e8243
Widen the type of the value passed to `wp.customize.Value#set()`.
westonruter 1fa8f87
Mark the callback optional for `wp.customize.control()` and `panel()`.
westonruter 11ffe58
Spell the Customizer's generic object types consistently.
westonruter 182301e
Correct the return documented for `wp.customize.Loader.open()`.
westonruter 9b65262
Describe what `wp.customize.Class` returns, and two Placement details.
westonruter a19d5df
Correct three Customizer types that misdescribe their values.
westonruter 32e8753
Correct three more Customizer parameters in customize-base.js.
westonruter bc02b24
Document the arguments of the remaining Customizer functions.
westonruter de984b7
Name the WordPress global `wp` in customize-base.js.
westonruter 8645a76
Put the `@output` docblock first in customize-widgets.js.
westonruter ffa1007
Name the Underscore and jQuery promise types in the Customizer.
westonruter 1485310
Use the jQuery namespace as it is actually declared.
westonruter 4f8350c
Say what the Customizer's arrays contain.
westonruter 7106b2a
Merge branch 'trunk' into trac-40831
westonruter 63bb030
Tidy the Customizer docblocks after code review.
westonruter 54c5826
Require the nav menu arguments the partial constructor insists on.
westonruter 5ceb336
Align the docblock lines that continue a description.
westonruter a95cec6
Correct two parameter descriptions that describe the wrong thing.
westonruter 49ddca4
Finish the parameter descriptions that were left incomplete.
westonruter d1c9b66
Merge branch 'trunk' of https://github.com/WordPress/wordpress-develo…
westonruter cda7c0e
Name the jQuery collection type so it resolves to a collection.
westonruter 384799f
Type the partial refresh promises as the jQuery promises they are.
westonruter e49b63e
Say what `this` actually is in the field toggle handler.
westonruter 51b1a2d
Fix three structural tags that name or classify the wrong thing.
westonruter 8279851
Name the selective refresh classes instead of inferring them.
westonruter bb77c4b
Finish the tag descriptions that trailed off mid-phrase.
westonruter d9f3663
Return the header image model, not a set of cropper options.
westonruter 9576a7a
Describe the parameters that were named but never explained.
westonruter 1287025
Correct three things the parameter tags got wrong.
westonruter File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Some comments aren't visible on the classic Files Changed page.
There are no files selected for viewing
Large diffs are not rendered by default.
Oops, something went wrong.
Large diffs are not rendered by default.
Oops, something went wrong.
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Oops, something went wrong.
Oops, something went wrong.
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Uh oh!
There was an error while loading. Please reload this page.