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

Highlight the Symbol you want to see while peeking to find it. #695

Merged
merged 4 commits into from
Aug 1, 2023
Merged

Conversation

AllTheLife
Copy link
Contributor

No description provided.

@manateelazycat
Copy link
Owner

能先把那些缩进的行去掉吗?要不是没办法看这个补丁

@AllTheLife
Copy link
Contributor Author

AllTheLife commented Jul 31, 2023

抱歉大佬,我开了自动格式化

现在把缩进去掉了,麻烦再看看 🙏

@manateelazycat
Copy link
Owner

@kongds 能帮忙看一下这个补丁吗?谢谢

@kongds
Copy link
Contributor

kongds commented Aug 1, 2023

@kongds 能帮忙看一下这个补丁吗?谢谢

好的

:extend t))
(put-text-property highlight-begin highlight-end 'face '(:background "#623d73" :foreground "white"))
(setq file-content (concat (buffer-substring beg end) "\n"))
(put-text-property highlight-begin highlight-end 'face '(:background (background-color-at-point))))
Copy link
Contributor

Choose a reason for hiding this comment

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

这块用remove-text-properties会不会更好一点

(lsp-bridge--add-face file-content
(list :background lsp-bridge-peek--bg
:extend t))
(put-text-property highlight-begin highlight-end 'face '(:background "#623d73" :foreground "white"))
Copy link
Contributor

Choose a reason for hiding this comment

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

这个highlight的face 定义一个单独face 可能会方便用户设置

Copy link
Contributor Author

Choose a reason for hiding this comment

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

改好了,还请大佬再过目一下

Copy link
Contributor

Choose a reason for hiding this comment

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

辛苦了 我这边没啥问题

@manateelazycat manateelazycat merged commit c902859 into manateelazycat:master Aug 1, 2023
1 check passed
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.

3 participants