-
Notifications
You must be signed in to change notification settings - Fork 22
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 custom background color #3814
feat: support custom background color #3814
Conversation
@@ -10,6 +10,7 @@ import { DefaultPageTitle, EnrollmentPageKeys } from './DefaultEnrollmentLayout. | |||
|
|||
const getEnrollmentPageStyles = () => ({ | |||
container: { | |||
minHeight: '90vh', |
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.
Open for ideas on this. The page only took up the size of the widgets, but that might not have been the full size. Also, App-platform adds a small padding at the bottom, so setting it to 90 instead of 100 means it will work as corrected.
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.
Seems to work and I don't have a better suggestion.
@@ -10,6 +10,7 @@ import { DefaultPageTitle, EnrollmentPageKeys } from './DefaultEnrollmentLayout. | |||
|
|||
const getEnrollmentPageStyles = () => ({ | |||
container: { | |||
minHeight: '90vh', |
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.
Seems to work and I don't have a better suggestion.
# [101.6.0](v101.5.0...v101.6.0) (2024-09-27) ### Features * support custom background color ([#3814](#3814)) ([22cfe58](22cfe58))
🎉 This PR is included in version 101.6.0 🎉 The release is available on:
Your semantic-release bot 📦🚀 |
Tech-summary:
#
followed by 6 characters