#38933 closed defect (bug) (fixed)
Document wp-admin/js/editor.js
Reported by: | terwdan | Owned by: | SergeyBiryukov |
---|---|---|---|
Milestone: | 4.8 | Priority: | normal |
Severity: | normal | Version: | |
Component: | Editor | Keywords: | has-patch commit |
Focuses: | javascript, docs | Cc: |
Description
Documented the wp-admin/js/editor.js file.
Please also give props to ireneyoast
Attachments (2)
Change History (9)
#4
@
8 years ago
- Keywords has-patch commit added
- Milestone changed from Awaiting Review to 4.8
@terwdan: editorjs.diff is a great start, thank you for the patch!
I've only made a couple of minor adjustments:
- The standards say that third-person singular verbs should be used for functions and closures, but I think regular comments that refer to the next few lines should still be "Do something" rather than "Does something".
- Some
@since
entries weren't fully blamed to the original change. We don't generally introduce new functions in minor releases :)
Does 38933.patch look good to you?
Note: See
TracTickets for help on using
tickets.
Missing/incomplete docs are a bug, per the Docs handbook and comment:3:ticket:37365: