Make WordPress Core

Opened 9 years ago

Closed 9 years ago

Last modified 8 years ago

#43499 closed defect (bug) (fixed)

Document src/wp-admin/js/tags-box.js

Reported by: carolinegeven Owned by: atimmer
Priority: normal Milestone: 5.1
Component: Taxonomy Version: 5.1
Severity: normal Keywords: has-patch
Cc: Focuses: javascript, docs

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)

document-tags-box-js.patch​ (3.7 MB ) - added by carolinegeven 9 years ago.
Document tags-box.js
document-tags-box-js.diff​ (9.0 KB ) - added by carolinegeven 9 years ago.
Document tags-box.js
documents-tags-box.patch​ (9.0 KB ) - added by DiedeExterkate 9 years ago.
Document tags-box.js

Change History (11)

@carolinegeven
9 years ago

Document tags-box.js

#1 @SergeyBiryukov
9 years ago

  • Component General → Taxonomy
  • Milestone Awaiting Review → 5.0

#2 @SergeyBiryukov
9 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?

@carolinegeven
9 years ago

Document tags-box.js

#3 @carolinegeven
9 years ago

@SergeyBiryukov Oops, sorry for that! Reuploaded the patch.

#4 @SergeyBiryukov
9 years ago

  • Keywords has-patch added; needs-refresh removed

#5 @birgire
9 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.

@DiedeExterkate
9 years ago

Document tags-box.js

#6 @DiedeExterkate
9 years ago

@kapteinbluf and I changed e to event and we've also changed all occurrences of @returns to @return

#7 @atimmer
9 years ago

  • Owner set to atimmer
  • Resolution → fixed
  • Status new → closed

In 42963:

Docs: Improve JSDoc for wp-admin/js/tags-box.js.

Props carolinegeven, andizer, ireneyoast, herregroen, jjcomack.
Fixes #43499.

#8 @johnbillion
8 years ago

  • Milestone 5.0 → 5.1
Note: See TracTickets for help on using tickets.