Skip to content
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.

Sign up for GitHub

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

Jump to bottom

Autorefresh stopped working on 0.7 #468

Closed
jve opened this issue Jun 10, 2015 · 3 comments
Closed

Autorefresh stopped working on 0.7 #468

jve opened this issue Jun 10, 2015 · 3 comments
Labels
bug
Milestone
Release 0.7.1

Comments

@jve
Copy link

jve commented Jun 10, 2015

Hi,

We just upgraded to tessera 0.7 and found that the autorefresh stopped working.
We are using firefox 38.

I had a quick peak in the code but I am in no way a javascript expert but did find that in the refresh function line 6237 in tessera.js that 'this' does not point to the manager object but the actual window which doesn't have the 'current' object and as such skips the load function all together.

Hope that helps.
Jozef
@aalpern
Copy link
Member

aalpern commented Jun 11, 2015

Whoops - thanks! Will get that fixed up shortly.

@aalpern aalpern added the bug label Jun 11, 2015
@aalpern aalpern added this to the Release 0.7.1 milestone Jun 12, 2015
aalpern added a commit that referenced this issue Jun 12, 2015
@aalpern
Fix auto refresh
00a2cc2 
This was a leftover `this` scope issue from moving to es6 classes.
Passing a closure to `window.setInterval()` instead of `this.refresh`
fixes it.

Fixes issue #468
@aalpern
Copy link
Member

aalpern commented Jun 12, 2015

Fixed

@aalpern aalpern closed this as completed Jun 12, 2015
@jve
Copy link
Author

jve commented Jun 12, 2015

Great thanks for the quick turnaround!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
bug
Projects
None yet
Milestone
Release 0.7.1
Development

No branches or pull requests
2 participants
@jve @aalpern