Skip to content

Conversation

@pphatdev
Copy link
Owner

This pull request includes updates to dependencies, refactors the ContactSection component, and improves the HomeHeroSection layout. The changes streamline the codebase, enhance functionality, and improve the user interface.

Dependency Updates

  • Added new dependencies: @radix-ui/react-dialog, @types/canvas-confetti, canvas-confetti, and vaul to package.json to support new features. [1] [2]

ContactSection Refactor

  • Replaced inline form and message-sent UI with reusable components ContactForm and SentChat. This simplifies the code and improves maintainability. [1] [2]
  • Introduced a celebrate function using canvas-confetti to trigger a confetti animation upon successful form submission. [1] [2]

HomeHeroSection Enhancements

  • Removed hardcoded technology stack logos and replaced them with a reusable TechStack component for better modularity. [1] [2] [3]
  • Adjusted layout and styles for better responsiveness and visual hierarchy, particularly on smaller screens. [1] [2]

Minor Updates

  • Removed unused ContactSection import from HomePage and updated the component structure accordingly. [1] [2]

pphatdev added 5 commits June 26, 2025 14:31
- Added @radix-ui/react-dialog for modal dialogs.
- Introduced a new contact form component with validation and submission handling.
- Implemented responsive design for the contact button using a drawer for mobile and a dialog for desktop.
- Updated profile and home page to remove the old contact section.
- Enhanced UI components with animations and improved accessibility.
- Added media query hook for responsive behavior.
- Updated package.json to include new dependencies.
…t card for improved responsiveness and visual consistency
…outTimeline, and ContactForm for improved user experience
…oSection and update styles for improved visual presentation
@pphatdev pphatdev requested a review from Copilot June 27, 2025 01:31
@pphatdev pphatdev self-assigned this Jun 27, 2025
@pphatdev pphatdev added the enhancement New feature or request label Jun 27, 2025
@vercel
Copy link

vercel bot commented Jun 27, 2025

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Comments Updated (UTC)
v4 ✅ Ready (Inspect) Visit Preview 💬 Add feedback Jun 27, 2025 1:35am

Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull Request Overview

This PR redesigns the “Get In Touch” section by extracting the contact form into reusable components with a confetti celebration, updates dependencies, and improves the HomeHeroSection’s layout by swapping hardcoded tech logos for a TechStack component and refining responsiveness.

  • Update dependencies and introduce Radix dialog, canvas-confetti, and Vaul drawer support
  • Refactor ContactSection to use ContactForm and SentChat components with a confetti animation
  • Replace static tech logos in HomeHeroSection with a reusable TechStack component and tweak layout for small screens

Reviewed Changes

Copilot reviewed 26 out of 27 changed files in this pull request and generated 3 comments.

Show a summary per file
File Description
src/lib/interfaces/contact-form.ts Define props interfaces for ContactForm and SentChat
src/hooks/projects.ts Remove hook implementation, leaving only imports
src/app/(web)/(home)/sections/contact.tsx Refactor ContactSection to use ContactForm/SentChat and add celebrate
src/app/globals.css Simplify keyframe transform by removing translate offsets
src/app/(web)/(home)/sections/hero-section.tsx Swap hardcoded logos for TechStack, adjust responsive classes
package.json Add @radix-ui/react-dialog, canvas-confetti, vaul, and related types
Comments suppressed due to low confidence (2)

src/hooks/projects.ts:1

  • This file contains only imports and no hook implementation or exports, which will break useProjects. Either restore the hook body or remove this unused file.
import { useState, useEffect } from 'react';

src/app/globals.css:100

  • Removing the translate(-50%, -50%) may break centering for elements relying on that transform; verify animation origin or restore translation if needed.
      transform: scale(1);

Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
@pphatdev pphatdev merged commit 0a8d4d0 into master Jun 27, 2025
3 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

enhancement New feature or request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant