Skip to content

Fix of BstPreviewLayout#11343

Closed
koppor wants to merge 4 commits into
mainfrom
fix-11338
Closed

Fix of BstPreviewLayout#11343
koppor wants to merge 4 commits into
mainfrom
fix-11338

Conversation

@koppor

@koppor koppor commented May 27, 2024

Copy link
Copy Markdown
Member

Fixes #11338

Open points

  • Let SnuggleTeX NOT fail on error, but "just" continue (we can accept lost characters)
  • Adapt tests to check for .body().text() and not for complete HTML
  • Currently, NBSP appears when ~ is rendered. Strange.

Some things need to be fixed upstream: davemckain/snuggletex#7

Mandatory checks

  • Change in CHANGELOG.md described in a way that is understandable for the average user (if applicable)
  • Tests created for changes (if applicable)
  • Manually tested changed features in running JabRef (always required)
  • Screenshots added in PR description (for UI changes)
  • Checked developer's documentation: Is the information available and up to date? If not, I outlined it in this pull request.
  • Checked documentation: Is the information available and up to date? If not, I created an issue at https://github.com/JabRef/user-documentation/issues or, even better, I submitted a pull request to the documentation repository.

koppor and others added 4 commits May 27, 2024 22:34
Co-authored-by: Christoph <siedlerkiller@gmail.com>
Co-authored-by: Carl Christian Snethlage <50491877+calixtus@users.noreply.github.com>
Co-authored-by: Carl Christian Snethlage <50491877+calixtus@users.noreply.github.com>
Co-authored-by: Carl Christian Snethlage <50491877+calixtus@users.noreply.github.com>
BibEntry testEntry = new BibEntry(StandardEntryType.InProceedings)
.withCitationKey("DBLP:conf/iscas/SadeghifarWG14")
// .withField(StandardField.AUTHOR, "Mohammad Reza Sadeghifar and J. Jacob Wikner and Oscar Gustafsson")
//.withField(StandardField.TITLE, "Linear programming design of semi-digital {FIR} filter and {\\(\\Sigma\\)}{\\(\\Delta\\)} modulator for {VDSL2} transmitter")

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

🚫 [reviewdog] reported by reviewdog 🐶
Comment text should start with space.

// .withField(StandardField.TIMESTAMP, "Sat, 05 Sep 2020 18:07:30 +0200")
// .withField(new UnknownField("biburl"), "https://dblp.org/rec/conf/iscas/SadeghifarWG14.bib")
// .withField(new UnknownField("bibsource"), "dblp computer science bibliography, https://dblp.org");
;

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

🚫 [reviewdog] <com.puppycrawl.tools.checkstyle.checks.whitespace.NoWhitespaceBeforeCheck> reported by reviewdog 🐶
';' is preceded with whitespace.

// .withField(StandardField.TIMESTAMP, "Sat, 05 Sep 2020 18:07:30 +0200")
// .withField(new UnknownField("biburl"), "https://dblp.org/rec/conf/iscas/SadeghifarWG14.bib")
// .withField(new UnknownField("bibsource"), "dblp computer science bibliography, https://dblp.org");
;

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

🚫 [reviewdog] <com.puppycrawl.tools.checkstyle.checks.whitespace.SeparatorWrapCheck> reported by reviewdog 🐶
';' should be on the previous line.

@github-actions

Copy link
Copy Markdown
Contributor

The build of this PR is available at https://builds.jabref.org/pull/11343/merge.

@Siedlerchr

Copy link
Copy Markdown
Member

To reduce the number of open PRs I will cose this due to inactivity

@Siedlerchr Siedlerchr closed this Oct 20, 2024
@koppor

koppor commented Oct 20, 2024

Copy link
Copy Markdown
Member Author

Ported to JabRef#698

@koppor koppor deleted the fix-11338 branch October 20, 2024 18:47
@koppor

koppor commented Oct 20, 2024

Copy link
Copy Markdown
Member Author

I deleted the branch, too - to reduce the number of branches 😅.

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.

Fix BST parsing for latex math symbols

2 participants