Warp icon indicating copy to clipboard operation
Warp copied to clipboard

Neovim theme looks different in iTerm2 and Warp

Open bookercodes opened this issue 2 years ago • 9 comments

Discord username (optional)

No response

Describe the bug

My Neovim theme looks different (especially the pink) in Warp compare to Iterm2.

To Reproduce

Steps to reproduce:

  1. Install https://github.com/wadackel/vim-dogrun
  2. Set colorscheme dogrun and set termguicolors
  3. Open a JavaScript file in iTerm and in Warp
  4. Compare

Expected behaviour

I expect Hyper to look like iTerm (which looks like the screenshots of the theme)

Screenshots

CleanShot 2022-04-18 at 16 23 29@2x

Operating System

MacOS

OS Version

12.0.1

Additional context

Hi team 👋! I wouldn't expect someone to go to the trouble of installing a Vim theme here! Just wondering what the likely suspect is?

bookercodes avatar Apr 18 '22 15:04 bookercodes

I can confirm that colors are very much off. I suspect 256 color support is not there

bijancn avatar Apr 20 '22 14:04 bijancn

same issue with ayu theme

Hyper vs Warp

image

wkei avatar May 03 '22 02:05 wkei

I have the same issue

xcpky avatar Jun 18 '22 09:06 xcpky

Does it still look off for yall?

elviskahoro avatar Jul 08 '22 19:07 elviskahoro

Seems to be fixed! (Warp is left, iterm2 is right. First 16 colors are user definable so they don't count) Screenshot 2022-07-11 at 16 53 17 Screenshot 2022-07-11 at 16 54 14 Screenshot 2022-07-11 at 16 53 49

bijancn avatar Jul 11 '22 14:07 bijancn

No, I am using v0.2022.07.04.09.08.stable_02 and I still have the same issue

xcpky avatar Jul 12 '22 03:07 xcpky

Yeah, I'm not sure that this is fixed yet. iTerm2 and Alacritty both look normal, Warp looks desaturated. Could it be related to lack of P3 gamut support? Screen Shot 2022-07-20 at 3 27 11 am

Brendonovich avatar Jul 19 '22 19:07 Brendonovich

Does it still look off for yall?

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 .inputrc now 😝!).

Comparisons

Warp (left) and Kitty (right) both running Neovim and the same colour scheme.

  • note the grey text is not working in Warp (directory names, line numbers, text, file info, etc.)
  • the dark green colour is not correct in Warp (branch name along the bottom, func keyword..)
Screen Shot 2022-07-26 at 11 54 19 AM image

jordanbrauer avatar Jul 26 '22 17:07 jordanbrauer

Going to CC @vorporeal and @gagata

elviskahoro avatar Jul 26 '22 18:07 elviskahoro

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)? It looks like (vim) background colors are shown OK (e.g. the blue background under Strings), but when they are set for the foreground, they are being overwritten.

BalintFulop 02-10 at 09 41@2x

Same code in Kitty:

BalintFulop 02-10 at 09 43@2x

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 avatar Oct 02 '22 07:10 bfulop

@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. :)

vorporeal avatar Oct 03 '22 15:10 vorporeal

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:

CleanShot 2022-11-10 at 15 17 22@2x

elviskahoro avatar Nov 10 '22 23:11 elviskahoro

Working great for me! Thank you for the fix!

bfulop avatar Nov 11 '22 09:11 bfulop