Skip to content

41 mobile frontend mainpage#46

Merged
IvanA673 merged 14 commits intodevelopfrom
41-mobile-frontend-mainpage
May 7, 2025
Merged

41 mobile frontend mainpage#46
IvanA673 merged 14 commits intodevelopfrom
41-mobile-frontend-mainpage

Conversation

@vkohli244
Copy link
Copy Markdown
Collaborator

@vkohli244 vkohli244 commented May 4, 2025

Developers: Vinayak Kohli, Tyler Min Hset

Closes : 41

Pull Request Summary

Added Responsive styling for landing page, updated mobile view to have header image. Font sizing changed to be dynamic to avoid wrapping on smaller screens. Removed hero image container on mobile view. Changed the way input fields are displayed on mobile view.

Modifications

Header.module.css

  • added media query breakpoint to check for mobile screens, added responsive styling based on that breakpoint

Header.tsx

  • Changed logo in header.tsx file to be the transparent one. Slight issue is doesn't have shadow like in figma design, may have to redesign image for that

page.module.css

  • Prizes and Why should I refer section styling
  • hero image container removed for mobile view to prevent large empty space on smaller screens
  • Dynamic font resizing for landing page text

ReferralForm.module.css

  • changed input box layout

Testing Considerations

  • Dynamic font resizing on different screens

Screenshot 2025-05-06 at 12 36 02 AM

Pull Request Checklist

  • Code is neat, readable, and works
  • Comments are appropriate
  • The commit messages follows our guidelines
  • The developer name is specified
  • The summary is completed
  • Assign reviewers

@IvanA673 IvanA673 self-requested a review May 7, 2025 04:44
Copy link
Copy Markdown
Collaborator

@IvanA673 IvanA673 left a comment

Choose a reason for hiding this comment

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

Great work!

@IvanA673 IvanA673 merged commit 874e071 into develop May 7, 2025
1 of 4 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