Skip to content

Fix 13 typos in comments, docs and site copy - #1221

Open
Avicennasis wants to merge 1 commit into
jsfiddle:developfrom
Avicennasis:docs/fix-typos
Open

Avicennasis wants to merge 1 commit into
jsfiddle:developfrom
Avicennasis:docs/fix-typos

Conversation

@Avicennasis

Copy link
Copy Markdown

Summary

Fixes 13 typos across 10 files - code comments, plus one user-facing sentence on the site. One change is not a comment; it is called out separately below.

Behaviour change - please review this one carefully

  • togetherjs/forms.js: this._editor().destory(); -> this._editor().destroy(); inside the TinyMCE tracker's destroy() method. _editor() returns a TinyMCE editor instance ($(this.element).data("tinyEditor")) and the TinyMCE API method is destroy(), so the current call raises a TypeError. destory occurs exactly once in the whole repository.

Site copy

  • site/index.html and site/js/custom.js: a unique TogetherJS url to a friend -> URL. The same sentence is duplicated in both files, and site/docs/index.md uses URL throughout.

Comment typos

  • togetherjs/videos.js: uneccesary -> unnecessary
  • togetherjs/session.js: succede -> succeed; arives -> arrives; weather -> whether
  • togetherjs/walkthrough.js: substititions -> substitutions
  • togetherjs/youtubeVideos.js: supress -> suppress; everytime -> every time
  • togetherjs/ot.js: preceed -> precede
  • togetherjs/jqueryPlugins.js: dimenensions -> dimensions
  • togetherjs/ui.js: draggin -> dragging

No files under togetherjs/libs/, togetherjs/tests/doctestjs/ or site/examples/friendlycode/ were touched - those are third-party/vendored copies.

Also fixes a misspelled TinyMCE API call (destory -> destroy).

Signed-off-by: Léon Avic Simmons <Avicennasis@gmail.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant