#43499 closed defect (bug) (fixed)
Document src/wp-admin/js/tags-box.js
Reported by: | carolinegeven | Owned by: | atimmer |
---|---|---|---|
Milestone: | 5.1 | Priority: | normal |
Severity: | normal | Version: | 5.1 |
Component: | Taxonomy | Keywords: | has-patch |
Focuses: | javascript, docs | Cc: |
Description
Adds documentation to src/wp-admin/js/tags-box.js
To see the files changed: https://github.com/Yoast/wordpress-develop-mirror/pull/40
Please give props to: @atimmer, @carolinegeven, @andizer, @ireneyoast, @herregroen, @jjcomack
Attachments (3)
Change History (11)
#1
@
7 years ago
- Component changed from General to Taxonomy
- Milestone changed from Awaiting Review to 5.0
#2
@
7 years ago
- Keywords needs-refresh added
Hi @carolinegeven, thanks for the patch!
It appears to be created from the stable branch and contains 3.7 MB of unrelated changes :) Could you please refresh it from WordPress trunk?
#5
@
7 years ago
In document-tags-box-js.diff I noticed a typo:
This line:
* @param {Event} e The window event to handle.
for:
$( 'input.newtag', ajaxtag ).keypress( function( event ) {
should be:
* @param {Event} event The window event to handle.
#6
@
7 years ago
@kapteinbluf and I changed e
to event
and we've also changed all occurrences of @returns
to @return
Note: See
TracTickets for help on using
tickets.
Document tags-box.js