[GEN-1471] feat: delete entity modal #42
Merged
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
This pull request introduces a new
DeleteEntityModal
component and integrates it into the existingOverviewDrawer
component. It also includes changes to the button component to support a new 'danger' variant, and minor adjustments to the modal and theme files.New Component Integration:
frontend/webapp/components/modals/delete-entity-modal/index.tsx
: Added a newDeleteEntityModal
component with props for title, description, modal state, and handlers for delete and close actions.frontend/webapp/containers/main/overview/overview-drawer/index.tsx
: Integrated theDeleteEntityModal
into theOverviewDrawer
component, including state management for the modal's visibility and handlers for opening and closing the modal. [1] [2] [3] [4]Component Exports:
frontend/webapp/components/index.ts
: Exported the newDeleteEntityModal
component.frontend/webapp/components/modals/index.tsx
: Added export for theDeleteEntityModal
component.Button Component Enhancements:
frontend/webapp/reuseable-components/button/index.tsx
: Added a 'danger' variant to the button component, including styles for different states (hover, active, focus). [1] [2] [3] [4] [5]Minor Adjustments:
frontend/webapp/styles/theme.ts
: Added a new color for the 'danger' variant.frontend/webapp/reuseable-components/modal/index.tsx
: Made the modal header optional. [1] [2] [3]frontend/webapp/containers/main/overview/overview-drawer/drawer-footer/index.tsx
: Removed text decoration and transformation fromButtonText
.