Opened 4 years ago
Closed 4 years ago
#51401 closed enhancement (fixed)
Docs: Fix bracket issue in autosave JS DocBlock
Reported by: | dilipbheda | Owned by: | SergeyBiryukov |
---|---|---|---|
Milestone: | 5.6 | Priority: | normal |
Severity: | normal | Version: | 5.6 |
Component: | Autosave | Keywords: | has-patch commit |
Focuses: | javascript, docs | Cc: |
Description
Fix inline documentation.
Attachments (2)
Change History (8)
#4
@
4 years ago
- Keywords commit added
- Type changed from defect (bug) to enhancement
51401.1.patch thanks for update.
Patch looks good. Marking for commit.
#5
@
4 years ago
- Milestone changed from Awaiting Review to 5.6
- Owner set to SergeyBiryukov
- Status changed from new to reviewing
Thanks for the patch!
I haven't found any other instances in core where we document object properties like this, but the closest match appears to be the "Record Type" from Google JavaScript Style Guide:
{{myNum: number, myObject}}
So it looks like double brackets are OK, let's just correct the second bracket placement.
Note: See
TracTickets for help on using
tickets.
Hi there!
The 51401.patch patch needs update because you have added an extra }.