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

Add Facebook and Twitter icons to the toolbar #3636

Merged
merged 1 commit into from
Jan 16, 2018
Merged

Add Facebook and Twitter icons to the toolbar #3636

merged 1 commit into from
Jan 16, 2018

Conversation

koppor
Copy link
Member

@koppor koppor commented Jan 15, 2018

Before:

before

After:

after

  • Also add entry "Twitter" to "JabRef resources"
  • Put "Opens JabRef's X page" together in JabRef_*.properties

Does not connect to Twitter directly, thus no spying by Twitter, Facebook, ... Refs https://www.heise.de/newsticker/meldung/Datenschutz-und-Social-Media-Der-c-t-Shariff-ist-im-Einsatz-2470103.html


  • 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?)
  • If you changed the localization: Did you run gradle localizationUpdate?

@koppor koppor added the status: ready-for-review Pull Requests that are ready to be reviewed by the maintainers label Jan 15, 2018
Copy link
Member

@LinusDietz LinusDietz left a comment

Choose a reason for hiding this comment

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

One typo, can be merged afterwards.

@@ -204,7 +207,7 @@
Localization.lang("See what has been changed in the JabRef versions"));
private final AbstractAction forkMeOnGitHubAction = new OpenBrowserAction("https://github.com/JabRef/jabref",
Localization.menuTitle("Fork me on GitHub"), Localization.lang("Opens JabRef's GitHub page"), IconTheme.JabRefIcon.GITHUB.getSmallIcon(), IconTheme.JabRefIcon.GITHUB.getIcon());
private final AbstractAction donationAction = new OpenBrowserAction("https://github.com/JabRef/jabref/wiki/Donations",
private final AbstractAction donationAction = new OpenBrowserAction("https://dontaions.jabref.org",
Copy link
Member

Choose a reason for hiding this comment

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

- Also add to "JabRef resources"
- Put "Opens JabRef's X page" together in JabRef_*.properties
@koppor koppor merged commit b00f907 into master Jan 16, 2018
@koppor koppor deleted the twitter branch January 16, 2018 14:34
@koppor koppor removed the status: ready-for-review Pull Requests that are ready to be reviewed by the maintainers label Feb 20, 2018
@koppor koppor mentioned this pull request Feb 20, 2018
5 tasks
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.

2 participants