Skip to content

feat: support real-time refresh for markdown file - #42

Merged
seagle0128 merged 1 commit into
seagle0128:masterfrom
elton2048:feat-support-real-time-refresh-for-md
Feb 13, 2026
Merged

seagle0128 merged 1 commit into
seagle0128:masterfrom
elton2048:feat-support-real-time-refresh-for-md

Conversation

@elton2048

Copy link
Copy Markdown
Contributor
  • Provided new variable grip-real-time-refresh to enable such feature
  • Changed to remove preview file even the process is not existed

Note

I've only tried using mdopen for the renderer, I assume there is no special handling for other renderer.

 - Provided new variable `grip-real-time-refresh` to enable such
 feature
 - Changed to remove preview file even the process is not existed
@seagle0128

seagle0128 commented Jan 9, 2026 •

Copy link
Copy Markdown
Owner

Correct. All renders handling are same.

Actually the feature of realtime/instant previews was added before, twice. But removed due to the performance issue. Currently mdopen and go-grip may have the better performance to render the pages. We can add it back.

I went through the changes. I got some questions:

  1. Did you check renders for org files? It was pretty slow before.
  2. There is only add-hook for after-change-functions, while not remove-hook.

Reference: 24e9f8e, 1a61bb7, 7f1effc

@elton2048

Copy link
Copy Markdown
Contributor Author
  1. No, I think rendering for org files is a bit more complicated to handle, thus I think it's better to do so after this PR, separating the afford.
  2. That's something I missed

I think it's better to see if this support is a really wish so to proceed the change or not, I could follow that up if so, or maybe reverting those commits are more easier. Anyway thanks for the review :)

@seagle0128
seagle0128 merged commit b8b9e60 into seagle0128:master Feb 13, 2026
21 checks passed
@seagle0128

Copy link
Copy Markdown
Owner

Okay. Then please move forward, please.

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.

2 participants