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

Make prompt text readable in dark mode #4092

Merged

Conversation

ripcurlx
Copy link
Contributor

@ripcurlx ripcurlx commented Mar 24, 2020

As this annoys me for a long time when using dark mode, I quickly fixed this issue.
No compensation necessary - just fixing this was satisfaction enough ;-)

Before

Bildschirmfoto 2020-03-24 um 10 41 02

After

Bildschirmfoto 2020-03-24 um 10 38 21

@ripcurlx ripcurlx requested a review from wiz March 24, 2020 09:45
@ripcurlx ripcurlx added the is:no-priority PR or issue marked with this label is not up for compensation right now label Mar 24, 2020
Copy link
Contributor

@wiz wiz left a comment

Choose a reason for hiding this comment

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

Excellent! For some reason, I never noticed that dark text on dark background before. I've tested this change while using dark theme and didn't notice any side effects 👍

Copy link
Contributor

@wiz wiz left a comment

Choose a reason for hiding this comment

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

Actually just noticed this

Mar 24, 2020 7:04:48 PM javafx.scene.CssStyleHelper calculateValue
WARNING: Caught 'java.lang.ClassCastException: java.base/java.lang.String cannot be cast to [email protected]/javafx.scene.paint.Color' while converting value for '-fx-prompt-text-fill' from rule '*.text-field' in stylesheet jar:file:/Applications/Bisq.app/Contents/Java/Bisq-1.2.9.jar!/bisq/desktop/bisq.css
Mar 24, 2020 7:04:48 PM javafx.scene.CssStyleHelper calculateValue
WARNING: Caught 'java.lang.ClassCastException: java.base/java.lang.String cannot be cast to [email protected]/javafx.scene.paint.Color' while converting value for '-fx-prompt-text-fill' from rule '*.text-field' in stylesheet jar:file:/Applications/Bisq.app/Contents/Java/Bisq-1.2.9.jar!/bisq/desktop/bisq.css

@ripcurlx
Copy link
Contributor Author

Actually just noticed this

Mar 24, 2020 7:04:48 PM javafx.scene.CssStyleHelper calculateValue
WARNING: Caught 'java.lang.ClassCastException: java.base/java.lang.String cannot be cast to [email protected]/javafx.scene.paint.Color' while converting value for '-fx-prompt-text-fill' from rule '*.text-field' in stylesheet jar:file:/Applications/Bisq.app/Contents/Java/Bisq-1.2.9.jar!/bisq/desktop/bisq.css
Mar 24, 2020 7:04:48 PM javafx.scene.CssStyleHelper calculateValue
WARNING: Caught 'java.lang.ClassCastException: java.base/java.lang.String cannot be cast to [email protected]/javafx.scene.paint.Color' while converting value for '-fx-prompt-text-fill' from rule '*.text-field' in stylesheet jar:file:/Applications/Bisq.app/Contents/Java/Bisq-1.2.9.jar!/bisq/desktop/bisq.css

😬 good catch - I'll have a look why this is happening

@ripcurlx ripcurlx force-pushed the make-prompt-text-readable-in-dark-mode branch from 9d92293 to 37751c7 Compare March 24, 2020 11:53
@ripcurlx
Copy link
Contributor Author

I fixed a bug with another bug 😝 Just force-pushed the correct implementation.

@ripcurlx ripcurlx requested a review from wiz March 25, 2020 09:12
Copy link
Member

@sqrrm sqrrm left a comment

Choose a reason for hiding this comment

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

utACK

@sqrrm sqrrm merged commit 7fd4cb5 into bisq-network:master Mar 26, 2020
@ripcurlx ripcurlx added this to the v1.3.0 milestone Apr 1, 2020
@ripcurlx ripcurlx deleted the make-prompt-text-readable-in-dark-mode branch July 16, 2021 07:14
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
is:no-priority PR or issue marked with this label is not up for compensation right now
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants