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

Add link to iterm theme port #80

Open
wants to merge 3 commits into
base: master
Choose a base branch
from
Open

Add link to iterm theme port #80

wants to merge 3 commits into from

Conversation

epszaw
Copy link

@epszaw epszaw commented Nov 22, 2020

No description provided.

@gkeep
Copy link
Contributor

gkeep commented Nov 22, 2020

Looks good!

Could you also add the dark theme to your repository, too?
I know terminal.sexy provides it via .xresources, but it would be nice to have it in a dedicated repository, too.

@epszaw
Copy link
Author

epszaw commented Nov 22, 2020

I think yes. Will made port today 👍

@epszaw
Copy link
Author

epszaw commented Nov 22, 2020

@gkeep done

@epszaw epszaw changed the title Add link to iterm light theme port Add link to iterm theme port Nov 22, 2020
@epszaw
Copy link
Author

epszaw commented Nov 24, 2020

@cocopon what do you think about the merge? 🙂

@cocopon
Copy link
Owner

cocopon commented Dec 9, 2020

Thank you for your contribution, and sorry for my late reply...

  • Could you fix the app name in readme from iterm2 to iTerm2?
  • I prefer that the cursor color is same as its foreground color. What do you think?
  • Could you re-check a color for Links and Cursor of the light theme? It is not a color in the palette.
    image

@epszaw
Copy link
Author

epszaw commented Dec 10, 2020

Yes, I will check it on this week.

@epszaw
Copy link
Author

epszaw commented Dec 10, 2020

@cocopon it's done for cursor color in the light theme and readme fix is also pushed.
But I not fully understood you about links colors. I checked blue and magenta colors (commonly links are highlighted with these colors) and they're right. Can you explain?

@cocopon
Copy link
Owner

cocopon commented Dec 12, 2020

it's done for cursor color in the light theme and readme fix is also pushed.

Thank you!

Here is a screenshot of Links:
(005bbb is not a part of Iceberg palette)
image

and it will be applied to URL-like text when command key is pressed:
image

@epszaw
Copy link
Author

epszaw commented Dec 12, 2020

Finally, I got it!
Done, @cocopon !

@cocopon
Copy link
Owner

cocopon commented Dec 13, 2020

Thank you! Could you re-check a color for Curosor? It seems to be back to the previous value.
image

@epszaw
Copy link
Author

epszaw commented Dec 15, 2020

@cocopon done. Again :)

@cocopon
Copy link
Owner

cocopon commented Dec 15, 2020

@epszaw
Copy link
Author

epszaw commented Dec 15, 2020

@cocopon again :D

@epszaw
Copy link
Author

epszaw commented Dec 17, 2020

@cocopon any new here?

@cocopon
Copy link
Owner

cocopon commented Dec 25, 2020

It seems to be fixed.

BTW, what is Bold? I noticed that this is also not a color of the palette.
bold

@epszaw
Copy link
Author

epszaw commented Dec 25, 2020

It seems to be fixed.

BTW, what is Bold? I noticed that this is also not a color of the palette.
bold

Which color will fit better here? What do you think?
I propose to use #33374c (foreground) or #8389a3 (white bright).

@cocopon
Copy link
Owner

cocopon commented Dec 27, 2020

Do you know how to check a bold text? I'd like to check it on the actual terminal but haven't found a way to print it yet.

@epszaw
Copy link
Author

epszaw commented Dec 27, 2020

That it.

Source code here (you need to install chalk):

const chalk = require('chalk')

console.log(chalk.bold('Bold text here!'))

image

@cocopon
Copy link
Owner

cocopon commented Jan 9, 2021

Thank you for sharing the code! I understand. white bright seems better for this case.

@epszaw
Copy link
Author

epszaw commented Jan 9, 2021

@cocopon done for the both themes! ❤️

@cocopon
Copy link
Owner

cocopon commented Jan 9, 2021

  • Dark Foreground should be #c6c8d1
  • Light Bold should be #262a3f
  • Light Links seems to be back to the previous value again...

@epszaw
Copy link
Author

epszaw commented Jan 9, 2021

I'm sorry, but iterm2 has some problems with themes exporting. Check it please again.

@cocopon
Copy link
Owner

cocopon commented Jan 14, 2021

Found additional issues:

  • Dark Cursor should be same as Foreground (#c6c8d1)
  • Dark Cursor text should be same as Background (#161821)
  • Dark Selected text should be same as Foreground (#c6c8d1)
  • Light Cursor text should be same as Background (#e8e9ec)
  • Light Selection should be same as Visual of Vim (#c9cdd7)
  • Cursor guide should be same as Cursorline of Vim
    • Dark: #1e2132
    • Light: #dcdfe7

@epszaw
Copy link
Author

epszaw commented Jan 14, 2021

@cocopon done here

@cocopon
Copy link
Owner

cocopon commented Jan 15, 2021

  • These values are rolled back... Please, please re-check ALL colors before/after commit:
    • Dark Foreground
    • Dark Bold
    • Light Bold
    • Light Links
  • Should Cursor guide have alpha? I think opaque color is better

@cocopon
Copy link
Owner

cocopon commented Jan 26, 2021

Feel free to reopen the issue if you have any updates.

@cocopon cocopon closed this Jan 26, 2021
@epszaw
Copy link
Author

epszaw commented Jan 26, 2021

@cocopon yeah, when I'll done it – I write you and reopen the issue

@epszaw
Copy link
Author

epszaw commented Jan 28, 2021

@cocopon seems like I fix here everything. I hope, that's really everything now.

@epszaw
Copy link
Author

epszaw commented Jan 28, 2021

So, I can't reopen this PR. Should I create new one?

@cocopon cocopon reopened this Jan 28, 2021
@epszaw
Copy link
Author

epszaw commented Feb 18, 2021

@cocopon can you review the last changes?

@jf jf mentioned this pull request May 21, 2023
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.

None yet

3 participants