Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Fix website redesign hero.svelte #938

Merged
merged 5 commits into from
Oct 21, 2023
Merged

Fix website redesign hero.svelte #938

merged 5 commits into from
Oct 21, 2023

Conversation

karnikaavelumani
Copy link
Collaborator

@karnikaavelumani karnikaavelumani commented Oct 18, 2023

The hero component had a couple errors so this PR is to fix those.

Changes

  • Removed +layout.svelte (removes duplicate footer)
  • Adjusted margins
  • Removed brute forced code (clamp and max-width screen size)
  • Fix multicolor text

Screenshots

image
image
image

@netlify
Copy link

netlify bot commented Oct 18, 2023

Deploy Preview for acmcsuf ready!

Name Link
🔨 Latest commit 58a51cf
🔍 Latest deploy log https://app.netlify.com/sites/acmcsuf/deploys/653368af7475770007c22288
😎 Deploy Preview https://deploy-preview-938--acmcsuf.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 site configuration.

@karnikaavelumani
Copy link
Collaborator Author

@EthanThatOneKid Can you properly check and see if the CSS is appropriately adjusting to all screen sizes? In comparison to the other components, I am noticing the behavior of the hero.svelte page is different from the others.

@tomasohCHOM I noticed why the multicolor text was not working originally. The color spans across the entire screen in tablet mode so we need to add a width to fix it. However, I could not center it properly. I added you for review if you would like to take a crack at it!

@tomasohCHOM
Copy link
Collaborator

@karnikaavelumani I will take a look at it right now and tell you how it goes!

@karnikaavelumani karnikaavelumani marked this pull request as ready for review October 18, 2023 16:15
Copy link
Owner

@EthanThatOneKid EthanThatOneKid left a comment

Choose a reason for hiding this comment

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

Looks pretty much good to go!

However this caught my eye. Why do we lose so many lines here?
9036E94D-B8EA-4910-9FC0-2E95B6CCF35E

@karnikaavelumani karnikaavelumani enabled auto-merge (squash) October 21, 2023 06:00
@karnikaavelumani karnikaavelumani merged commit ed0c934 into main Oct 21, 2023
5 checks passed
@karnikaavelumani karnikaavelumani deleted the fix/hero-issues branch October 21, 2023 06:00
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.

None yet

3 participants