Add back the `wait` time to Drupal.debounce function call.

CommentFileSizeAuthor
#3 flame-old.png167.61 KBnod_
#3 flame-new.png163.95 KBnod_
add-debounce-wait-time.patch540 bytesdroplet
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

droplet’s picture

droplet CreditAttribution: droplet commented
Status: Active » Needs review
nod_’s picture

nod_
Primary language French
Location Lille
CreditAttribution: nod_ commented
Status: Needs review » Reviewed & tested by the community

I could have sworn we had a default 200ms on our drupal debounce function.

We need this otherwise using debounce is useless.

nod_’s picture

nod_
Primary language French
Location Lille
CreditAttribution: nod_ commented
FileSize
flame-new.png163.95 KB
flame-old.png167.61 KB

Compare how busy JS is before and after.

alexpott’s picture

alexpott
he/they
Primary language English
Location 🇪🇺🌍
CreditAttribution: alexpott at Chapter Three commented
Status: Reviewed & tested by the community » Fixed

This issue is a normal bug fix, and doesn't include any disruptive changes, so it is allowed per https://www.drupal.org/core/beta-changes. Committed 1c3f898 and pushed to 8.0.x. Thanks!

  • alexpott committed 1c3f898 on 8.0.x 
    Issue #2474439 by droplet: Missing Drupal.debounce wait time in...

Status: Fixed » Closed (fixed)

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