Fix minor bugs and minor coding standards violations in edit.module, discovered by backporting [#2149599]

The minor bugs:

There's a bizarre bug in jQuery where calling `.removeAttr('contentEditable')` fails and calling `.removeAttr('contenteditable')` works. So changed to the latter.
See http://bugs.jquery.com/ticket/7792.

core/modules/edit/js/util.js is listed *after* edit.js; in rare situations it's possible that code in util.js is called before it has loaded, changing the order fixes that

Comment	File	Size	Author
#1	edit_minor_bugs_and_cleanup-2149599-1.patch	10.84 KB	wim leers

Comments

Comment #1

wim leers

Ghent 🇧🇪🇪🇺

commented 4 December 2013 at 11:25

Issue summary:	View changes
Status:	Active	» Needs review

Status	File	Size
new	edit_minor_bugs_and_cleanup-2149599-1.patch	10.84 KB

As said in the issue title: this patch is brought to you by the D7 backport of Edit module :)

Comment #2

jessebeach commented 9 December 2013 at 19:17

Status:	Needs review	» Reviewed & tested by the community
Related issues:		+#2153191: Removing an image from an article with quick edit does not trigger a field content change; the image is not removed.

I did manual regression testing for this change. I did not find any regressions related to this issue, but I did find a bug:

#2153191: Removing an image from an article with quick edit does not trigger a field content change; the image is not removed.

Again, unrelated as far as I can tell from this refactor that just changes some comments and variable names.