New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Use Headless Chrome over PhantomJS for (QUnit) tests #89
base: develop
Are you sure you want to change the base?
Conversation
…|scroll)' over just 'estatico.events.(resize|scroll)'
@@ -57,7 +57,7 @@ class WindowEventListener { | |||
|
|||
// Save to global namespace | |||
$.extend(true, estatico, { events: {} }); | |||
estatico.events[eventName] = debouncedEventName; | |||
estatico.events[debouncedEventName.split('.')[0]] = debouncedEventName; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This is necessary if we want to have both debounced
and throttled
variants of scroll
/resize
. Otherwise the one registered later will overwrite the first one. That's what led to the slideshow tests failing.
I have updated docs/Coding_Guidelines.md
accordingly.
Is this a breaking change for anyone?
Added to new implementation |
Goals:
js:qunit
TODO:
js:test
since it will also check for console.error unrelated to specific QUnit tests