#38933 closed defect (bug) (fixed)
Document wp-admin/js/editor.js
| Reported by: | terwdan | Owned by: | SergeyBiryukov |
|---|---|---|---|
| Priority: | normal | Milestone: | 4.8 |
| Component: | Editor | Version: | |
| Severity: | normal | Keywords: | has-patch commit |
| Cc: | Focuses: | javascript, docs |
Description
Documented the wp-admin/js/editor.js file.
Please also give props to ireneyoast
Attachments (2)
Change History (9)
#4
@
10 years ago
- Keywords has-patch commit added
- Milestone Awaiting Review → 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
@sinceentries 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.
![(please configure the [header_logo] section in trac.ini)](/chrome/site/your_project_logo.png)
Missing/incomplete docs are a bug, per the Docs handbook and comment:3:ticket:37365: