Skip to content

Bring back the context menu#5352

Merged
tobiasdiez merged 2 commits into
masterfrom
contextmenu
Sep 26, 2019
Merged

Bring back the context menu#5352
tobiasdiez merged 2 commits into
masterfrom
contextmenu

Conversation

@tobiasdiez

Copy link
Copy Markdown
Member

As explained in #5254 (comment) it is no longer possible to customize the context menu items, because the classes related to the text field behavior are not accessible. Thus, instead I simply copied the relevant code from openjdk. Pretty ugly workaround... Fixes #5254 and refs javafxports/openjdk-jfx#583.

Moreover, I removed the custom tab handling because it is now working as expected (i.e. #2902 is fixed in Java 9+).


  • 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?)

As explained in #5254 (comment) it is no longer possible to customize the context menu items, because the classes related to the text field behavior are not accessible. Thus, instead I simply copied the relevant code from openjdk. Pretty ugly workaround... Fixes #5254 and refs javafxports/openjdk-jfx#583.

Moreover, I removed the custom tab handling because it is now working as expected (i.e. #2902 is fixed in Java 9+).
@tobiasdiez tobiasdiez added the status: ready-for-review Pull Requests that are ready to be reviewed by the maintainers label Sep 25, 2019
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.

The context menu for entries is gone

2 participants