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.

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

Replacing low-res with downloaded high-res image flickers/flashes on new FireFox #96

Closed
rpearce opened this issue Dec 4, 2017 · 2 comments · Fixed by #117
Closed

Replacing low-res with downloaded high-res image flickers/flashes on new FireFox #96

rpearce opened this issue Dec 4, 2017 · 2 comments · Fixed by #117
Labels

Comments

@rpearce
Copy link
Owner

rpearce commented Dec 4, 2017

This issue is a:

  • Bug report

Expected behavior

I expected there to not be a flicker when the the larger image was loaded and replaced the old one.

Actual behavior

What actually happened: it flickers.

Steps to reproduce behavior

  1. open up the new firefox
  2. go to the example page (link below)
  3. click the first image (a bridge)

Link to repository/example of behavior

link to example

@rpearce rpearce added the bug label Dec 4, 2017
@kingdido999
Copy link

I have encountered the exact issue before and fixed it by force computing the hi-res image in DOM: kingdido999/zooming#7. Hope it helps.

@rpearce
Copy link
Owner Author

rpearce commented Jan 13, 2018

@kingdido999 Thanks for sharing! I'll check it out ASAP

rpearce added a commit that referenced this issue Jan 15, 2018
rpearce added a commit that referenced this issue Jan 15, 2018
rpearce added a commit that referenced this issue Jan 15, 2018
rpearce added a commit that referenced this issue Jan 15, 2018
@rpearce rpearce mentioned this issue Jan 15, 2018
5 tasks
rpearce added a commit that referenced this issue Jan 16, 2018
rpearce added a commit that referenced this issue Jan 16, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants