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

Explore ways to avoid ignoring line-height from custom pygments themes #720

Open
pradyunsg opened this issue Sep 9, 2023 Discussed in #438 · 0 comments
Open

Explore ways to avoid ignoring line-height from custom pygments themes #720

pradyunsg opened this issue Sep 9, 2023 Discussed in #438 · 0 comments
Labels
enhancement New feature or request

Comments

@pradyunsg
Copy link
Owner

Discussed in #438

Originally posted by asmeurer May 11, 2022
In our custom pygments theme for SymPy, we set the line-height attribute using _pre_style: https://github.com/sympy/sympy/blob/68c37df362e8585d72fc7ef490013bb8eff16e3e/doc/src/_pygments/styles.py#L25-L29

However, this is being ignored by the docs
Screen Shot 2022-05-10 at 5 58 33 PM

We need this so that Unicode pretty printed output doesn't add extra space between lines. See for example https://docs.sympy.org/dev/tutorial/printing.html#unicode-pretty-printer and compare to our previous theme which set this attribute correctly https://docs.sympy.org/latest/tutorial/printing.html#unicode-pretty-printer

I can work around this by adding it to the custom.css, but it seems preferable to include it in the pygments theme since pygments allows this.

@pradyunsg pradyunsg added the enhancement New feature or request label Sep 9, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

No branches or pull requests

1 participant