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

Open Jdk problems with Look and Feels #1116

Merged
merged 4 commits into from
Apr 7, 2016
Merged

Open Jdk problems with Look and Feels #1116

merged 4 commits into from
Apr 7, 2016

Conversation

stefan-kolb
Copy link
Member

See #393 #638.

Still not 100% sure if this only applies to OpenJDK and GTK L&F or also to Oracle JDK and GTK L&F.
Metal L&F seems to work but looks ugly.

Idea is to try Metal on OpenJDK and wait if more bug reports pop up even for Oracle.

Also see https://bugs.launchpad.net/ubuntu/+source/openjdk-8/+bug/1510009

Especially see
https://bugs.debian.org/cgi-bin/bugreport.cgi?bug=813143

Maybe we can just ignore this and wait for it to be fixed in OpenJDK and add a hint to our FAQ and documentation?

@stefan-kolb stefan-kolb added ui os: linux status: ready-for-review Pull Requests that are ready to be reviewed by the maintainers labels Apr 6, 2016
# Conflicts:
#	src/main/java/net/sf/jabref/JabRef.java
@stefan-kolb
Copy link
Member Author

When merging this in please use squash merge to make it easy to revert this change when this is obsolete for OpenJDK later on.

@simonharrer
Copy link
Contributor

👍

@stefan-kolb stefan-kolb added this to the v3.3 milestone Apr 6, 2016
@simonharrer simonharrer merged commit ac0bae5 into master Apr 7, 2016
@stefan-kolb stefan-kolb deleted the jdk-bug branch April 7, 2016 14:45
@koppor
Copy link
Member

koppor commented Apr 8, 2016

Oh, where did 4aa0d5b go? As far as I can see, only ac0bae5 (which is very short in comparison to the changes shown above) has been cherry-picked?

@mlep
Copy link
Contributor

mlep commented Apr 8, 2016

#1145 : related?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
os: linux status: ready-for-review Pull Requests that are ready to be reviewed by the maintainers ui
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants