Skip to content

Change the region highlighting during type enclosing#83

Merged
xvw merged 3 commits into
mainfrom
temporary-pulse-instead-of-sit
Jan 20, 2026
Merged

Change the region highlighting during type enclosing#83
xvw merged 3 commits into
mainfrom
temporary-pulse-instead-of-sit

Conversation

@xvw

@xvw xvw commented Jan 20, 2026

Copy link
Copy Markdown
Member

Non blocking enclosing highlighting
cc @bcc32

Comment thread ocaml-eglot-util.el
(unwind-protect (sit-for 60) (delete-overlay overlay))))
(let ((beg (eglot--lsp-position-to-point (cl-getf range :start)))
(end (eglot--lsp-position-to-point (cl-getf range :end))))
(pulse-momentary-highlight-region beg end face)))

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I think it would be nice if we bound pulse-flag to nil here, so that it will be highlighted until the next command. This makes using grow/shrink easier.

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks. Done in b609988

@xvw xvw force-pushed the temporary-pulse-instead-of-sit branch from b609988 to 19b2fdf Compare January 20, 2026 15:44
@xvw xvw merged commit aadfa57 into main Jan 20, 2026
5 checks passed
@xvw xvw deleted the temporary-pulse-instead-of-sit branch January 20, 2026 16:03
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