iTerm2 icon indicating copy to clipboard operation
iTerm2 copied to clipboard

Adding a new color preference setting that defines a custom color for underlining.

Open rsmmr opened this issue 9 years ago • 4 comments

This is on top of the v3.1 branch. I believe I have followed most of the instructions in #286, except for two things:

  • I had to modify [self.colorMap colorForKey:kColorMapUnderline] to account for the fact that we now expect it to return nil in this case. It would normally have returned a hardcoded red. Not so nice, maybe I'm missing something?
  • I only added 1 one of the 3 test images, the non-retina version, as that's what I'm working on here. Not sure if/how I can create a retina version on a non-retina system? Also, I wasn't sure what exactly "run the test suite in Travis" means, so I left that part out. I would appreciate if you could add the missing images for testing when merging.

Please have a look. Feel free to send back to me if there's more to do.

rsmmr avatar Jun 13 '16 04:06 rsmmr

Good work! Very close to submittable; I just wrote a few small comments. I'll take care of adding the golden image for Travis.

gnachman avatar Jun 14 '16 05:06 gnachman

Updated with your feedback, except for the change to result, see above.

rsmmr avatar Jun 15 '16 03:06 rsmmr

Merged. Thank you!

gnachman avatar Jun 15 '16 22:06 gnachman

Thanks for merging and the feedback, and more generally for all your great work on iTerm2!

rsmmr avatar Jun 15 '16 23:06 rsmmr

Hi, is there anyway I can change iTerm2 Colorspace from sRGB to P3 or to my current display's colorspace?

josepheiba avatar Feb 02 '24 06:02 josepheiba

@josepheiba The beta uses p3

gnachman avatar Feb 05 '24 20:02 gnachman