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

Citation Styles that should be cross-checked in the code #11

Closed
2 of 4 tasks
ThiloteE opened this issue Jul 19, 2024 · 6 comments
Closed
2 of 4 tasks

Citation Styles that should be cross-checked in the code #11

ThiloteE opened this issue Jul 19, 2024 · 6 comments
Labels
bug Something isn't working Priority: low wontfix This will not be worked on

Comments

@ThiloteE
Copy link
Collaborator

ThiloteE commented Jul 19, 2024

JabRef version

Latest development branch build (please note build date below)

Operating system

Windows

Details on version and operating system

Windows 10

Checked with the latest development build (copy version output from About dialog)

  • I made a backup of my libraries before testing the latest development version.
  • I have tested the latest development version and the problem persists

Steps to reproduce the behaviour

This is a living issue. Can be edited and updated on the fly.
These are styles that differ between entry preview and cited in Libreoffice. I started at History of the Human Sciences and am working my way up the list.

  • Histoire & Mesure (Français)

      Smith, Bill, Bob Jones, & Jeff Williams, “Title of the test entry,” BibTeX Journal, 2016, vol. 34, no. 3, pp. 45–67.
    

    the space after "vol."

  • Histoire@Politique. Politique, culture, société (Français)

      Smith Bill, Jones Bob and Williams Jeff, “Title of the test entry,” BibTeX Journal, vol. 34, no. 3, July 2016, pp. 45–67.
    

    the space after "vol."

Appendix

No response

@ThiloteE ThiloteE added the bug Something isn't working label Jul 19, 2024
@subhramit
Copy link
Owner

Expected:
image
image
Got:
image

Seems fine to me, there is a space expected after vol. in the input HTML as well. Could you confirm if this is what you were talking about?

@ThiloteE
Copy link
Collaborator Author

ThiloteE commented Jul 21, 2024

Yes, I know it looks similar to human eye, but try with copy and then ctrl + f. It doesn't match.

@subhramit
Copy link
Owner

subhramit commented Jul 21, 2024

Yes, I know it looks similar to human eye, but try with copy and then ctrl + f. It doesn't match.

Oh, that has a different implication, actually.
Remember we were trying to copy the preview from the select styles dialog, and on pasting as plain text, the text changed?

It's probably a difference in how text is formatted and parsed when produced in an OO document vs in the Preview.

For instance, take a look at the Histoire@Politique (the second style you mentioned in the issue):
image
The one on the left is copy pasted from what's rendered in OO, the one on the right is copy pasted from the preview.
Diff checker suggests that there is some difference in the entry," BibTeX part, and not at the vol. 34.

My guess: When we copy paste entries from the preview, the format is not parsed the same way as it was being done in the preview (here, "BibTeX" was supposed to be in italics, as it appears in the preview, but on copy-paste, that's lost).
If you copy to the OO doc as well, the italics go away:
image
Above: Rendered by cite, below: copy-pasted from Preview.
Similarly, when I copy pasted both in diff-checker, both lost the italics formatting.

Another example: what's rendered as a normal space in the preview may be rendered as a non-breaking space (&nbsp in escaped HTML) in OO, hence even if they look same to the eye, ctrl+f won't be able to match them. It means that the preview generator's processor parsed &nbsp but converted/produced it as a normal space.

@ThiloteE ThiloteE added depends-on-external Issues that we cannot easily fix on our own. wontfix This will not be worked on and removed depends-on-external Issues that we cannot easily fix on our own. labels Jul 22, 2024
@ThiloteE
Copy link
Collaborator Author

Won't fix for now.

@subhramit
Copy link
Owner

subhramit commented Jul 22, 2024

Update: Not an issue from our side, we are rendering citations as per the specification format. Implications:

  1. Ctrl+F won't always work with keyboard-typed special characters e.g. spaces (what we might be looking for may be a non-breaking space, which we rendered into the document correctly as per the specification).
  2. We are unsure how Preview is rendered, and how it interprets the escaped HTML characters. This may cause disparity between the actual citation and the preview, although not to the human eye. (Issue with the preview viewer, not CSL4LibreOffice).
    Ref. Citation Styles have new line after starting numeric in Entry Preview. Differs from OO Cite. #10

@subhramit
Copy link
Owner

I am closing this issue, as it is not a definite issue here.
Superseded by #18 and #13

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working Priority: low wontfix This will not be worked on
Projects
None yet
Development

No branches or pull requests

2 participants