-
Notifications
You must be signed in to change notification settings - Fork 2.3k
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: LEAP-1467: Adapt Textarea result container to new style #6413
base: develop
Are you sure you want to change the base?
feat: LEAP-1467: Adapt Textarea result container to new style #6413
Conversation
…h the current brand style. * Moves the edit button outside the container. * Ensure result containers expand the full width so that buttons are aligned on the right. * Adds tooltips to controls.
✅ Deploy Preview for label-studio-docs-new-theme canceled.
|
✅ Deploy Preview for heartex-docs canceled.
|
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.
looks good! one thing to call out for the Tokenization initiative is we have 4px, 8px, 12px and 24px reused through out the application for border radius, gap and spacing related styles so it would be good to be changed to a css variable as part of that initiative - what do you think?
Co-authored-by: hlomzik <[email protected]>
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.
Awesome work, I applied the suggestion for spacing for you so we can merge when ready 💪
Thank you for pointing this out. I'll introduce these variables. |
Description
Adapts the result containers of the TextArea tag to match the new look and feel
Testing
editable="true"
andtranscription="false"
Screenshots
Before:
After: