Skip to content

Dis 31 landing page comments section#21

Merged
Qiming-Liu merged 15 commits intomainfrom
DIS-31-Landing-Page-Comments-Section
Apr 27, 2025
Merged

Dis 31 landing page comments section#21
Qiming-Liu merged 15 commits intomainfrom
DIS-31-Landing-Page-Comments-Section

Conversation

@Whiskey-Taste
Copy link
Copy Markdown
Contributor

@Whiskey-Taste Whiskey-Taste commented Apr 20, 2025

📝 Summary

This PR introduces a standalone, responsive testimonial section using styled-components, independently styled without relying on the global MUI theme context. The testimonial content is currently hardcoded and rendered through a reusable <TestimonialCard /> component.

WeChatf3499ce6300781960764227390122f1f


🎯 Changes Included

  • ✨ Created TestimonialCard.tsx:
    • Quote icon
    • Testimonial quote
    • Name and title fields
  • ✨ Created TestimonialsSection.tsx:
    • Section wrapper and heading
    • Two testimonial cards rendered responsively (stacked on mobile, horizontal on desktop)
  • 🎨 Styles are implemented using styled-components with static CSS (not theme-driven)
  • ✅ Media query for min-width: 768px applied for layout switching

📂 File Structure

src/app/landing/components/TestimonialCard.tsx
src/app/landing/components/TestimonialsSection.tsx

🚫 Not Included

  • No usage of MUI ThemeProvider or theme.palette / theme.typography
  • No dynamic data integration (currently hardcoded)

✅ Example Usage

// src/app/landing/page.tsx

import TestimonialsSection from './components/TestimonialsSection';

export default function LandingPage() {
  return (
    <main>
      <TestimonialsSection />
    </main>
  );
}

📌 Next Steps

  • 🔄: Migrate to global theme

@Qiming-Liu Qiming-Liu requested a review from Copilot April 23, 2025 05:54

This comment was marked as off-topic.

@Qiming-Liu Qiming-Liu merged commit 30051f7 into main Apr 27, 2025
1 check passed
@Qiming-Liu Qiming-Liu deleted the DIS-31-Landing-Page-Comments-Section branch April 27, 2025 15:12
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