Skip to content

Comments

feat(onboarding): add gradient background and onboarding quiz content#15

Merged
magali-la merged 3 commits intomainfrom
feat/onboarding
Oct 20, 2025
Merged

feat(onboarding): add gradient background and onboarding quiz content#15
magali-la merged 3 commits intomainfrom
feat/onboarding

Conversation

@magali-la
Copy link
Collaborator

🎯Type of Change:

  • Feat ✨: New feature or functionality.

This PR...

Adds the desktop layout to the onboarding page, with half-page gradient, and onboarding quiz content in the container.

📝 What I Did:

  • Added the half-page gradient background for desktop view
  • Added container card and shadow classes
  • Added percent-based width and height to container by using vh and vw instead of pixels

🧪 How to Test:

  1. Navigate to /onboarding path
  2. Check in inspect that the outer card container is responsive as a percent of the viewport height and viewport width
  3. Note: content is not yet responsive, buttons and text need to be scaled down

🤔 What I learned:

Learned how to get tailwind background gradients to stop mid-page and introduce another color for the other half. Also learned it's sometimes better to avoid including tailwind arbitrary pixel values and opt for vh and vw values to ease responsiveness.

@netlify
Copy link

netlify bot commented Oct 19, 2025

Deploy Preview for mint-chip ready!

Name Link
🔨 Latest commit 8e13129
🔍 Latest deploy log https://app.netlify.com/projects/mint-chip/deploys/68f527204d3825000820fd58
😎 Deploy Preview https://deploy-preview-15--mint-chip.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify project configuration.

Copy link
Collaborator

@nkabembo nkabembo left a comment

Choose a reason for hiding this comment

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

Responsiveness is working fine.
Query: I noticed the padding of the card was small and mobile view spacing is not change. I am guessing it is not within the scope of this commit?

@magali-la
Copy link
Collaborator Author

Responsiveness is working fine. Query: I noticed the padding of the card was small and mobile view spacing is not change. I am guessing it is not within the scope of this commit?

correct, I'll submit another PR with the column layout on mobile and full bg, once i get the button sizes responsive so they can fit within the container

@CMolinaBetancourt
Copy link
Collaborator

The onboarding page looks great, it is just pending the responsiveness you mentioned.

@magali-la magali-la merged commit 496460c into main Oct 20, 2025
5 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