Refactor: Replace Section component imports with EngagementModal in … #781
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
…various containers - Updated imports in blog, brand, contact, course, cta, donate, event, faq, funfact, gradation, newsletter, profile, quote, register-guide, service, team, testimonial, timeline, video, and zoom-meetings containers to use EngagementModal instead of the previous Section component. - Added EngagementModal component with custom animations and accessibility features. - Introduced EngagementModal styles for improved UI/UX.
|
The latest updates on your projects. Learn more about Vercel for Git ↗︎
|
There was a problem hiding this comment.
Pull Request Overview
This pull request refactors the import paths for the Section component to use the EngagementModal component and introduces debugging enhancements to the EngagementModal.
- Replaced all Section imports from "@ui/section" with "@components/ui/engagement-modal".
- Added useEffect hooks in EngagementModal for logging mount and open state, and removed the obsolete hasInitialized ref.
Reviewed Changes
Copilot reviewed 64 out of 64 changed files in this pull request and generated no comments.
Show a summary per file
| File | Description |
|---|---|
| src/containers/contact-info/layout-01/index.tsx | Updated import to use EngagementModal. |
| src/containers/contact-form/layout-02/index.tsx | Updated import to use EngagementModal. |
| src/containers/contact-form/layout-01/index.tsx | Updated import to use EngagementModal. |
| src/containers/brand/layout-02/index.tsx | Updated import to use EngagementModal. |
| src/containers/brand/layout-01/index.tsx | Updated import to use EngagementModal. |
| src/containers/blog/layout-03/index.tsx | Updated import to use EngagementModal. |
| src/containers/blog/layout-02/index.tsx | Updated import to use EngagementModal. |
| src/containers/blog/layout-01/index.tsx | Updated import to use EngagementModal. |
| src/containers/blog-full/layout-05/index.tsx | Updated import to use EngagementModal. |
| src/containers/blog-full/layout-04/index.tsx | Updated import to use EngagementModal. |
| src/containers/blog-full/layout-03/index.tsx | Updated import to use EngagementModal. |
| src/containers/blog-full/layout-02/index.tsx | Updated import to use EngagementModal. |
| src/containers/blog-full/layout-01/index.tsx | Updated import to use EngagementModal. |
| src/containers/app-download/index.tsx | Updated import to use EngagementModal. |
| src/containers/about/layout-02/index.tsx | Updated import to use EngagementModal. |
| src/containers/about/layout-01/index.tsx | Updated import to use EngagementModal. |
| src/components/vwc-grid/index.tsx | Updated import to use EngagementModal. |
| src/components/ui/engagement-modal/EngagementModal.tsx | Added debugging logs and removed redundant ref. |
Comments suppressed due to low confidence (1)
src/components/ui/engagement-modal/EngagementModal.tsx:34
- Consider conditionally disabling or removing the debug logs in the EngagementModal component before deploying to production to avoid exposing internal state.
}, []);
|
🔍 Code Quality Score Breakdown:
💡 Recommendations:
|
jonulak
added a commit
that referenced
this pull request
Jun 18, 2025
commit b56e409 Author: Jerome Hardaway <jerome@vetswhocode.io> Date: Tue Jun 17 18:24:35 2025 -0400 Refactor: Replace Section component imports with EngagementModal in various containers (#781) - Updated imports in blog, brand, contact, course, cta, donate, event, faq, funfact, gradation, newsletter, profile, quote, register-guide, service, team, testimonial, timeline, video, and zoom-meetings containers to use EngagementModal instead of the previous Section component. - Added EngagementModal component with custom animations and accessibility features. - Introduced EngagementModal styles for improved UI/UX. commit bc3c9d2 Author: Jerome Hardaway <jerome@vetswhocode.io> Date: Tue Jun 17 13:22:46 2025 -0400 refactor: remove unused EngagementModal component and associated animations commit 8add330 Author: Jerome Hardaway <jerome@vetswhocode.io> Date: Tue Jun 17 08:44:42 2025 -0400 Add Engagement Modal with custom animations and accessibility features (#780) - Implemented EngagementModal component with props for headline, body, and call-to-action buttons. - Added custom CSS animations for emoji wiggle effect on button hover. - Integrated Framer Motion for smooth modal transitions. - Included accessibility features such as focus trapping and ESC key close functionality. - Added session storage logic to prevent multiple modal displays in a session. - Provided a development-only button to test modal functionality. commit e4c58e6 Author: Jerome Hardaway <jerome@vetswhocode.io> Date: Wed Jun 11 18:44:26 2025 -0400 refactor: streamline request body validation and Slack message formatting
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
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 debugging enhancements to the
EngagementModalcomponent and consolidates imports across multiple files to use the updated path for theSectioncomponent. The most important changes include adding debugging logs to theEngagementModaland replacing@ui/sectionimports with@components/ui/engagement-modal.Debugging Enhancements to
EngagementModal:useEffecthooks to log the mount state andopenstate for debugging purposes insrc/components/ui/engagement-modal/EngagementModal.tsx.hasInitializedref and its associated logic as it is no longer needed.Import Path Updates:
Sectionimports from@ui/sectionto@components/ui/engagement-modalacross multiple files, includingsrc/containers/about,src/containers/blog,src/containers/contact-form, and others. [1] [2] [3] [4] [5] [6] [7] [8] [9] [10] [11] [12] [13] [14] [15] [16] [17]…various containers