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 #3559, its successor is #3561. 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 removes all occurences of QTabBar (SignVerifyMessageDialog, OptionsDialog, RPCConsole). Instead it adds a QStackedWidget as replacement with QPushButtons in a QHBoxLayout grouped into a QButtonGroup above it to simulate the tabs. This is because the QTabBar items can't be stretched and styled the way it's possible with the QPushButtons. It also adds stylesheets to give the buttons a nice look.

@xdustinface xdustinface force-pushed the pr-ui-7-tabbar-replacement branch 2 times, most recently from e4018b7 to fba4e68 Compare June 27, 2020 20:22
@xdustinface xdustinface marked this pull request as draft June 30, 2020 20:19
This allows to stretch the buttons in the dialog window.
This allows to stretch the buttons in the dialog window.
This allows to stretch the buttons in the dialog window.
@xdustinface xdustinface force-pushed the pr-ui-7-tabbar-replacement branch from fba4e68 to efbe42a Compare July 12, 2020 21:57
@xdustinface
Copy link
Author

Ready for review!

@xdustinface xdustinface marked this pull request as ready for review July 12, 2020 21:59
@UdjinM6 UdjinM6 added this to the 17 milestone 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
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.

general utACK... merge conflicts for future backports are really starting to scare me though...

@UdjinM6 UdjinM6 merged commit 83420a1 into dashpay:develop Jul 14, 2020
xdustinface added a commit to xdustinface/dash that referenced this pull request Sep 1, 2020
* qt: OptionsDialog - Replaced QTabBar with QStackedWidget

This allows to stretch the buttons in the dialog window.

* qt: RPCConsole - Replaced QTabBar with QStackedWidget

This allows to stretch the buttons in the dialog window.

* qt: SignVerifyMessageDialog - Replaced QTabBar with QStackedWidget

This allows to stretch the buttons in the dialog window.

* qt: Add styles for tabbar replacement buttons
christiancfifi added a commit to christiancfifi/dash that referenced this pull request Sep 11, 2021
The Dash UI changes were introduced in 83420a1 ("qt: Replace usage of QTabBar with custom replacement (dashpay#3560)", 2020-07-14)
christiancfifi added a commit to christiancfifi/dash that referenced this pull request Sep 11, 2021
The Dash UI changes were introduced in 83420a1 ("qt: Replace usage of QTabBar with custom replacement (dashpay#3560)", 2020-07-14)
gades pushed a commit to cosanta/cosanta-core that referenced this pull request Mar 3, 2022
* qt: OptionsDialog - Replaced QTabBar with QStackedWidget

This allows to stretch the buttons in the dialog window.

* qt: RPCConsole - Replaced QTabBar with QStackedWidget

This allows to stretch the buttons in the dialog window.

* qt: SignVerifyMessageDialog - Replaced QTabBar with QStackedWidget

This allows to stretch the buttons in the dialog window.

* qt: Add styles for tabbar replacement buttons
gades pushed a commit to cosanta/cosanta-core that referenced this pull request May 9, 2022
The Dash UI changes were introduced in 83420a1 ("qt: Replace usage of QTabBar with custom replacement (dashpay#3560)", 2020-07-14)
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