Skip to content

Modifications to improve contrast of UI elements#4758

Merged
tobiasdiez merged 2 commits into
JabRef:masterfrom
samiyac:fix-for-issue-4583
Mar 14, 2019
Merged

Modifications to improve contrast of UI elements#4758
tobiasdiez merged 2 commits into
JabRef:masterfrom
samiyac:fix-for-issue-4583

Conversation

@samiyac

@samiyac samiyac commented Mar 12, 2019

Copy link
Copy Markdown
Contributor

Fixes #4583

modification


  • 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 tobiasdiez 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.

Thanks for your PR. The changes look good to me.

@Siedlerchr

Copy link
Copy Markdown
Member

LGTM! Thanks for the contribution! @tobiasdiez Do we need to adjust the dark theme as well?

@Siedlerchr Siedlerchr added the status: ready-for-review Pull Requests that are ready to be reviewed by the maintainers label Mar 14, 2019
@tobiasdiez

Copy link
Copy Markdown
Member

I think in the dark theme everything is dark, so need for action ;-).

@tobiasdiez tobiasdiez merged commit bd5b67b into JabRef:master Mar 14, 2019
@samiyac samiyac deleted the fix-for-issue-4583 branch March 15, 2019 14:05
Siedlerchr added a commit that referenced this pull request Mar 16, 2019
…erbibtexkey

* upstream/master: (827 commits)
  [#4306] Disable renaming (#4727)
  Feature: implement search filter in show preferences (#4759)
  Enable import button only if entry selected (#4761)
  Improve Remote messaging (#4760)
  Add changelog entry for #4520
  Modifications to improve contrast of UI elements (#4758)
  Rework external changes dialog in JavaFX (#4693)
  Changes the title of Group Dialog to Add subgroup when adding a new subgroup (#4757)
  Optimize data fetching (#4520)
  Adds a browse button next to the path text field for aux-based groups (#4743)
  Saving changes and exiting application (#4729)
  Code optimized and created a reusable method to get writer output (#4750)
  Bump mvvmfx from 1.7.0 to 1.8.0 (#4747)
  Bump guava from 27.0.1-jre to 27.1-jre (#4748)
  Bump mvvmfx-validation from 1.7.0 to 1.8.0 (#4749)
  Remove obsolete swing components in Preferences and OpenOffice (#4740)
  Move library specific key pattern dialog call to library menu (#4744)
  Revert "Revert "Fix: bibkey generated does not handle diacritics" (#4741)" (#4742)
  Revert "Fix: bibkey generated does not handle diacritics" (#4741)
  Rename method to removeUnwantedCharacters
  ...
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.

3 participants