-
Notifications
You must be signed in to change notification settings - Fork 204
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
Conversation
能先把那些缩进的行去掉吗?要不是没办法看这个补丁 |
抱歉大佬,我开了自动格式化 现在把缩进去掉了,麻烦再看看 🙏 |
@kongds 能帮忙看一下这个补丁吗?谢谢 |
好的 |
lsp-bridge-peek.el
Outdated
: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)))) |
There was a problem hiding this comment.
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-peek.el
Outdated
(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")) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
这个highlight的face 定义一个单独face 可能会方便用户设置
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
改好了,还请大佬再过目一下
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
辛苦了 我这边没啥问题
No description provided.