-
Notifications
You must be signed in to change notification settings - Fork 83
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
feat: support multiple themes #64
Conversation
07aab09
to
816b679
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Please also add it to the nginx configuration, so the theme can be set for each channel
8cd863f
to
01940f5
Compare
1bae6ab
to
e21a3d7
Compare
6f17ef0
to
2a5aa9d
Compare
edce062
to
613f268
Compare
613f268
to
b5b97b3
Compare
b5b97b3
to
24aab08
Compare
24aab08
to
e4ff967
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
All is fine now.
We still need to wait for the definition of the new B2B color and the fitting logo images.
Might I suggest doing that later in another PR? There is plenty of business value already packed in here, we can release it for projects to use and also provide schematics creating a new theme here. (see #77) |
8afc5dd
to
0db0937
Compare
PR Type
What Is the Current Behavior?
Issue Number:
#49
What Is the New Behavior?
multiple styles & matrix-parameter switch
Does this PR Introduce a Breaking Change?