-
-
Notifications
You must be signed in to change notification settings - Fork 3.4k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Using CreateElement for adding HTML #12907
Conversation
5726940
to
756c655
Compare
Current coverage is 54.18% (diff: 100%)@@ master #12907 diff @@
==========================================
Files 466 466
Lines 69619 69619
Methods 2126 2126
Messages 0 0
Branches 0 0
==========================================
Hits 37724 37724
Misses 31895 31895
Partials 0 0
|
Signed-off-by: Shreyas Sinha <shreyas.sinha14@gmail.com>
var version_information_message_link = document.createElement('a'); | ||
version_information_message_link.href = url; | ||
version_information_message_link.class = 'disableAjax'; | ||
version_information_message_link.innerHTML = escapeHtml(data.version); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I think it would be better to use text here to avoid need for escapeHtml....
} | ||
var $liPmaVersion = $('#li_pma_version'); | ||
$liPmaVersion.find('span.latest').remove(); | ||
$liPmaVersion.append(version_information_message); | ||
document.getElementById("li_pma_version").appendChild(version_information_message); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Why getting element again, while we already have it as $liPmaVersion
?
ad0e0aa
to
3c2c392
Compare
Signed-off-by: Shreyas Sinha <shreyas.sinha14@gmail.com>
Merged, thanks for your contribution! |
#12764
Before submitting pull request, please check that every commit: