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

fix color scheme of offer book in market view #443

Merged
merged 3 commits into from
Sep 24, 2022

Conversation

DomPro11
Copy link
Contributor

Fixed the white on white text color in the market view when light theme is enabled by adding definitions to theme-light.css

Fixed the white on white text color in the market view when light theme is enabled by adding definitions to theme-light.css
@DomPro11 DomPro11 requested a review from woodser as a code owner September 21, 2022 00:32
@erciccione erciccione linked an issue Sep 21, 2022 that may be closed by this pull request
Copy link
Contributor

@erciccione erciccione left a 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!

Copy link
Contributor

@erciccione erciccione left a 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
@erciccione
Copy link
Contributor

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.

@DomPro11
Copy link
Contributor Author

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 goToOfferView() function defined in desktop/src/main/java/bisq.desktop/main/market/offerbook/OfferBookChartViewModel.java line 223. The BUY/SELL XMR buttons call this function and behave as expected (can still go back to Market View), but there's a buyTableRowSelectionListener in desktop/src/main/java/bisq.desktop/main/market/offerbook/OfferBookChartView.java which calls this function, and I don't think deactivate() is ever called which contains the removeListener() method. Again I haven't confirmed whether this is related to the cause, these are just my thoughts briefly after discovering the bug.

Copy link
Contributor

@erciccione erciccione left a comment

Choose a reason for hiding this comment

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

Thanks @DomPro11 and thanks for looking the issue up! I opened #449 to keep track of the bug, this PR can move on in the meantime IMO :)

@woodser woodser merged commit 6209c95 into haveno-dex:master Sep 24, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

fix color scheme of offer book in market view [0.5 XMR]
3 participants