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 #3557, its successor is #3559. 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

If the "UI Debug Mode" (-debug-ui) added by this PR is enabled the content of the css files will become pushed to the UI as soon as there are changes detected. It also forces some UI elements to show up which are usually only shown under special conditions (e.g. watch balance labels). Its kind of required to set a custom css directory with -custom-css-dir to debug the UI otherwise you only get the benefit of seeing the extra widgets.

Its pretty nice and efficient now to develop stylesheets with this mode if you run DashCore with -debug-ui -custom-css-dir=<path-to-your-dash-git-repo>/src/qt/res/css. Allows you to see changes applied to the css files of your git repo live in the UI of DashCore while keeping track of them with git.

@xdustinface xdustinface force-pushed the pr-ui-5-debug-ui branch 2 times, most recently from 5f173d1 to c7accf6 Compare June 27, 2020 20:22
@xdustinface xdustinface marked this pull request as draft June 30, 2020 20:17
The stylesheet is inherited from the parent so its enough to set it for the main window and use this as parent for all other windows.

Except for the ShutdownDialog. This needs to have its own stylesheet set because we can't have the main window as parent of the ShutdownDialog.
This is required now as RPCConsole does not longer show as extra window with a parent set (For stylesheet inheritance).
@xdustinface xdustinface marked this pull request as ready for review July 12, 2020 21:37
@xdustinface
Copy link
Author

Ready for review!

If this mode is enabled the content of the css files will become pushed to the UI if there were any changes made to the stylesheet files in the custom css directory. It also forces some UI elements to show up which are actually only shown under special conditions (e.g. watch balance labels). Its required to set a custom css directory with -custom-css-dir to enable this.
@xdustinface
Copy link
Author

@UdjinM6 Applied the suggestions!

@UdjinM6 UdjinM6 added this to the 17 milestone Jul 13, 2020
UdjinM6
UdjinM6 previously approved these changes Jul 13, 2020
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.

Slightly tested ACK

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.

utACK

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

@UdjinM6 UdjinM6 merged commit fc6fe50 into dashpay:develop Jul 14, 2020
xdustinface added a commit to xdustinface/dash that referenced this pull request Sep 1, 2020
* qt: Set parents properly to inherit css and remove redundant loading

The stylesheet is inherited from the parent so its enough to set it for the main window and use this as parent for all other windows.

Except for the ShutdownDialog. This needs to have its own stylesheet set because we can't have the main window as parent of the ShutdownDialog.

* qt: Adding Qt::Window flag to the constructor of RPCConsole

This is required now as RPCConsole does not longer show as extra window with a parent set (For stylesheet inheritance).

* qt: Add UI debug mode (-debug-ui)

If this mode is enabled the content of the css files will become pushed to the UI if there were any changes made to the stylesheet files in the custom css directory. It also forces some UI elements to show up which are actually only shown under special conditions (e.g. watch balance labels). Its required to set a custom css directory with -custom-css-dir to enable this.

* qt: Adjust the description of -debug-ui
@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: Set parents properly to inherit css and remove redundant loading

The stylesheet is inherited from the parent so its enough to set it for the main window and use this as parent for all other windows.

Except for the ShutdownDialog. This needs to have its own stylesheet set because we can't have the main window as parent of the ShutdownDialog.

* qt: Adding Qt::Window flag to the constructor of RPCConsole

This is required now as RPCConsole does not longer show as extra window with a parent set (For stylesheet inheritance).

* qt: Add UI debug mode (-debug-ui)

If this mode is enabled the content of the css files will become pushed to the UI if there were any changes made to the stylesheet files in the custom css directory. It also forces some UI elements to show up which are actually only shown under special conditions (e.g. watch balance labels). Its required to set a custom css directory with -custom-css-dir to enable this.

* qt: Adjust the description of -debug-ui
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