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

Redesigned about dialog in JavaFX #1947

Merged
merged 3 commits into from
Sep 12, 2016

Conversation

boceckts
Copy link
Contributor

As discussed in #968 the about dialog is still not satisfying and therefore I redesigned it. The new design is a more flat like design with all important information in one place. The version still clickable and copies the version information to the clipboard and the logo is a hyperlink to www.jabref.org.
I also changed the copied version string to include JabRef, Java and OS version so it is the same as in the master branch.

screen shot 2016-09-10 at 14 59 58

  • Change in CHANGELOG.md described
  • Tests created for changes
  • Screenshots added (for bigger UI changes)
  • Manually tested changed features in running JabRef
  • Check documentation status (Issue created for outdated help page at help.jabref.org?)

@tschechlovdev tschechlovdev added status: ready-for-review Pull Requests that are ready to be reviewed by the maintainers and removed stupro-ready-for-internal-review labels Sep 11, 2016
@boceckts boceckts changed the title [WIP] Redesigned about dialog in JavaFX Redesigned about dialog in JavaFX Sep 11, 2016
private final String LIBRARIES = "https://github.com/JabRef/jabref/blob/master/external-libraries.txt";
private final String GITHUB = "https://github.com/JabRef/jabref";
private final String CHANGELOG = "https://github.com/JabRef/jabref/blob/master/CHANGELOG.md";
private final String LICENSE = "https://opensource.org/licenses/mit-license.php";
Copy link
Member

Choose a reason for hiding this comment

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

@tobiasdiez
Copy link
Member

LGTM, just change the link to the license and I think this can be merged.
@koppor please check that the new library is included in the correct way. Thanks

@koppor
Copy link
Member

koppor commented Sep 12, 2016

Please merge javafx and add new screenshot.

Id: de.codecentric.centerdevice
Project: javafxsvg
URL: https://github.com/codecentric/javafxsvg
License: BSD 3-Clause
Copy link
Member

Choose a reason for hiding this comment

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

👍

@boceckts
Copy link
Contributor Author

@koppor Merged javafx, why do you want a new screenshot? The layout hasn't changed, only difference is that there is a small JabRef icon in the title bar of the dialog.

@tobiasdiez tobiasdiez merged commit bd3dbe8 into JabRef:javafx Sep 12, 2016
private final String DONATION = "http://www.jabref.org/#donations";
private final String LIBRARIES = "https://github.com/JabRef/jabref/blob/master/external-libraries.txt";
private final String GITHUB = "https://github.com/JabRef/jabref";
private final String CHANGELOG = "https://github.com/JabRef/jabref/blob/master/CHANGELOG.md";
Copy link
Member

@koppor koppor Sep 12, 2016

Choose a reason for hiding this comment

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

Can we add a deep link here?

I would suggest https://github.com/koppor/keep-a-changelog/blob/master/CHANGELOG.md#unreleased for the development versions and add a release TODO to replace this by something like https://github.com/JabRef/jabref/blob/master/CHANGELOG.md#36---2016-08-26 follow up at #1959

@koppor
Copy link
Member

koppor commented Sep 12, 2016

@boceckts I want to double check that the fix of #1954 also works here. I made a screenshot for myself at #1959 to be able to review the current status.

@koppor koppor removed the status: ready-for-review Pull Requests that are ready to be reviewed by the maintainers label Sep 12, 2016
@koppor koppor mentioned this pull request Sep 12, 2016
5 tasks
@boceckts boceckts deleted the RedesignAboutDialog branch September 13, 2016 11:53
@koppor
Copy link
Member

koppor commented Sep 28, 2016

The about dialog is not centered on the screen. Does anyone else experience such an issue?

@boceckts
Copy link
Contributor Author

I think it is centered on the main frame, not the screen.

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