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

allow terminal256 formatter to show italics... #1288

Merged
merged 1 commit into from
Dec 1, 2019

Conversation

birkenfeld
Copy link
Member

Ported from https://bitbucket.org/birkenfeld/pygments-main/pull-requests/711

OSX users will have to follow instructions at: http://www.eddieantonio.ca/blog/2015/04/16/iterm-italics/

tldr;
infocmp xterm-256color > /tmp/xterm-256color.terminfo
echo 'sitm=\E[3m, ritm=\E[23m,' >> /tmp/xterm-256color.terminfo
tic /tmp/xterm-256color.terminfo

make sure iTerm2.settings.profiles.terminal.terminal type=xterm-256color

OSX users will have to follow instructions at: http://www.eddieantonio.ca/blog/2015/04/16/iterm-italics/

tldr;
infocmp xterm-256color > /tmp/xterm-256color.terminfo
echo 'sitm=\E[3m, ritm=\E[23m,' >> /tmp/xterm-256color.terminfo
tic /tmp/xterm-256color.terminfo

make sure iTerm2.settings.profiles.terminal.terminal type=xterm-256color
Copy link
Collaborator

@Anteru Anteru left a comment

Choose a reason for hiding this comment

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

I'm fine with the change as-is, but I'm worried about the OS X instructions as the link on how to set it up does 404 these days. I guess we'll learn soon enough if that is a problem on OS X for real.

@birkenfeld
Copy link
Member Author

A google search finds a stackoverflow article, that should be ok. In any case, italic support in terminals is not universal anyway.

@birkenfeld birkenfeld merged commit 20f903d into master Dec 1, 2019
@Anteru Anteru deleted the 711/kneufeld/terminal256-italics branch February 9, 2020 09:12
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