Here's the offending code:

     if ($('.connection-settings-update-filetransfer-default-wrapper').length > 0) {
       console.log($('.connection-settings-update-filetransfer-default-wrapper'));
       $('.connection-settings-update-filetransfer-default-wrapper').css('float', 'none');
     }

It looks like the console.log got in there by mistake. This also needs to be fixed in D7

CommentFileSizeAuthor
#6 1333314-6.patch585 bytesericduran
#1 0001-Issue-1333314-by-ericduran-Fixed-Left-over-console.l.patch954 bytesericduran

Comments

ericduran’s picture

ericduran commented
Status: Active » Needs review
StatusFileSize
new 0001-Issue-1333314-by-ericduran-Fixed-Left-over-console.l.patch954 bytes

Simple enough.

ericduran’s picture

ericduran commented
Issue tags: +JavaScript, +Update manager, +Needs backport to D7

Tagging.

xjm’s picture

xjm
she/her
Primary language English
commented
Status: Needs review » Reviewed & tested by the community

Yep.

ksenzee’s picture

ksenzee
she/her
commented

Ouch. Yes please.

catch’s picture

catch
he/him
Primary language English
commented
Version: 8.x-dev » 7.x-dev
Status: Reviewed & tested by the community » Patch (to be ported)

Whoopsie.

Committed/pushed to 8.x. This needs a re-roll for 7.x.

ericduran’s picture

ericduran commented
StatusFileSize
new 1333314-6.patch585 bytes

Simple enough.

catch’s picture

catch
he/him
Primary language English
commented
Status: Patch (to be ported) » Reviewed & tested by the community
webchick’s picture

webchick
she/they
Primary language English
Location Vancouver 🇨🇦
commented
Status: Reviewed & tested by the community » Fixed

Oops. :P

Committed and pushed to 7.x. Thanks!

ericduran’s picture

ericduran commented

Ahhh, if only all patches were this simple ;)

Automatically closed -- issue fixed for 2 weeks with no activity.