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

fix: Restructures theme name and fixes issue with status bar color on dark theme #604

Merged
merged 1 commit into from
Oct 27, 2020

Conversation

mikaelbr
Copy link
Collaborator

No description provided.

@mikaelbr mikaelbr requested a review from rikkekantega October 27, 2020 12:36
@mikaelbr mikaelbr changed the title Restructures theme name and fixes issue with status bar color on dark theme fix: Restructures theme name and fixes issue with status bar color on dark theme Oct 27, 2020
Copy link
Contributor

@rikkekantega rikkekantega left a comment

Choose a reason for hiding this comment

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

Sweet!

@rikkekantega rikkekantega merged commit 4877fc3 into rikke/theming-usage Oct 27, 2020
@rikkekantega rikkekantega deleted the mikael/theme-name branch October 27, 2020 12:40
rikkekantega added a commit that referenced this pull request Oct 28, 2020
* feat: Text wrapper component for default theme text styles

* feat: Use themed text where appropriate, themed buttons

* fix: Skip ThemedText in "no-raw-text" lint

* feat: themed messageboxes

* fix: Rename text wrapper

* feat: Theming for svgs

* feat: More theming, dark mode onboarding

* feat:  Implement more theming

* fix: Swap button borter

* fix: Current location arrow

* feat: Dark theme for compact map and map controls

* fix: Renaming after resolving conflicts

* feat: Theming on invitemodal

* fix: Fixes status bar color and theme name (#604)

* fix: update theme on colorscheme change

* fix: border color regression

* fix: Some adjustments on dark theme colors to provide better contrasts

* fix: Text optional type prpo

* feat: color adjustments

Co-authored-by: Mikael Brevik <[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.

2 participants