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

Remove JavaFX-Font-Tweak #5330

Merged
merged 3 commits into from
Oct 14, 2019
Merged

Conversation

calixtus
Copy link
Member

As asked by @tobiasdiez in the gitter-chat, a seperate PR to test if the font tweak can be safely removed, as probably the dependencies were removed in JavaFX anyways. See https://bugs.java.com/bugdatabase/view_bug.do?bug_id=8187147


  • Change in CHANGELOG.md described
  • Tests created for changes
  • Manually tested changed features in running JabRef
  • Screenshots added in PR description (for bigger UI changes)
  • Ensured that the git commit message is a good one
  • Check documentation status (Issue created for outdated help page at help.jabref.org?)

@tobiasdiez
Copy link
Member

As the original author of #3305, @halirutan could you please check if this font tweak is still necessary? Thanks a lot!

@halirutan
Copy link
Collaborator

@tobiasdiez Yes I can, but I'm not at home on my Linux mache until the weekend. But then, I can have a look.

@Siedlerchr Siedlerchr added the status: waiting-for-feedback The submitter or other users need to provide more information about the issue label Sep 18, 2019
@tobiasdiez
Copy link
Member

@halirutan Have you already found the time to look at this?

@halirutan
Copy link
Collaborator

@tobiasdiez I looked at this tonight and I cannot see a difference between the tweak turned on or off. So I guess it can be removed.

@tobiasdiez
Copy link
Member

Thanks for having a look at this @halirutan

@tobiasdiez tobiasdiez removed the status: waiting-for-feedback The submitter or other users need to provide more information about the issue label Oct 13, 2019
@Siedlerchr Siedlerchr merged commit 824b103 into JabRef:master Oct 14, 2019
calixtus added a commit to calixtus/jabref that referenced this pull request Oct 14, 2019
@calixtus calixtus deleted the remove_javafx_fontfix branch October 14, 2019 11:38
tobiasdiez pushed a commit that referenced this pull request Oct 15, 2019
* Initial

* Fix special key, refactor visual appearance, l10n

* Fixed Indent and wrong value set

* Refurbished css

* Added BibtexKeyPatternTable, fixed bugs

* Fixed bugs in reset, added reset all

* Refactored by PR-suggestions, added jumpToPressedKey, highlite default
column, set defaultItem to top of list,

* l10n

* Changelog

* Fixed pattern saving, converted default to property

* Changed comments

* Added help button

* Removed fontTweaksLinux according to PR #5330

* l10n

* Changed font size limits
koppor pushed a commit that referenced this pull request Mar 29, 2021
e1acabe Upload Styles (#5351)
d9cbf08 Update and rename journal-of-eta-maritime-science.csl to dependent/journal-of-eta-maritime-science.csl (#5353)
988ecca Update clinical-and-translational-immunology.csl (#5348)
1691078 Update molecular-plant-pathology.csl (#5349)
932a5bf Create neurospine.csl (#5334)
d663ffc Update sylwan.csl (#5341)
b7be05d Update pakistan-journal-of-agricultural-sciences.csl (#5343)
d742832 Please rename to Tijdschrift-voor-geneeskunde-en-gezondheidszorg (#5344)
6c6be4d Update et-al settings for journal-of-the-american-society-of-nephrology.csl (#5333)
0263204 Correct citation format of numeric AOSIS styles (#5335)
e4cd348 Create the-horticulture-journal.csl (#5327)
5804a6f Add AOSIS dependents (#5332)
2b4510b Create natura-croatia.csl (#5328)
03c6789 Create groundwater.csl (#5329)
e08fa09 Create hystrix-the-italian-journal-of-mammalogy.csl (#5330)
1d730aa update page-range-format for Anglia (#5331)
9bb8724 Sort American Physiological Society citation style according to updated author instructions (#5305)
a8b53ca Create african-online-scientific-information-systems-vancouver.csl (#5326)
c7d8f90 Update yozgat-bozok-universitesi-fen-bilimleri-enstitusu.csl (#5319)
87dc0eb Update and rename tijdschrift-voor-geneeskunde.csl to tijdschrift-voo… (#5325)
89e07b0 Create university-of-gothenburg-apa7-swedish-legislations (#5316)
4582cb6 Create chinese-law-journals.csl (#5314)
6412083 Create stellenbosch-law-review.csl (#5322)

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