-
Notifications
You must be signed in to change notification settings - Fork 9
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
[CP-2720] Modal refactor (dot notation) #1832
Conversation
# Conflicts: # libs/generic-view/ui/src/lib/predefined/import-contacts/import-contacts-list.tsx
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.
I've encountered dot notation before, often misused or overdone. Here, I believe this convention fits perfectly for a component that defines a domain-specific group of sub-components you can utilize.
This approach reduces the number of imports and enhances the readability of the component and its custom derivatives.
Great job on this implementation!
❤️ ❤️ ❤️
libs/generic-view/ui/src/lib/interactive/modal/helpers/modal-size-controller.tsx
Show resolved
Hide resolved
# Conflicts: # libs/generic-view/ui/src/lib/interactive/form/input/search-input.tsx # libs/generic-view/ui/src/lib/interactive/interactive.ts # libs/generic-view/ui/src/lib/predefined/backup-restore/backup-restore-error.tsx # libs/generic-view/ui/src/lib/predefined/backup-restore/backup-restore.tsx # libs/generic-view/ui/src/lib/predefined/backup/backup-error.tsx # libs/generic-view/ui/src/lib/predefined/import-contacts/import-contacts-error.tsx # libs/generic-view/ui/src/lib/predefined/import-contacts/import-contacts-list.tsx # libs/generic-view/ui/src/lib/predefined/import-contacts/import-contacts-progress.tsx # libs/generic-view/ui/src/lib/predefined/import-contacts/import-contacts-success.tsx # libs/generic-view/ui/src/lib/predefined/import-contacts/import-contacts.tsx
Co-authored-by: Łukasz Kowalczyk <freelancer.lukasz.kowalczyk@gmail.com> Co-authored-by: Łukasz Kowalczyk <37898730+lkowalczyk87@users.noreply.github.com>
JIRA Reference: CP-2720
📝 Description ️
💪 Checklist before requesting a review - nice to have
❗ Checklist before merging a pull request