-
-
Notifications
You must be signed in to change notification settings - Fork 32.1k
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
[Paper] Allow the box shadow color to be changed #3003
Conversation
@@ -109,6 +109,16 @@ export default function getMuiTheme(baseTheme, muiTheme) { | |||
}, | |||
paper: { | |||
backgroundColor: palette.canvasColor, | |||
zDepthShadows: [null].concat([ |
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.
Any reason for using concat
?
If you think the code is okay now, I can squash the commits to have a cleaner merge ;) |
[14, 45, 0.25, 10, 18, 0.22], | ||
[19, 60, 0.30, 15, 20, 0.22], | ||
].map(d => { | ||
const shadowColor = palette.shadowColor || Colors.fullBlack; |
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.
I don't know what to think about palette.shadowColor
.
That's kind of an undocumented hidden feature.
That makes me think that we should add it to the lightBaseTheme
.
(I know, I asked you to remove it, sorry 😁)
@alitaheri @newoga What do you think about using palette.shadowColor
in this file and adding shadowColor: Colors.fullBlack
in the lightBaseTheme.jsx
?
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.
Yeah makes sense 👍 👍
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.
Yup I agree, I think that makes more sense 👍
I squashed everything into one small commit to have a cleaner log ;) |
[Paper] Allow the box shadow color to be changed
@tibdex Merci! |
This allows dark or custom themes to change the box-shadow color of paper elements