Skip to content

Overhaul Entry Editor Preview (#7093)#329

Merged
Siedlerchr merged 1 commit into
JabRef:masterfrom
AtrusRiven:Overhaul-entry-preview-(#7093)
Nov 22, 2020
Merged

Overhaul Entry Editor Preview (#7093)#329
Siedlerchr merged 1 commit into
JabRef:masterfrom
AtrusRiven:Overhaul-entry-preview-(#7093)

Conversation

@AtrusRiven

@AtrusRiven AtrusRiven commented Nov 13, 2020

Copy link
Copy Markdown

Entry Editor Preview overhauled, added screenshots etc.

@calixtus calixtus 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.

LGTM, but we should keep in mind, that in the mid future we try to automate the process of taking screenshots with TestFX. @koppor knows more? But for now it's good.
Thanks.

@AtrusRiven

Copy link
Copy Markdown
Author

Why LGTM?

@calixtus

Copy link
Copy Markdown
Member

LGTM = "Looks good to me"

@AtrusRiven

AtrusRiven commented Nov 19, 2020

Copy link
Copy Markdown
Author

Interesting, known to me as "Literally Garbage To Me" ;-)

@Siedlerchr Siedlerchr merged commit f4b1fe4 into JabRef:master Nov 22, 2020
@koppor

koppor commented Nov 22, 2020

Copy link
Copy Markdown
Member

Automated screenshots: See JabRef/jabref#6089 for code and discussions. Especially JabRef/jabref#6089 (comment)

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.

4 participants