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

Adds gradients for breakpoints larger than xs #795

Merged
merged 1 commit into from
Jan 25, 2024

Conversation

shc314
Copy link
Contributor

@shc314 shc314 commented Jan 25, 2024

READY FOR REVIEW

Summary

  • Fixes the hero gradient for larger breakpoints

Review By (Date)

  • When possible

Review Tasks

Setup tasks and/or behavior to test

  1. Check out this branch
  2. Navigate to or create a page with a hero component that has an image.
  3. Change the Hero Gradient to one of the choices other than Default: Brick, Palo Alto Dark, or White.
  4. Verify that:
    At the xs breakpoint, the gradient starts at the top with the specified color at 30% opacity and then goes to #181D1C 100% opacity at the bottom.
    For all widths larger than xs, the gradient starts at #181D1C 0% opacity at the top and then goes to the specified color 100% opacity at the bottom.

Associated Issues and/or People

  • JIRA ticket: HOM-1
  • @William-Misiaszek

Comment on lines +138 to +140
brick: 'xs:su-from-brick su-to-brick/30',
'palo-alto-dark': 'xs:su-from-palo-alto-dark su-to-palo-alto-dark/30',
white: 'xs:su-from-white su-to-white/30',
Copy link
Member

@sherakama sherakama Jan 25, 2024

Choose a reason for hiding this comment

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

@yvonnetangsu

You removed the xs from these here: 048f9a3#r1443322032 in #710

But they are not working as intended on the current site. This is to fix that. Can you look at this again please?

Copy link
Member

Choose a reason for hiding this comment

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

Going to review this now - thanks for the ping @sherakama !

Copy link
Member

@yvonnetangsu yvonnetangsu left a comment

Choose a reason for hiding this comment

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

I see - so using xs: to increase CSS specificity to override the default su-from-saa-black gradient start color 🤔 We usually do a conditional check to add a class instead of counting on the override, but this works so I'm going to approve! 😄

@yvonnetangsu yvonnetangsu merged commit b4674ed into dev Jan 25, 2024
6 checks passed
@yvonnetangsu yvonnetangsu deleted the bug/HOM-1_fix-hero-banner-gradients branch January 25, 2024 23:07
@yvonnetangsu yvonnetangsu mentioned this pull request Jan 25, 2024
@yvonnetangsu
Copy link
Member

It is live :)
https://github.com/SU-SWS/saa_alumni/releases/tag/4.3.7

@shc314
Copy link
Contributor Author

shc314 commented Jan 25, 2024

Thank you @yvonnetangsu and @sherakama !

@yvonnetangsu
Copy link
Member

Thank you for the fix @shc314 ! Sorry I messed it up 😂

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