Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Only load valid themes, fallback to "Light" theme otherwise #3285

Merged
merged 3 commits into from
Jan 14, 2020

Conversation

UdjinM6
Copy link

@UdjinM6 UdjinM6 commented Jan 13, 2020

This should help to avoid issues for users still using old themes we removed in #3141 - without this patch they would see some ugly non-styled version the first time they run v0.15 and would have to pick the theme manually.

@UdjinM6 UdjinM6 changed the title Only load valid themes, fallback to "light" otherwise Only load valid themes, fallback to "Light" theme otherwise Jan 13, 2020
codablock
codablock previously approved these changes Jan 14, 2020
Copy link

@codablock codablock left a comment

Choose a reason for hiding this comment

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

utACK

nmarley
nmarley previously approved these changes Jan 14, 2020
Copy link

@nmarley nmarley left a comment

Choose a reason for hiding this comment

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

utACK

Slight refactor if you wanna take it : nmarley@0f5298d

@UdjinM6 UdjinM6 dismissed stale reviews from nmarley and codablock via 07f3018 January 14, 2020 17:05
@UdjinM6
Copy link
Author

UdjinM6 commented Jan 14, 2020

@nmarley Cherry-picked, thanks!

Copy link

@codablock codablock 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

@nmarley nmarley 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 1be2f94 into dashpay:develop Jan 14, 2020
codablock pushed a commit to codablock/dash that referenced this pull request Jan 17, 2020
…3285)

* Only load valid themes, fallback to "light" otherwise

* Refactor PR3285 a bit

* fix

Co-authored-by: Nathan Marley <[email protected]>
barrystyle pushed a commit to PACGlobalOfficial/PAC that referenced this pull request Jan 22, 2020
…3285)

* Only load valid themes, fallback to "light" otherwise

* Refactor PR3285 a bit

* fix

Co-authored-by: Nathan Marley <[email protected]>
FornaxA pushed a commit to ioncoincore/ion that referenced this pull request Jul 6, 2020
…3285)

* Only load valid themes, fallback to "light" otherwise

* Refactor PR3285 a bit

* fix

Co-authored-by: Nathan Marley <[email protected]>
Signed-off-by: cevap <[email protected]>
@UdjinM6 UdjinM6 deleted the loadlight branch November 26, 2020 13:27
cryptolinux pushed a commit to cryptolinux/ion that referenced this pull request Feb 6, 2021
…3285)

* Only load valid themes, fallback to "light" otherwise

* Refactor PR3285 a bit

* fix

Co-authored-by: Nathan Marley <[email protected]>
Signed-off-by: cevap <[email protected]>
ckti pushed a commit to ckti-gitian-ion/ion that referenced this pull request Mar 29, 2021
…3285)

* Only load valid themes, fallback to "light" otherwise

* Refactor PR3285 a bit

* fix

Co-authored-by: Nathan Marley <[email protected]>
Signed-off-by: cevap <[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