Skip to content

Allow basic markup syntax custom previews#6232

Merged
koppor merged 9 commits into
JabRef:masterfrom
AlexanderGirgis:allow-basic-markup-in-comment-fixes-6194
Apr 3, 2020
Merged

Allow basic markup syntax custom previews#6232
koppor merged 9 commits into
JabRef:masterfrom
AlexanderGirgis:allow-basic-markup-in-comment-fixes-6194

Conversation

@AlexanderGirgis

@AlexanderGirgis AlexanderGirgis commented Apr 1, 2020

Copy link
Copy Markdown
Contributor

Add MarkdownFormatter using https://github.com/vsch/flexmark-java/ to format markdown.
To configure Markdown in custom previews add the "Markdown" formatter.
Markdown is enabled by default for the comment field as requested in #6194

Fixes #6194

image

  • Change in CHANGELOG.md described (if applicable)
  • Tests created for changes (if applicable)
  • Manually tested changed features in running JabRef (always required)
  • Screenshots added in PR description (for bigger UI changes)
  • Checked documentation: Is the information available and up to date? If not: Issue created at https://github.com/JabRef/user-documentation/issues.

Add MarkdownFormatter using https://github.com/vsch/flexmark-java/ to format markdown.
To configure Markdown in custom previews add the "Markdown" formatter.
Markdown is enabled by default for the comment field as requested in JabRef#6194
…urnsOnlyOneLine to actually test newlines only because the rest is covered elsewhere
@koppor koppor added the status: ready-for-review Pull Requests that are ready to be reviewed by the maintainers label Apr 1, 2020

@tobiasdiez tobiasdiez left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Welcome to the world of open-source! I'm impressed by your first PR. The code looks mostly good to me and I've only a few minor comments.

One bigger suggestion: Right now the article description (title author etc) takes quite some space in the preview. I would thus suggest to replace the article preview in the comment tab by a html rendering of only the content of the comment field.

Comment thread src/main/java/module-info.java Outdated
Comment thread src/main/java/org/jabref/logic/layout/format/MarkdownFormatter.java
Comment thread src/main/java/org/jabref/migrations/PreferencesMigrations.java Outdated
@koppor koppor removed the status: ready-for-review Pull Requests that are ready to be reviewed by the maintainers label Apr 2, 2020
@koppor

koppor commented Apr 3, 2020

Copy link
Copy Markdown
Member

Needed to merge master to sync the PR with your state @AlexanderGirgis. Refs https://twitter.com/githubstatus/status/1245817118542553089.

Could you fix CheckStyle?

@koppor koppor merged commit d4e46b0 into JabRef:master Apr 3, 2020
koppor pushed a commit that referenced this pull request Oct 1, 2022
7bde3e4 Add style for the Geographical Analysis journal (#6145)
6fa1551 Create taylor-and-francis-chicago-b-author-date.csl (#6232)
eba2e8c Create taylor-and-francis-ama.csl (#6221)
dda9d57 ACS, AMA, Vancouver: Remove hardcoded space after `citation-number` (#6248)
8f5fe92 GitHub Workflows security hardening (#6246)
284bc10 Create angiology.csl (#6122)
eb141cc Update society-of-biblical-literature-fullnote-bibliography.csl (#6157)
dddb459 Rewrite law-citation-manual.csl (#6171) (#6171)
b975c96 Update Cell to numeric-superscript style (#6245)
3a41b08 Create isara-iso-690.csl (#6201)
5a128fe Create Biomembranes.csl (#6200)
da2e0c0 Capitalize-first short titles for legislation ("Statute"). (#6241)
af7f08d Create proceedings-of-the-estonian-academy-of-sciences-author-date.csl (#6209)

git-subtree-dir: buildres/csl/csl-styles
git-subtree-split: 7bde3e4
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.

Allow basic markup syntax in the comments tab

3 participants