Skip to content

qt: Introduce runtime theme changes - #3559

Merged
UdjinM6 merged 3 commits into
dashpay:developfrom
xdustinface:pr-ui-6-runtime-theme-changes
Jul 19, 2020
Merged

qt: Introduce runtime theme changes#3559
UdjinM6 merged 3 commits into
dashpay:developfrom
xdustinface:pr-ui-6-runtime-theme-changes

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 #3558, its successor is #3560. 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

This PR enables runtime theme changes. Means no more client restart required if the theme gets changed in the options dialog.

@xdustinface
xdustinfaceforce-pushed the pr-ui-6-runtime-theme-changes branch 2 times, most recently from dc80192 to 5eaf6a3CompareJune 27, 2020 20:22
@xdustinface
xdustinface marked this pull request as draft June 30, 2020 20:18
This allows enabling them again on theme changes
Runtime theme changes means no more client restart required if the
theme gets changed in the options dialog.
In the RPCConsole's StyleChange event make sure following things are still correct after a runtime theme change:
- Hide prompt icon for dash themes in rpc console if dash theme gets
activated.
- Clear rpc console on theme changes to make sure fonts/sizes/colors are correct.
@xdustinface
xdustinfaceforce-pushed the pr-ui-6-runtime-theme-changes branch from 5eaf6a3 to ceeccfbCompareJuly 14, 2020 22:06
@xdustinface
xdustinface marked this pull request as ready for review July 14, 2020 22:06
@xdustinface

Copy link
Copy Markdown
Author

Ready for review!

@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, seems to be working as expected

@UdjinM6UdjinM6 added this to the 17 milestone Jul 16, 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.

light code review, tested ACK

@UdjinM6
UdjinM6 merged commit 4a7bb86 into dashpay:developJul 19, 2020
xdustinface added a commit to xdustinface/dash that referenced this pull request Sep 1, 2020
* qt: Set the default theme properly
* qt: Keep track of disabled rects for macOS
This allows enabling them again on theme changes
* qt: Introduce runtime theme changes
Runtime theme changes means no more client restart required if the
theme gets changed in the options dialog.
In the RPCConsole's StyleChange event make sure following things are still correct after a runtime theme change:
- Hide prompt icon for dash themes in rpc console if dash theme gets
activated.
- Clear rpc console on theme changes to make sure fonts/sizes/colors are correct.
@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: Set the default theme properly
* qt: Keep track of disabled rects for macOS
This allows enabling them again on theme changes
* qt: Introduce runtime theme changes
Runtime theme changes means no more client restart required if the
theme gets changed in the options dialog.
In the RPCConsole's StyleChange event make sure following things are still correct after a runtime theme change:
- Hide prompt icon for dash themes in rpc console if dash theme gets
activated.
- Clear rpc console on theme changes to make sure fonts/sizes/colors are correct.
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