WordPress.org
Get WordPress

Make WordPress Core

Opened 9 years ago

Closed 8 years ago

#43230 closed defect (bug) (fixed)

JSDoc: wp-includes/js/media/views/spinner.js

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

Description

Added documentation.

Attachments (2)

spinner.diff​ (1009 bytes ) - added by avillegasn 9 years ago.
JS documentation added
spinner.2.diff​ (1008 bytes ) - added by avillegasn 9 years ago.
Fixed JS documentation.

Download all attachments as: .zip

Change History (11)

@avillegasn
9 years ago

JS documentation added

#1 @avillegasn
9 years ago

  • Keywords has-patch added

@avillegasn
9 years ago

Fixed JS documentation.

#2 @atimmer
9 years ago

Thank you for your contribution!

As far as I could see the Spinner was introduced in this commit: ​https://github.com/WordPress/wordpress-develop/commit/15533d1d1fc6c249ac3b99be8b30234463b6ced3. The first release that was in was 3.9.0, so I changed to @since tags.

@access public is not necessary because this is the default access. I've also added a comment about the delay that's in the show method. This helps people understand the code.

#3 @atimmer
9 years ago

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

In 42675:

Docs: Improve JSDoc for media/views/spinner.js.

Props avillegasn.
Fixes #43230.

#4 @netweb
9 years ago

  • Focuses docs javascript → docs, javascript
  • Milestone Awaiting Review → 5.0

#5 @johnbillion
8 years ago

  • Milestone 5.0 → 5.0.1
  • Resolution fixed
  • Status closed → reopened

#6 @pento
8 years ago

  • Milestone 5.0.1 → 5.0.2

#7 @pento
8 years ago

  • Milestone 5.0.2 → 5.0.3

#8 @audrasjb
8 years ago

  • Milestone 5.0.3 → 5.1

Hi,

Per today's bug scrub, we are going to address Docs enhancements in 5.1, coming next month.

#9 @desrosj
8 years ago

  • Resolution → fixed
  • Status reopened → closed

I don't think anything further is required here. It was reopened for backport considerations, but there will not be a 5.0.4.

Note: See TracTickets for help on using tickets.