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
chore: bug: adjust WDS caption text style #31167
Conversation
WalkthroughThe changes involve several updates across the application's codebase, focusing on typography adjustments, feature flag checks, and widget style configurations. The modifications include reordering typography variants for consistency, enhancing feature flag evaluation for specific flags, and adjusting widget style options to meet design requirements. These changes aim to improve the application's usability and align with design system standards. Changes
Assessment against linked issues
Thank you for using CodeRabbit. We offer it for free to the OSS community and would appreciate your support in helping us grow. If you find it useful, would you consider giving us a shout-out on your favorite social media? TipsChatThere are 3 ways to chat with CodeRabbit:
Note: Be mindful of the bot's finite context window. It's strongly recommended to break down tasks such as reading entire modules into smaller chunks. For a focused discussion, use review comments to chat about specific files and their changes, instead of using the PR comments. CodeRabbit Commands (invoked as PR comments)
Additionally, you can add CodeRabbit Configration File (
|
@vadim Note: In the PR, I have just reverted the order of caption and body. The reason is in the code, we don't specifically create sizes for body,caption,footnote etc. We just create some sizes and assign them by index. return Object.keys(TYPOGRAPHY_VARIANTS).reduce((prev, current, index) => {
return {
...prev,
[current]: styles[index],
};
}, {} as Typography); |
@jsartisan this seems to change the effective body size (and as a result all the widgets having text of that size), can we avoid that? |
Few solutions i can think of:
|
|
@ichik You're right, but everything seems to be correct here. If we want to change the values, we can tweak the config and make the growth slower or something. |
I was just wondering why people are getting confused with caption is because of its name? maybe they expect "caption" to be smaller than "body". so i thought just changing the name would have cleared their confusion. just a random thought. |
Fixes #30896
Summary by CodeRabbit
true
for specific flags, improving feature toggling.