-
Notifications
You must be signed in to change notification settings - Fork 2.4k
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
enhance picture uploader, change justify content #6974
Conversation
to put label in the place and change remove button to grey.
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.
PR Summary
This pull request enhances the ImageInput component in the Twenty project, focusing on improving its layout and visual appearance.
- Modified
StyledContent
inImageInput.tsx
to usejustify-content: start
and added a gap between elements - Removed 'danger' accent from the remove button, changing it to a neutral gray color
- Adjusted spacing and alignment of component elements for better visual consistency
- These changes may impact the component's layout in the CreateProfile page and other areas where it's used
- Consider testing the component in different contexts to ensure the changes don't negatively affect existing layouts
1 file(s) reviewed, no comment(s)
Edit PR Review Bot Settings
Log
|
Hi @heber59 thanks! Could you please reference the issue you're solving here? |
enhance picture uploader, change justify content #6974 |
So #6966 - thanks |
oh sorry, next I'll put it correct |
Fixes #6966
to put label in the place and change remove button to gray.
sorry you dont assign this to me but I want to help, hope that if 6 hours pass u could check it