Skip to content

qt: General CSS related redesigns - #3563

Merged
UdjinM6 merged 24 commits into
dashpay:developfrom
xdustinface:pr-ui-10-css-updates
Jul 16, 2020
Merged

qt: General CSS related redesigns#3563
UdjinM6 merged 24 commits into
dashpay:developfrom
xdustinface:pr-ui-10-css-updates

Conversation

@xdustinface

@xdustinfacexdustinface commented Jun 26, 2020

Copy link
Copy Markdown

This PR ist part of a series of +-25 PRs related to UI redesigns. Its ancestor is #3562, its successor is #3564. I did not screenshot every single PR and its changes, instead i made "walk through all screen" videos with the result of this PR series and also with the 0.15 UI. If there are any concrete screenshots wanted, just let me know. To build with the full set of changes you can build from the branch xdustinface:pr-ui-redesign which always contains all changes.

-> Walk through 0.15
-> Walk through Redesign

I tried to give the commits enough text to make things obvious without a lot description for each PR. Also here, if you want more description for this specific PR, let me know.

About this PR

See individual commit messages.

@xdustinface
xdustinfaceforce-pushed the pr-ui-10-css-updates branch 2 times, most recently from 5256ad9 to 568d1b9CompareJune 27, 2020 20:23
@xdustinface
xdustinface marked this pull request as draft June 30, 2020 20:19
@xdustinface
xdustinfaceforce-pushed the pr-ui-10-css-updates branch from 568d1b9 to 542b4eeCompareJuly 14, 2020 14:59
@xdustinfacexdustinface changed the title qt: General css related fixed and updatesqt: General CSS related redesignsJul 14, 2020
@xdustinface
xdustinface marked this pull request as ready for review July 14, 2020 15:01
@xdustinface

Copy link
Copy Markdown
Author

Ready for review!

I combined all of #3563, #3564, #3565, #3566 into this one here. If thats too much let me know and i can split it again but i thought its all only css related... so why not :)

PastaPastaPasta
PastaPastaPasta previously approved these changes Jul 14, 2020

@PastaPastaPastaPastaPastaPasta left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

utACK, made sure changes only touch css.

One question though

Comment threadsrc/qt/res/css/general.css Outdated

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

can you explain the reason for this?

Copy link
Copy Markdown
Author

Choose a reason for hiding this comment

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

Just that relevant colors are all in the same #aarrggbb format and that its catched by contrib/devtools/update-css-files.py. There is still the named color red but thats only a placeholder to show an undefined behaviour in the UI. So if there is a bright red somewhere in the UI it means the related color in general.css isn't overwritten in the theme file. I want to write some UI development style guidelines anytime later where i will explain such thoughts i had.

@UdjinM6UdjinM6 added this to the 17 milestone Jul 14, 2020
@UdjinM6

Copy link
Copy Markdown

Looks very nice in general 👍 traditional theme is a bit weird though and needs few more fixes imo, pls see 6639cb3e6e

@xdustinface

Copy link
Copy Markdown
Author

thanks :) the thing with the traditional is weird.. will push the fix for sure but when i made the walkthrough video i hadn't this bugs in there and now i wonder where on the road i messed with that.. need to investigate here a little first if i messed something else too 🤔

@xdustinface

Copy link
Copy Markdown
Author

Ok no, all-clear, nothing is lost you just got me confused haha.. its the same like back there i just didn't realise that i had the text color twice in there when looked at your diff earlier and then i thought the tab button texts are red but thats overwritten below and you just removed it 🙈 Picked your commit and updated the colors sections in the css files.

@xdustinface
xdustinfaceforce-pushed the pr-ui-10-css-updates branch from 5a90cae to 2cbd642CompareJuly 14, 2020 23:36

@UdjinM6UdjinM6 left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

ACK

@PastaPastaPastaPastaPastaPasta left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

re-utACK

@UdjinM6
UdjinM6 merged commit d5410fd into dashpay:developJul 16, 2020
xdustinface added a commit to xdustinface/dash that referenced this pull request Sep 1, 2020
* qt: User border-image in place of image
For some reason image produced weird outputs on some resolution screens.
* qt: Fix layout issues in TransactionView
* qt: Fix layout issues in OverviewPage
* qt: Replace transparent with #00000000 in stylesheets
* qt: Use #a84832 as red where possible.
This includes css files and any color usage in code.
* qt: Finetune QPushButton style
* qt: Customized QSlider (used in the Debug window -> NetworkTraffic)
* qt: Finetune QToolbar style
* qt: Customized QProgressBar
* qt: Customized QCalendarWidget
* qt: Finetune QStatusBar style
- Give the progress label a name as css reference
- Set min height and margins
* qt: Finetune QMenu style
- Set a light rounded border
- Color for disabled items
- Color/Style for seperator
- Give the items padding
* qt: Customized QGroupBox
* qt: Customized QToolTip
* qt: Give inputs widgets a proper hover/focus/selected/disabled style
* qt: Finetune SendCoinsDialog
- Add light line below the coincontrol frame
- Add light line above fee frame
- Increase header font size
- Remove button layout changes of CoinControl button
- Fix Layout/Alignment of balance labels
* qt: Customized RPC console's autocompletition popup
* qt: Add stylesheets to Intro
This is the datadir selection dialog.
* qt: OptionsDialog - Give the reset button the "light button" style
* qt: Finetune shared styles in general.css
* qt: Finetune coloring in dark.css
* qt: Finetune coloring in light.css
* More fixes for traditional.css
No text colors, s/transparent/#00000000/
* qt: Run update-css-files.py
Co-authored-by: UdjinM6 <UdjinM6@users.noreply.github.com>
@UdjinM6UdjinM6 modified the milestones: 17, 16Sep 4, 2020
gades pushed a commit to cosanta/cosanta-core that referenced this pull request Mar 3, 2022
* qt: User border-image in place of image
For some reason image produced weird outputs on some resolution screens.
* qt: Fix layout issues in TransactionView
* qt: Fix layout issues in OverviewPage
* qt: Replace transparent with #00000000 in stylesheets
* qt: Use #a84832 as red where possible.
This includes css files and any color usage in code.
* qt: Finetune QPushButton style
* qt: Customized QSlider (used in the Debug window -> NetworkTraffic)
* qt: Finetune QToolbar style
* qt: Customized QProgressBar
* qt: Customized QCalendarWidget
* qt: Finetune QStatusBar style
- Give the progress label a name as css reference
- Set min height and margins
* qt: Finetune QMenu style
- Set a light rounded border
- Color for disabled items
- Color/Style for seperator
- Give the items padding
* qt: Customized QGroupBox
* qt: Customized QToolTip
* qt: Give inputs widgets a proper hover/focus/selected/disabled style
* qt: Finetune SendCoinsDialog
- Add light line below the coincontrol frame
- Add light line above fee frame
- Increase header font size
- Remove button layout changes of CoinControl button
- Fix Layout/Alignment of balance labels
* qt: Customized RPC console's autocompletition popup
* qt: Add stylesheets to Intro
This is the datadir selection dialog.
* qt: OptionsDialog - Give the reset button the "light button" style
* qt: Finetune shared styles in general.css
* qt: Finetune coloring in dark.css
* qt: Finetune coloring in light.css
* More fixes for traditional.css
No text colors, s/transparent/#00000000/
* qt: Run update-css-files.py
Co-authored-by: UdjinM6 <UdjinM6@users.noreply.github.com>
Sign up for freeto 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.

3 participants

@xdustinface@UdjinM6@PastaPastaPasta