-
-
Notifications
You must be signed in to change notification settings - Fork 131
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
fix color scheme of offer book in market view #443
Conversation
Fixed the white on white text color in the market view when light theme is enabled by adding definitions to theme-light.css
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Tested ACK
Thanks @DomPro11!
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Actually i just hit a bug where clicking the "Market" doesn't work and the terminal throws a warning:
WARNING: Caught 'java.lang.ClassCastException' while converting value for '-fx-text-fill' from rule '*.label' in stylesheet jar:file:/home/calcifer/DEV/haveno/haveno-dex/haveno/lib/javafx-controls-15-linux.jar!/com/sun/javafx/scene/control/skin/modena/modena.bss
Sep 21, 2022 11:27:50 AM javafx.scene.CssStyleHelper calculateValue
I couldn't find the trigger, but once it happens it's impossible to navigate to the "Market" page.
Fixed the javafx.scene.CssStyleHelper calculateValue warning being spammed in the console
I keep hitting that bug where, i cannot click on the "market" tab after i left it. I deleted local files and done a clean build. Not sure what the problem is. The warning is not there anymore tough. |
I discovered more details about the bug, but since it is present in the current master branch I think it should be addressed as a separate issue (which I would be happy to work on). I'm able to reliably reproduce it after cloning and clean building the master branch by clicking on my own offer in the Market View. The UI will then take me to the Buy (or Sell) tab, and switching back to the Market tab will have the program immediately switch to the Buy/Sell tab again. Unconfirmed, but I think this has something to do with the |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Fixed the white on white text color in the market view when light theme is enabled by adding definitions to theme-light.css