Skip to content

Conversation

@xdustinface
Copy link

@xdustinface xdustinface commented Jun 26, 2020

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 xdustinface force-pushed the pr-ui-10-css-updates branch 2 times, most recently from 5256ad9 to 568d1b9 Compare June 27, 2020 20:23
@xdustinface xdustinface marked this pull request as draft June 30, 2020 20:19
@xdustinface xdustinface force-pushed the pr-ui-10-css-updates branch from 568d1b9 to 542b4ee Compare July 14, 2020 14:59
@xdustinface xdustinface changed the title qt: General css related fixed and updates qt: General CSS related redesigns Jul 14, 2020
@xdustinface xdustinface marked this pull request as ready for review July 14, 2020 15:01
@xdustinface
Copy link
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 :)

Copy link
Member

@PastaPastaPasta PastaPastaPasta left a comment

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

Copy link
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
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.

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

UdjinM6 commented Jul 14, 2020

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

@xdustinface
Copy link
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
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 xdustinface force-pushed the pr-ui-10-css-updates branch from 5a90cae to 2cbd642 Compare July 14, 2020 23:36
Copy link

@UdjinM6 UdjinM6 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
Member

@PastaPastaPasta PastaPastaPasta left a comment

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:develop Jul 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 <[email protected]>
@UdjinM6 UdjinM6 modified the milestones: 17, 16 Sep 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 <[email protected]>
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.

3 participants