Add documentation for js/media/views/attachments.js

[ireneyoast]

Add documentation for js/media/views/attachments.js
Please also give props to ​herregroen, ​andizer, and ​jipmoors

[atimmer]

I think this is a duplicate of https://core.trac.wordpress.org/ticket/42832.

[netweb]

There are some differences in the media-views-attachments-documentation.diff​ patch here in this ticket to that of what was committed in [42415]. Probably worth confirming from the original GitHub to see if [42415] had been merged into the GitHub repo or not.

Can anyone confirm any of this please? cc @ireneyoast, @herregroen, @andizer, and @jipmoors

[atimmer]

I've made some changes to the actual commit that are necessary for the JSDoc parser to be able to parse the documentation correctly. In this specific commit, if there is both documentation on the constructor (initialize with @constructs) and the class (@class) the end result is duplicated documentation.

Like @ireneyoast I have removed all the @summary tags from the previous patch. And fixed the line length. I can ask @ireneyoast to merge master into her branch on GitHub to verify that these differences are all there is. But I am fairly confident that what is on trunk now is correct.

[ireneyoast]

@netweb I merged master into our branch, and confirmed that all changes we made are reflected in @atimmer's committed patch.

[netweb]

Duplicate of #42832.

Thanks @ireneyoast & @atimmer +1