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

Fix for issue 7416: font size of the preferences dialog does not update with the rest of the GUI. #7509

Merged
merged 11 commits into from
Mar 27, 2021

Conversation

XDZhelheim
Copy link
Contributor

Fix #7416

The font size of preferences dialog does not update after changing the main font size. I think this problem is about the style settings of the dialog. Therefore, in the constructor of PreferencesDialogView class, I assigned the main font size to the font size of the dialog via setStyle().

  • Change in CHANGELOG.md described in a way that is understandable for the average user (if applicable)
  • Tests created for changes (if applicable)
  • Manually tested changed features in running JabRef (always required)
  • Screenshots added in PR description (for UI changes)
  • Checked documentation: Is the information available and up to date? If not created an issue at https://github.com/JabRef/user-documentation/issues or, even better, submitted a pull request to the documentation repository.

fix7416

Copy link
Member

@Siedlerchr Siedlerchr left a comment

Choose a reason for hiding this comment

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

Thanks for your contribution an the fix!

@Siedlerchr Siedlerchr added the status: ready-for-review Pull Requests that are ready to be reviewed by the maintainers label Mar 9, 2021
@Siedlerchr Siedlerchr requested a review from calixtus March 9, 2021 16:40
@Landi29
Copy link
Contributor

Landi29 commented Mar 11, 2021

@XDZhelheim Thank you for the fix. I was unaware that I could use preferencesService.getAppearancePreferences().getMainFontSize(). This should fix the problem. It seems that the CHANGELOG file has some conflicts that must be resolved.

@XDZhelheim
Copy link
Contributor Author

Thanks for your acknowledgement and notice! I made some mistakes on my IDE's auto-formatting, however, it seems there is no conflict now.

@calixtus
Copy link
Member

Hi, thanks for the effort you put in this fix here. As this PR may be conflicting with a huge PR @docrjp is preparing in #7336 I would like to ask you to be patient and wait for his PR to be merged, before we review and merge this one.

@XDZhelheim
Copy link
Contributor Author

I'm OK, take your time : )

Copy link
Member

@calixtus calixtus left a comment

Choose a reason for hiding this comment

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

Since the other PR seems to take some more time, I'll approve this and think we should merge these changes to be ready for the next release.

@Siedlerchr Siedlerchr merged commit 1d1156f into JabRef:master Mar 27, 2021
Siedlerchr added a commit that referenced this pull request Mar 28, 2021
* upstream/master: (191 commits)
  Fix for issue 7416: font size of the preferences dialog does not update with the rest of the GUI. (#7509)
  Fix school/instituation is printed twice (#7574)
  Dsiable notarisation until we hae an account for JabRef e.V. (#7572)
  Fix citation keys unintentionally being overwritten on import (#7443)
  Fix AuthentificationPlugin not declared in mergedModule (#7570)
  Suggestions for changes in caching latex free authors (#7301)
  Add simple Unit Tests (#7542)
  Fix drag and drop into empty library (#7555)
  Bump richtextfx from 0.10.4 to 0.10.6 (#7563)
  Bump pdfbox from 2.0.22 to 2.0.23 (#7561)
  Bump org.eclipse.jgit (#7560)
  Bump fontbox from 2.0.22 to 2.0.23 (#7562)
  Bump guava from 30.1-jre to 30.1.1-jre (#7564)
  Bump xmpbox from 2.0.22 to 2.0.23 (#7565)
  Bump hmarr/auto-approve-action from v2.0.0 to v2.1.0 (#7566)
  Add gource (#7193)
  UI: Fix for group icon (#7552)
  Fix for issue 6487: Opening BibTex file (doubleclick) from Folder with spaces not working (#7551)
  add ability to insert arxivId (#7549)
  Fixed missing trigger for linked file operations (#7548)
  ...
Siedlerchr added a commit that referenced this pull request Mar 29, 2021
* upstream/master:
  Bump classgraph from 4.8.102 to 4.8.104 (#7587)
  Bump checkstyle from 8.41 to 8.41.1 (#7586)
  Bump org.beryx.jlink from 2.23.3 to 2.23.5 (#7588)
  Revert "Re-add LibericaJDK"
  Re-add LibericaJDK
  Update gradle from 6.8 to 6.8.3 (#7583)
  Fix for issue 7416: font size of the preferences dialog does not update with the rest of the GUI. (#7509)
  Fix school/instituation is printed twice (#7574)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
status: ready-for-review Pull Requests that are ready to be reviewed by the maintainers
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Font size of the preferences dialog does not update with the rest of the GUI.
4 participants