-
Notifications
You must be signed in to change notification settings - Fork 17
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
[PLAY-1633] Display Title #4242
base: master
Are you sure you want to change the base?
Conversation
128px | ||
); | ||
} | ||
|
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.
Dynamic classes attempt to simulate % of screen. They don't work as intended. Could be replaced with sm, md, lg if we go this route.
A change to documentation files was detected in your PR. Please visit this link to preview changes: https://portal-staging.powerapp.cloud/docs?filters[kind]=all&filters[user]=all&filters[namespaceFilter]=play-1633-display-title |
What does this PR do? A clear and concise description with your runway ticket url.
Story
This PR adds a "display" option for the existing size prop. It also adds a new
displaySize
prop for creating dynamically responsive titles at various sizes. We are still deciding whether to keepdisplaySize
or not.Screenshots: Screenshots to visualize your addition/change

How to test? Steps to confirm the desired behavior:
Checklist:
enhancement
,bug
,improvement
,new kit
,deprecated
, orbreaking
. See Changelog & Labels for details.milano
label to show I'm ready for a review.