Skip to content

Dis 27 landing page hero section chloe#16

Merged
Qiming-Liu merged 21 commits intomainfrom
DIS-27-Landing-Page-Hero-Section-Chloe
May 2, 2025
Merged

Dis 27 landing page hero section chloe#16
Qiming-Liu merged 21 commits intomainfrom
DIS-27-Landing-Page-Hero-Section-Chloe

Conversation

@ChloeXiao0409
Copy link
Copy Markdown
Contributor

Chloe- Create Hero Section based on Zeplin design:

Design reference: Hero Section
Desktop:
image

Copilot AI review requested due to automatic review settings April 17, 2025 11:39
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 pull request implements the hero section for the landing page based on the Zeplin design.

  • Introduces a new HeroSection component using Material UI components.
  • Implements responsive typography, button layouts, and a styled image container.
Files not reviewed (2)
  • package.json: Language not supported
  • pnpm-lock.yaml: Language not supported

@ChloeXiao0409
Copy link
Copy Markdown
Contributor Author

Stage 1 hero section, checked
Start working on insert global theme varaibles

@ChloeXiao0409 ChloeXiao0409 reopened this Apr 23, 2025
Copy link
Copy Markdown
Collaborator

@Qiming-Liu Qiming-Liu left a comment

Choose a reason for hiding this comment

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

Change the file structure

@ChloeXiao0409
Copy link
Copy Markdown
Contributor Author

Change the file structure

Just changed the file structure aligned with the current main structure with the hero section and footer; request to review.
Thank you!

@Qiming-Liu Qiming-Liu merged commit 1a872ac into main May 2, 2025
1 check passed
@Qiming-Liu Qiming-Liu deleted the DIS-27-Landing-Page-Hero-Section-Chloe branch May 2, 2025 06:39
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.

4 participants