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

Load a 2nd webview, and swap it out with the visible one when it's done rendering. #1216

Open
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

adamwulf
Copy link
Contributor

Renders updated markdown into a 2nd webview behind the currently visible webview. when it's done loading, the visible view is removed and replaced with the one beneath it. this way, the content doesn't flicker as you type.

Fixes #1104

@FranklinYu
Copy link
Member

The blinking is indeed fixed, but on my machine the rendering delay is significantly longer than current master branch.

@handrews
Copy link

@FranklinYu How much longer is "significantly longer"? I find the preview pane unusable due to the constant flickering, so an update that lags would be much preferable. There could be a preference to support whichever option people prefer since it is a tradeoff.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Rendering pane flashing
3 participants