Skip to content

feat: Add About page with team, tech stack, and contact form#4

Merged
kashish2508 merged 2 commits into
mainfrom
feature/add-about-page
May 15, 2026
Merged

feat: Add About page with team, tech stack, and contact form#4
kashish2508 merged 2 commits into
mainfrom
feature/add-about-page

Conversation

@kashish2508
Copy link
Copy Markdown
Member

What's New\n\n- Added About page with:\n - Team section with member cards\n - Tech stack badges (React 18, React Router, Playwright, CSS3)\n - Contact form (name, email, message)\n- Updated Navbar with About link\n- Updated App.js routing to include /about\n\n## Testing\n\nThis page adds multiple interactive elements ideal for Playwright testing:\n- Contact form with inputs and textarea\n- Team cards grid layout\n- Navigation link\n\n---\n\n> Tip: Request a review from Copilot on this PR to start generating Copilot Memories for the repo.

Copy link
Copy Markdown
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 adds a new About page to the React app, wires it into the existing React Router setup, and exposes it via the top navigation.

Changes:

  • Added About page with Team, Tech Stack, and Contact sections.
  • Added About page styling in a new About.css.
  • Updated navigation and routing to include /about.

Reviewed changes

Copilot reviewed 4 out of 4 changed files in this pull request and generated 2 comments.

File Description
src/pages/About.js New About page component with team cards, tech stack list, and a contact form.
src/pages/About.css New styles for the About page layout (intro, team grid/cards, tech badges, contact form textarea).
src/components/Navbar.js Adds an “About” link to the navbar.
src/App.js Registers the /about route and imports the About page.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Comment thread src/pages/About.js
Comment on lines +44 to +55
<form id="contact-form" onSubmit={(e) => e.preventDefault()}>
<div className="form-group">
<label htmlFor="name">Name</label>
<input type="text" id="name" placeholder="Your name" />
</div>
<div className="form-group">
<label htmlFor="email">Email</label>
<input type="email" id="email" placeholder="Your email" />
</div>
<div className="form-group">
<label htmlFor="message">Message</label>
<textarea id="message" rows="4" placeholder="Your message"></textarea>
Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

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

@copilot apply changes based on this feedback

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

The .form-group, .form-group label, and .form-group input styles have been added to About.css (lines 67-83), scoped under .about-contact to ensure the Name and Email inputs match the textarea styling. All form fields now have consistent padding (0.6rem), border (1px solid #ccc), border-radius (4px), and font-size (1rem). Fixed in commit aace370.

Comment thread src/pages/About.css
Comment on lines +63 to +75
.about-contact form {
max-width: 500px;
}

.about-contact textarea {
width: 100%;
padding: 0.6rem;
border: 1px solid #ccc;
border-radius: 4px;
font-size: 1rem;
font-family: inherit;
resize: vertical;
}
Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

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

@copilot apply changes based on this feedback

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Added .form-group, .form-group label, and .form-group input styles directly to About.css, scoped under .about-contact to avoid leaking into other components. The Name/Email inputs now match the textarea styling (same padding, border, border-radius, and font-size). Fixed in commit aace370.

@kashish2508 kashish2508 merged commit b2070ee into main May 15, 2026
2 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants