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 test for ordinals #2752

Merged
merged 2 commits into from
Apr 17, 2017
Merged

Add test for ordinals #2752

merged 2 commits into from
Apr 17, 2017

Conversation

koppor
Copy link
Member

@koppor koppor commented Apr 16, 2017

This is a follow up for #2596 and #2601. I'd like to test how I can reproduce the behavior of ordinals.

Can someone explain me, why I don't get Unicode as result here?

@LinusDietz
Copy link
Member

I have fixed the ordinals, with the side-effect that e.g., italics are now also converted into unicode. What do you think about that?

Copy link
Member

@tobiasdiez tobiasdiez left a comment

Choose a reason for hiding this comment

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

I think it's a nice additional touch that JabRef understands textit.

@koppor
Copy link
Member Author

koppor commented Apr 17, 2017

Refs #160

I think, I begin to like it, because the display of JabRef in the UI will improve. As stated in #2601 (comment), I will step away from biblatex in papers and so, we can move on here.

@lenhard
Copy link
Member

lenhard commented Jul 13, 2017

As indicated in #2784, this breaks support for italics in the main table, which was working before. We should revert it and find another way to support ordinals.

@LinusDietz
Copy link
Member

LinusDietz commented Jul 13, 2017

Hi @lenhard, the problem is you always loose something.
The key for the conversions lies in the Normalizer settings of the LaTexToUnicodeConverter#24

I have made this handy chart of the four options and what works when (on my linux system).

Ordinals textit Special Chars, eg. polish names, acute, umlauts
NFD Yes Yes No
NKC Yes Yes Yes
NFKD No No No
NFKC No No Yes

In conclusion I would suggest to investigate what factors influence the bug that so far was only seen on @AEgit's system.

@AEgit
Copy link

AEgit commented Jul 13, 2017

@lynyus : Let me know which additional configuration details are needed. I still encounter the bug in:
JabRef 4.0.0-dev--snapshot--2017-07-10--master--e80f450ee
Windows 10 10.0 amd64
Java 1.8.0_131

@LinusDietz
Copy link
Member

LinusDietz commented Jul 13, 2017

@AEgit the problem is: I have no idea what could help us :/

@LinusDietz
Copy link
Member

One solution might be to convert italics back into regular ascii after the latex2unicode conversion?

@AEgit
Copy link

AEgit commented Jul 13, 2017

Hmmm, ok - am I really the only one with a Windows system, who encounters this problem? Because if that's the case, it might be my/my system's fault. But as far as I can tell @Siedlerchr confirmed the issue on his Windows 10 installation (#2784). Therefore it sounds like a Windows issue. What about the Mac users?

@LinusDietz
Copy link
Member

Okay, I'm on my Windows now and I have these problems. Somehow I must have overread @Siedlerchr's comment or misinterpreted it :/ All the time I was assuming that I had checked it successfully on my windows machine.

Mea culpa, I'll be working on a fix now.

@AEgit
Copy link

AEgit commented Jul 13, 2017

No worries, and thank you for your help!

@lenhard
Copy link
Member

lenhard commented Jul 13, 2017

@lynyus Nice table, thanks for the overview :) Sure, NKC is the right option here. The problem is that we're using a Swing table as maintable and as far as I've found it just refuses to work with unicode italics (unless you have appropriate fonts installed in your system, but thats not the default). The best solution would be to get rid of the maintable in our move to JavaFX. Let's do that in the future, but let's not do it right now, because we should rather build a stable entry editor.

I'll suggest something different: @AEgit How important is it really that the text in the main table is in italics? Would it be acceptable if the text would just be displayed as plain text? Because in that case, we could just eliminate the \textit command before we do the unicode conversion.

@AEgit
Copy link

AEgit commented Jul 13, 2017

@lenhard To be honest, I'm not really bothered by the fact that the text in the main table is not in italics. As long as the entry preview shows it in italics (which it currently does), I'm pretty happy with it. Having the proper format in the main table as well, would be nice, but it is not a show-stopper for me at the moment.

@LinusDietz
Copy link
Member

LinusDietz commented Jul 13, 2017

I have a PR now on this: #3008

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.

5 participants