Closed Eyetsko closed 1 week ago
The recent updates focus on enhancing the Editable
component in the shared atomic design library. These changes include substituting the existing save button implementation with a Button
component featuring additional styling and a streamlined simple
prop. Additionally, modifications have been made to organize code imports automatically in the VSCode settings.
File Path | Change Summary |
---|---|
.vscode/settings.json |
Modified the setting for editor.codeActionsOnSave to use explicit organize imports. |
libs/shared-atomic-design-components/.../Editable.scss |
Introduced specific styling for the save button and its disabled state, including margin, font size, color, text transformation, letter spacing, cursor, and a new .save__button--disabled class. |
libs/shared-atomic-design-components/.../Editable.tsx |
Updated the class names for the save button, changed text content, replaced a div with a Button component including new simple prop, and added a line break before attributes. |
In the code where changes blend, where imports sort and styles mend,
A button brand new, now stands in place, with simpler props and graceful space.
The save within our edit tale, now looks so sleek, where we prevail.
A rabbit cheers this neat design, in lines of code we redefine! 🌸
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?
Thank you very much, dear @Eyetsko!
Dear @Eyetsko, any updates here? Thanks a lot!
Used "Button" component suggested by Kate - screenshots attached
completed all of the above requested by @katamatata - see attached screenshots
@ericbolikowski, @helloanil, it seems that {"source.organizeImports": true}
in the settings.json
is deprecated in favor of {"source.organizeImports": "explicit"}
.
Should be fine to commit it then, right?
@ericbolikowski, @helloanil, it seems that
{"source.organizeImports": true}
in thesettings.json
is deprecated in favor of{"source.organizeImports": "explicit"}
.Should be fine to commit it then, right?
Hey @katamatata, yes it seems like since a specific version of VSCode, "explicit" is used. (My source: https://stackoverflow.com/questions/77637621/vscode-workspace-settings-change-on-its-own) We can keep it in the PR.
What Github issue does this PR relate to? Insert link.
https://github.com/talent-connect/connect/issues/672
What should the reviewer know?
I changed the checkmark to the word "Save". I removed the hover effect. The functionality is otherwise the same: grey when disabled, orange when enabled.
I also tested using a different icon that was a disc, I added the "save" svg to the icons list and took screenshots, just to see this option. Added both here
Summary by CodeRabbit
New Features
Editable
component to use a newButton
component for the save button.Style