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

Icons in Entry Edit Window not visible in Dark Mode #7853

Closed
dirkziegenbalg opened this issue Jun 28, 2021 · 4 comments · Fixed by #7994
Closed

Icons in Entry Edit Window not visible in Dark Mode #7853

dirkziegenbalg opened this issue Jun 28, 2021 · 4 comments · Fixed by #7994
Labels
bug Confirmed bugs or reports that are very likely to be bugs good first issue An issue intended for project-newcomers. Varies in difficulty. ui

Comments

@dirkziegenbalg
Copy link

JabRef version 5.3--2021-06-27--aa60dd6 on Windows 10

I have tested the latest development version from http://builds.jabref.org/main and the problem persists

The icons in the entry edit window are frequently too dark to be visible in dark mode. A bright color would be better suited

Steps to reproduce the behavior:

  1. change to dark mode
  2. edit an entry
  3. view icons

Screenshot 2021-06-28 132651

@Siedlerchr Siedlerchr added good first issue An issue intended for project-newcomers. Varies in difficulty. bug Confirmed bugs or reports that are very likely to be bugs ui labels Jun 28, 2021
@JJJoex
Copy link

JJJoex commented Jul 17, 2021

I think we can redesign the dark-theme by adjusting the color of components then it will looks prettier. And Is there anyone work on the issue?

@Siedlerchr
Copy link
Member

@Joe11028 Thanks for your interest! Feel free to work on this!
'You will find the Dark Theme and the Base.css here https://github.com/JabRef/jabref/blob/main/src/main/java/org/jabref/gui/Dark.css

See also this reference guide for the javafx componnents https://openjfx.io/javadoc/16/javafx.graphics/javafx/scene/doc-files/cssref.html

@JJJoex
Copy link

JJJoex commented Jul 17, 2021

@Siedlerchr Okay, thank you for providing informations. I think we should only changed the specific components' color rather than redesign the dark theme, I will try my best to work on this issue.

@JJJoex
Copy link

JJJoex commented Jul 18, 2021

@Siedlerchr I'm now can sure the issue caused by the icon from the imported package(org.kordamp.ikonli.materialdesign). The icons' color can not be changed after we switch to dark theme. So we have two ways to solve this issue:
Add another set of icons that suitable for dark theme. Or don't modify icons, but let the icon-button background color be different from the main body background color, so you can find the icons(buttons) easily.
How do you think of that and which one would you prefer?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Confirmed bugs or reports that are very likely to be bugs good first issue An issue intended for project-newcomers. Varies in difficulty. ui
Projects
Archived in project
Archived in project
Development

Successfully merging a pull request may close this issue.

3 participants