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 style issues #3876

Merged
merged 6 commits into from
Jan 8, 2020
Merged

Fix style issues #3876

merged 6 commits into from
Jan 8, 2020

Conversation

ripcurlx
Copy link
Contributor

@ripcurlx ripcurlx commented Jan 8, 2020

This will fix a text style issue in the offer book that occurs every now and then depending on the other cell row text colors.
Before
Bildschirmfoto 2020-01-08 um 18 02 58
After
Bildschirmfoto 2020-01-08 um 18 03 03

Also it moves all color codes to the top in the dark mode to be more consistent.

@ripcurlx ripcurlx requested a review from sqrrm as a code owner January 8, 2020 16:28
@wiz
Copy link
Contributor

wiz commented Jan 8, 2020

Would you mind putting the before and after screenshot of what's changed?

@ripcurlx
Copy link
Contributor Author

ripcurlx commented Jan 8, 2020

Added another fix for the small label icon in dark mode

Before
Bildschirmfoto 2020-01-08 um 17 37 29
After
Bildschirmfoto 2020-01-08 um 17 59 11

wiz
wiz previously approved these changes Jan 8, 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.

You can remove the "(beta)" label now 😂

@ripcurlx
Copy link
Contributor Author

ripcurlx commented Jan 8, 2020

Just recognized that the switch icon (fixed/percentage based price) is also not visible in dark mode. My fault: it is visible, just not when taking an existing offer.

@ripcurlx
Copy link
Contributor Author

ripcurlx commented Jan 8, 2020

I had to add back a manual workaround as it wasn't possible to solve it otherwise 😞

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.

ACK

Copy link

@RiccardoMasutti RiccardoMasutti left a comment

Choose a reason for hiding this comment

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

ACK

cssThemeFolder + "bisq.css",
cssThemeFolder + "images.css",
cssThemeFolder + "CandleStickChart.css",
// load base styles first

Choose a reason for hiding this comment

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

Indent ?

Copy link
Member

Choose a reason for hiding this comment

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

It was badly indented before, should be 8 for arguments.

Copy link
Contributor

Choose a reason for hiding this comment

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

sorry this was one of my early contributions and I had my text editor set to hard tabs or something

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

cssThemeFolder + "bisq.css",
cssThemeFolder + "images.css",
cssThemeFolder + "CandleStickChart.css",
// load base styles first
Copy link
Member

Choose a reason for hiding this comment

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

It was badly indented before, should be 8 for arguments.

@sqrrm sqrrm merged commit 59021e2 into bisq-network:master Jan 8, 2020
@ripcurlx ripcurlx added this to the v1.2.5 milestone Jan 9, 2020
@ripcurlx ripcurlx deleted the fix-css-issues branch July 16, 2021 07:13
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.

4 participants