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

Rendering/conversion of π in Abstract/preview panel #7291

Closed
evanhorn opened this issue Jan 4, 2021 · 3 comments · Fixed by #7296
Closed

Rendering/conversion of π in Abstract/preview panel #7291

evanhorn opened this issue Jan 4, 2021 · 3 comments · Fixed by #7296

Comments

@evanhorn
Copy link

evanhorn commented Jan 4, 2021

JabRef version on
JabRef 5.3--2021-01-04--eff250b
Linux 5.4.0-58-generic amd64
Java 15.0.1

In the abstract and comment fields, the LaTeX command $\pi$ renders "pi" rather than the expected "π" (lower case pi) in the preview panel. However, the LaTeX form of all other Greek letters, including Π (upper case pi) renders correctly in the preview panel. If "π" is present in the abstract field and Convert -> "Unicode to LaTeX" is used, "π" is converted to "$\phi$" and is rendered as "φ" in the preview panel.

Note: While I tested the rendering of all LaTeX Greek letters I did not test the conversion of all Greek letters from Unicode to LaTeX

Steps to reproduce the behavior:

  1. Enter "$\pi$ in Abstract field.
  2. Observe "pi" in preview panel.

Steps to reproduce the behavior:

  1. Paste π into Abstract field.
  2. Select Convert -> "Unicode to LaTeX" in context menu
  3. Observe "$\phi$" in Abstract field.
  4. Observe "φ" in preview panel.
@Siedlerchr
Copy link
Member

Thanks for reporting that issue. Thanks to @k3KAW8Pnf7mkmdSMPHz27 it's now fixed in the latest development version (available in approx 30 minutes)

@evanhorn
Copy link
Author

evanhorn commented Jan 4, 2021

Thanks for the quick response, this fixed the issue!

@k3KAW8Pnf7mkmdSMPHz27
Copy link
Member

Thank you for the detailed description, it makes it easier to narrow things down 😉

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 a pull request may close this issue.

3 participants