From de427dc58959775f104776b4316349a90190fd1d Mon Sep 17 00:00:00 2001 From: Mike Greiling Date: Sun, 11 Dec 2016 21:35:19 -0600 Subject: [PATCH] use $ sign to make it clear when dealing with jQuery object --- app/assets/javascripts/preview_markdown.js | 18 +++++++++--------- 1 file changed, 9 insertions(+), 9 deletions(-) diff --git a/app/assets/javascripts/preview_markdown.js b/app/assets/javascripts/preview_markdown.js index e2260b37f91..6d9e40d4e91 100644 --- a/app/assets/javascripts/preview_markdown.js +++ b/app/assets/javascripts/preview_markdown.js @@ -16,19 +16,19 @@ MarkdownPreview.prototype.ajaxCache = {}; - MarkdownPreview.prototype.showPreview = function(form) { + MarkdownPreview.prototype.showPreview = function($form) { var mdText, preview; - preview = form.find('.js-md-preview'); - mdText = form.find('textarea.markdown-area').val(); + preview = $form.find('.js-md-preview'); + mdText = $form.find('textarea.markdown-area').val(); if (mdText.trim().length === 0) { preview.text('Nothing to preview.'); - this.hideReferencedUsers(form); + this.hideReferencedUsers($form); } else { preview.text('Loading...'); this.renderMarkdown(mdText, (function(response) { preview.html(response.body); preview.renderGFM(); - this.renderReferencedUsers(response.references.users, form); + this.renderReferencedUsers(response.references.users, $form); }).bind(this)); } }; @@ -58,13 +58,13 @@ }); }; - MarkdownPreview.prototype.hideReferencedUsers = function(form) { - form.find('.referenced-users').hide(); + MarkdownPreview.prototype.hideReferencedUsers = function($form) { + $form.find('.referenced-users').hide(); }; - MarkdownPreview.prototype.renderReferencedUsers = function(users, form) { + MarkdownPreview.prototype.renderReferencedUsers = function(users, $form) { var referencedUsers; - referencedUsers = form.find('.referenced-users'); + referencedUsers = $form.find('.referenced-users'); if (referencedUsers.length) { if (users.length >= this.referenceThreshold) { referencedUsers.show();