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

feat(Flyout): Added optional background prop #2627

Merged
merged 16 commits into from
Dec 15, 2022
Merged

Conversation

zippyzow
Copy link
Contributor

@zippyzow zippyzow commented Dec 12, 2022

Overview

Allow gamut Flyout component to have changeable background color.

PR Checklist

Testing Instructions

  1. Open storybook link
  2. Navigate to Flyout section
  3. Scroll down to list of props and see new bg prop
  4. Change that value to some color
  5. Click "Tell me more" above and see this color in the flyout

PR Links and Envs

Repository PR Link PR Env
Monolith Monolith PR Monolith Env
Monorepo Portal PR Portal Env

@nx-cloud
Copy link

nx-cloud bot commented Dec 12, 2022

☁️ Nx Cloud Report

CI is running/has finished running commands for commit b8a43e9. As they complete they will appear below. Click to see the status, the terminal output, and the build insights.

📂 See all runs for this branch


✅ Successfully ran 4 targets

Sent with 💌 from NxCloud.

@zippyzow zippyzow changed the title test feat(Flyout): Added optional background prop Dec 12, 2022
@codecademydev
Copy link
Collaborator

📬Published Alpha Packages:

@codecademy/[email protected]
@codecademy/[email protected]
@codecademy/[email protected]
@codecademy/[email protected]
@codecademy/[email protected]

@saghdaey saghdaey marked this pull request as ready for review December 14, 2022 22:04
@saghdaey saghdaey requested a review from a team as a code owner December 14, 2022 22:04
@codecademydev
Copy link
Collaborator

🚀 Styleguide deploy preview ready!

https://639a48b4d4946a60593747ea--gamut-preview.netlify.app

Deploy Logs

Copy link
Contributor

@dreamwasp dreamwasp left a comment

Choose a reason for hiding this comment

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

🔥

@saghdaey saghdaey merged commit ad14636 into main Dec 15, 2022
@saghdaey saghdaey deleted the hr-disc-1007-flyout-bg branch December 15, 2022 16:34
jakemhiller added a commit that referenced this pull request Dec 16, 2022
* origin/main:
  chore(release): publish
  chore: update local node version to v16 [WEB-2205]
  chore(release): publish
  refactor(ScoreSummary): make ScoreSummary responsive
  chore(release): publish
  feat(Flyout): Added optional background prop (#2627)
  chore(release): publish
  fix: update width of page alert to match page contents
jakemhiller added a commit that referenced this pull request Jan 6, 2023
* main: (30 commits)
  chore(release): publish
  feat(FormGroupLabel, GridForm): assorted embedded user-form features
  chore(release): publish
  fix(Flyout): move background color to fix overflow
  chore(release): publish
  chore: update local node version to v16 [WEB-2205]
  chore(release): publish
  refactor(ScoreSummary): make ScoreSummary responsive
  chore(release): publish
  feat(Flyout): Added optional background prop (#2627)
  chore(release): publish
  fix: update width of page alert to match page contents
  chore(release): publish
  chore(AppHeader)!: Remove bookmarks and favorites
  chore(release): publish
  fix(RatingsBar): Conditional box shadow (hide if 0%)
  chore(release): publish
  refactor(ScoreSummary): make trackSlug and tracking optional in ScoreSummary
  chore(release): publish
  feat(Footer): rename Pro Membership link
  ...
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