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
fix: improved widgets layout #31471
fix: improved widgets layout #31471
Conversation
WalkthroughWalkthroughThis series of changes focuses on enhancing the design system and widgets within a client application. It includes updates to CSS styling for better alignment and presentation, adjustments to widget properties for consistency in label positioning, and removals of unused or redundant configurations. The modifications aim to streamline the user interface and improve the overall user experience by refining visual elements and simplifying widget configurations. Changes
Related 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 (
|
@@ -98,7 +98,7 @@ export const AnvilFlexComponent = forwardRef( | |||
ref={ref} | |||
style={anvilWidgetStyleProps} | |||
> | |||
<div className="h-full w-full">{children}</div> | |||
{children} |
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.
cc @riodeuno
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.
@KelvinOm Removing this is causing issues in the fill widget width in the deploy mode. Let's make sure that issue doesn't re-occur. Then we can safely merge this.
Please call it out if you'd like someone from Anvil to own this.
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.
@riodeuno I have checked that everything is working as expected. I'll write a message to Anvil channel anyway after after DP update.
Description
PR fixes following issue(s)
Fixes # (issue number)
#29088
#31250
Type of change
Testing
How Has This Been Tested?
Checklist:
Dev activity