$(document).ready() and $(window).ready() are equivalent, so the use of the later one in basic/js/source/scripts.js could cause confusion to users.
See http://stackoverflow.com/questions/17278972/why-would-you-use-jquery-win...

CommentFileSizeAuthor
#2 get_rid_of-2568535-2.patch1.71 KBjoelpittet
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

e5sego created an issue. See original summary.

joelpittet’s picture

joelpittet
Primary language English
Location Vancouver
CreditAttribution: joelpittet commented
Version: 7.x-4.2 » 7.x-4.x-dev
Category: Plan » Task
FileSize
get_rid_of-2568535-2.patch1.71 KB

Agreed, here is the fix.

  • joelpittet committed d029008 on 7.x-4.x 
    Issue #2568535 by joelpittet, e5sego: Get rid of $(window).ready() in...
joelpittet’s picture

joelpittet
Primary language English
Location Vancouver
CreditAttribution: joelpittet commented
Status: Active » Fixed

Thanks pushed to -dev

Status: Fixed » Closed (fixed)

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