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 #3556, its successor is #3558. 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 allows to load a custom directory as css directory. The css files in the provided directory will be used as stylesheets for the UI instead of the app internal ones.

@xdustinface xdustinface force-pushed the pr-ui-4-custom-css branch 2 times, most recently from b6af742 to 28ae550 Compare June 27, 2020 20:22
@xdustinface xdustinface marked this pull request as draft June 30, 2020 20:17
Required for the introduction of a custom css directory which will
happen with the follow-up commits.
@xdustinface xdustinface force-pushed the pr-ui-4-custom-css branch from 28ae550 to 9acce41 Compare July 8, 2020 23:35
@xdustinface xdustinface marked this pull request as ready for review July 8, 2020 23:36
@xdustinface
Copy link
Author

ready for review

@xdustinface xdustinface force-pushed the pr-ui-4-custom-css branch 4 times, most recently from 4219d60 to 3030216 Compare July 9, 2020 02:20
@xdustinface
Copy link
Author

xdustinface commented Jul 9, 2020

Those force pushes were mostly only to figure how to reorder the commits in here properly because the order was messed. Now they appear in the correct order. One did change the command line description a bit.

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.

One nit/typo, otherwise looks good imo

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

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.

One comment, haven't tested, overall looks fine

src/qt/dash.cpp Outdated
Copy link
Member

Choose a reason for hiding this comment

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

Why the double new line?

Copy link
Author

Choose a reason for hiding this comment

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

To get a bit space before the list of missing files. Thanks for asking haven't tested that for a while and it now realised that i messed with it over the time, will force push soon.
Bildschirmfoto 2020-07-11 um 05 07 14

This allows to load a custom directory as css directory.

The custom directory currently needs to have the files:

- general.css
- dark.css
- light.css
- traditional.css
- scrollbars.css
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.

re-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 41eceb8 into dashpay:develop Jul 12, 2020
xdustinface added a commit to xdustinface/dash that referenced this pull request Sep 1, 2020
* qt: Change themes path from ":themes" to ":css"

Required for the introduction of a custom css directory which will
happen with the follow-up commits.

* qt: Renamed trad.css to traditional.css

* qt: Add -custom-css-dir startup parameter

This allows to load a custom directory as css directory.

The custom directory currently needs to have the files:

- general.css
- dark.css
- light.css
- traditional.css
- scrollbars.css
@UdjinM6 UdjinM6 removed this from the 17 milestone Sep 4, 2020
@UdjinM6 UdjinM6 added this to the 16 milestone Sep 4, 2020
gades pushed a commit to cosanta/cosanta-core that referenced this pull request Mar 3, 2022
* qt: Change themes path from ":themes" to ":css"

Required for the introduction of a custom css directory which will
happen with the follow-up commits.

* qt: Renamed trad.css to traditional.css

* qt: Add -custom-css-dir startup parameter

This allows to load a custom directory as css directory.

The custom directory currently needs to have the files:

- general.css
- dark.css
- light.css
- traditional.css
- scrollbars.css
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