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

perf(ui): ui twick & update theme-related config #383

Merged
merged 5 commits into from
Dec 24, 2022
Merged

perf(ui): ui twick & update theme-related config #383

merged 5 commits into from
Dec 24, 2022

Conversation

CharlesChiuGit
Copy link
Collaborator

@CharlesChiuGit CharlesChiuGit commented Dec 24, 2022

Add some nice visual effects based on current plugins.

  1. wilder
    wilder

  2. indent-blacnkline
    indent-line

  3. virtual-text
    v-text

@ayamir
Copy link
Owner

ayamir commented Dec 24, 2022

I have just been diagnosed with COVID-19 and need to rest for a few days.

Copy link
Collaborator

@Jint-lzxy Jint-lzxy left a comment

Choose a reason for hiding this comment

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

IMO I will accept the changes for wilder. However, for indent-blankline and virtualtexts, they are intentionally not background highlighted as this would affect performance when too many such objects are rendered in the same screenview (for example, scrolling quickly will cause deferred rendering).

-- For native lsp configs.
DiagnosticVirtualTextError = { bg = cp.none },
DiagnosticVirtualTextWarn = { bg = cp.none },
DiagnosticVirtualTextInfo = { bg = cp.none },
DiagnosticVirtualTextHint = { fg = cp.rosewater, bg = cp.none },

lua/modules/tools/config.lua Outdated Show resolved Hide resolved
@CharlesChiuGit
Copy link
Collaborator Author

CharlesChiuGit commented Dec 24, 2022

@Jint-lzxy fair enough, will remove those part.

@CharlesChiuGit
Copy link
Collaborator Author

@ayamir take care!

@CharlesChiuGit
Copy link
Collaborator Author

@Jint-lzxy PTAL.

@CharlesChiuGit CharlesChiuGit removed the request for review from ayamir December 24, 2022 14:57
Copy link
Collaborator

@Jint-lzxy Jint-lzxy left a comment

Choose a reason for hiding this comment

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

LGTM 🎉 Thank you for your efforts!

@Jint-lzxy Jint-lzxy merged commit d93d457 into ayamir:main Dec 24, 2022
@CharlesChiuGit CharlesChiuGit deleted the perf-ui branch December 24, 2022 16:19
vkingw pushed a commit to vkingw/nvimdots that referenced this pull request Feb 7, 2023
* perf(ui): ui twick

* revert virtual text bg highlisht

* use it without wrapper function

* revert indent-blankline & use it without wrapper function

* remove wrapper function
YanTree pushed a commit to YanTree/nvim that referenced this pull request Apr 9, 2023
* perf(ui): ui twick

* revert virtual text bg highlisht

* use it without wrapper function

* revert indent-blankline & use it without wrapper function

* remove wrapper function
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