Support for Drupal 7 is ending on 5 January 2025—it’s time to migrate to Drupal 10! Learn about the many benefits of Drupal 10 and find migration tools in our resource center.
The link that toggles a textarea from plain to rich text uses an anchor tag with javascript in the href property. In some cases this could cause the browser to execute the javascript twice or more and is generally considered bad form. Instead javascript should be executed in the onclick property and href should point to a valid url (or "javascript:void(0);").
This patch changes the toggle link from
<a href="javascript:toggletinyMCE('edit-body')">rich text editor</a>
to
<a href="javascript:void(0);" onclick="javascript:toggletinyMCE('edit-body')">rich text editor</a>
Comment | File | Size | Author |
---|---|---|---|
javascript-toggle.patch | 1 KB | draenen |
Comments
Comment #1
draenen CreditAttribution: draenen commented