-
Notifications
You must be signed in to change notification settings - Fork 308
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
Neovim theme looks different in iTerm2 and Warp #1110
Comments
I can confirm that colors are very much off. I suspect 256 color support is not there |
same issue with ayu theme Hyper vs Warp |
I have the same issue |
Does it still look off for yall? |
No, I am using v0.2022.07.04.09.08.stable_02 and I still have the same issue |
Unfortunately, yes @elviskahoro 😞 .. It seems that 256 is supported & consistent, but anything outside of those colours (using hexcodes) is washed/desaturated or simply does not render :( At this point I would say that this is the biggest hurdle for me in adopting Warp as my terminal. You guys have done an excellent job at the UX (just needs support for vim mode from ComparisonsWarp (left) and Kitty (right) both running Neovim and the same colour scheme.
|
Going to CC @vorporeal and @gagata |
It looks like Warp has no issues displaying all the colors, see the left side of my screenshot. I'm using a color highlighter plugin inside Neovim, and all the colors are properly showing. It somehow then overrides the theme colors with some random ones. Maybe it's trying to apply it's own color highlighting over Neovim (treesitter)? Same code in Kitty: It's a pity because in my own, subjective tests, Warp is the fastest among all the terminal emulators, even Neovim some GUIs! 🚀 👏 To test, I used a Neovim smooth scrolling plugin, on a 4K screen, created vertical window splits inside Neovim and tested (smooth) scrolling in the middle one. It was only in Warp where the scrolling stayed smooth no matter how many splits I created. Kitty, Alacritty, Fvim were all slower to begin with (one split, Fvim) or started to drop frames when adding new (vertical) splits (e.g. Kitty). |
@bfulop Thanks for the detailed comment! We've identified the source of the problem and have a fix in the works; we're aiming to get it out in next week's release. :) |
Sharing this across various color related issues. The team fixed one of the core issues with colors and there's a new toggle under Settings > Appearance that let's you configure whether to enforce a minimum contrast (one of the issues). Please check if you're still having color issues again and test tweaking the toggle: |
Working great for me! Thank you for the fix! |
Discord username (optional)
No response
Describe the bug
To Reproduce
Steps to reproduce:
colorscheme dogrun
andset termguicolors
Expected behaviour
Screenshots
Operating System
MacOS
OS Version
12.0.1
Additional context
The text was updated successfully, but these errors were encountered: