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)
Change History (31)
#5
@
12 years ago
- Boil down
wp.media()description. Don't say that it opens a frame when it doesn't. - Fix up and add to
media.controller.Regionevent documentation. - Add some
stateconstructor descriptions. - Methods are mapped from a
StateMachineto itsstatescollection, not the other way around. - More minutia in diff.
#6
@
12 years ago
- everything in attachment:28459.3.diff:
media.view.RouterItem.clickdocumentation
#11
@
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.
#19
@
12 years ago
attachment:28459.8.diff supplants attachment:28459.7.diff.
#20
@
12 years ago
In attachment:28459.9.diff, add document the selection attribute for media.controller.Library constructor.
This supplants attachment:28459.8.diff.
Note:
See TracTickets
for help on using tickets.
![(please configure the [header_logo] section in trac.ini)](/chrome/site/your_project_logo.png)
First pass: attachment:28459.2.diff
sidebarinmedia.controller.Librarydefaults. (I'd guess sidebar was previously a region, which was consumed by AttachmentsBrowser at some point)Selection defaults,Initial region modes, andAttachments browser defaultstogether in each state's constructor defaults.(Previously removed
describefrom a few constructors in attachment:28459.diff thinking it isn't used, but realized it is)