Make WordPress Core

Opened 12 years ago

Closed 12 years ago

#28459 closed defect (bug) (fixed)

Improve wp.media documentation

Reported by: ericlewis Owned by: wonderboymusic
Priority: normal Milestone: 4.1
Component: Media Version: 3.5
Severity: normal Keywords: has-patch commit
Cc: Focuses: javascript, docs

Description

Catch all ticket for me improving documentation in wp.media. Brisk review and acceptance appreciated.

Attachments (9)

28459.diff​ (8.3 KB ) - added by ericlewis 12 years ago.
28459.2.diff​ (8.2 KB ) - added by ericlewis 12 years ago.
28459.3.diff​ (8.5 KB ) - added by ericlewis 12 years ago.
28459.4.diff​ (8.8 KB ) - added by ericlewis 12 years ago.
28459.5.diff​ (2.5 KB ) - added by ericlewis 12 years ago.
28459.6.diff​ (12.6 KB ) - added by DrewAPicture 12 years ago.
Some @since tags
28459.7.diff​ (10.0 KB ) - added by ericlewis 12 years ago.
28459.8.diff​ (16.6 KB ) - added by ericlewis 12 years ago.
28459.9.diff​ (25.0 KB ) - added by ericlewis 12 years ago.

Download all attachments as: .zip

Change History (31)

@ericlewis
12 years ago

@ericlewis
12 years ago

#1 @ericlewis
12 years ago

First pass: attachment:28459.2.diff​

  • remove unused sidebar in media.controller.Library defaults. (I'd guess sidebar was previously a region, which was consumed by AttachmentsBrowser at some point)
  • group Selection defaults, Initial region modes, and Attachments browser defaults together in each state's constructor defaults.

(Previously removed describe from a few constructors in attachment:28459.diff​ thinking it isn't used, but realized it is)

#2 @DrewAPicture
12 years ago

  • Focuses docs added

#3 @wonderboymusic
12 years ago

In 28675:

Media modal documentation:

Add some clarity to default values for controllers.

Props ericlewis.
See #28459.

#4 @wonderboymusic
12 years ago

  • Milestone Awaiting Review → 4.0

@ericlewis
12 years ago

#5 @ericlewis
12 years ago

In attachment:28459.3.diff​,

  • Boil down wp.media() description.
  • Fix up and add to media.controller.Region event documentation.
  • Add some state constructor descriptions.
  • Methods are mapped from a StateMachine to its states collection, not the other way around.
  • More minutia in diff.
Version 0, edited 12 years ago by ericlewis (next)

@ericlewis
12 years ago

#6 @ericlewis
12 years ago

In attachment:28459.4.diff​:

#7 @wonderboymusic
12 years ago

In 28688:

Improve inline doc descriptions in media-views.js.

Props ericlewis.
See #28459.

#8 @wonderboymusic
12 years ago

  • Resolution → fixed
  • Status new → closed

#9 @DrewAPicture
12 years ago

\o/ Nice work guys :-)

#10 @ericlewis
12 years ago

  • Resolution fixed
  • Status closed → reopened

This ain't over till it's over

@ericlewis
12 years ago

#11 @ericlewis
12 years ago

In attachment:28459.5.diff​ add docs to media.view.MediaFrame.Select methods. Remove a couple @this context documentation bits which were incorrect.

#12 @DrewAPicture
12 years ago

  • Owner set to wonderboymusic
  • Status reopened → reviewing

#13 @wonderboymusic
12 years ago

In 28989:

Add more docs for media.view.MediaFrame.Select methods.

Props ericlewis.
See #28459.

#14 @DrewAPicture
12 years ago

  • Type enhancement → defect (bug)

Missing docs are a bug.

@DrewAPicture
12 years ago

Some @since tags

#15 @DrewAPicture
12 years ago

  • Milestone 4.0 → Future Release

I have a feeling this will be an ongoing effort.

#16 @wonderboymusic
12 years ago

In 29739:

Add some @since annotations to media-views.js.

Props DrewAPicture.
See #28459.

#17 @wonderboymusic
12 years ago

  • Milestone Future Release → 4.1

@ericlewis
12 years ago

#18 @ericlewis
12 years ago

  • Keywords has-patch commit added

@ericlewis
12 years ago

@ericlewis
12 years ago

#20 @ericlewis
12 years ago

In attachment:28459.9.diff​, add document the selection attribute for media.controller.Library constructor.

This supplants attachment:28459.8.diff​.

#21 @wonderboymusic
12 years ago

In 30245:

Improve wp.media documentation.

Props ericlewis.
See #28459.

#22 @ocean90
12 years ago

  • Resolution → fixed
  • Status reviewing → closed

Closing as fixed for 4.1.

Note: See TracTickets for help on using tickets.