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

Overlay updates for Alpha::Dialog #2225

Merged
merged 14 commits into from
Aug 30, 2022
Merged

Overlay updates for Alpha::Dialog #2225

merged 14 commits into from
Aug 30, 2022

Conversation

langermank
Copy link
Contributor

@langermank langermank commented Aug 29, 2022

What are you trying to accomplish?

Size enhancements and bug fixes for Alpha::Dialog primer/view_components#1214

  • Adds logic for a new size param replacing height/width
  • Bottom sheets 100vw
  • Side sheets 100vh

What should reviewers focus on?

There should be no breaking changes here. Keep an eye out! 🤞

Can these changes ship as is?

  • Yes, this PR does not depend on additional changes. 🚢

@changeset-bot
Copy link

changeset-bot bot commented Aug 29, 2022

🦋 Changeset detected

Latest commit: 9c23d7d

The changes in this PR will be included in the next version bump.

This PR includes changesets to release 1 package
Name Type
@primer/css Patch

Not sure what this means? Click here to learn what changesets are.

Click here if you're a maintainer who wants to add another changeset to this PR

@github-actions github-actions bot temporarily deployed to Storybook Preview August 29, 2022 03:57 Inactive
@langermank langermank marked this pull request as ready for review August 30, 2022 02:52
@langermank langermank requested a review from a team as a code owner August 30, 2022 02:52
@langermank langermank requested a review from simurai August 30, 2022 02:52
@langermank langermank temporarily deployed to github-pages August 30, 2022 02:58 Inactive
@github-actions github-actions bot temporarily deployed to Storybook Preview August 30, 2022 02:58 Inactive
src/overlay/overlay.scss Outdated Show resolved Hide resolved
@github-actions github-actions bot temporarily deployed to Storybook Preview August 30, 2022 18:24 Inactive
@github-actions github-actions bot temporarily deployed to Storybook Preview August 30, 2022 18:25 Inactive
@langermank langermank temporarily deployed to github-pages August 30, 2022 18:26 Inactive
@github-actions github-actions bot temporarily deployed to Storybook Preview August 30, 2022 18:27 Inactive
@github-actions github-actions bot temporarily deployed to Storybook Preview August 30, 2022 18:45 Inactive
@github-actions github-actions bot temporarily deployed to Storybook Preview August 30, 2022 18:45 Inactive
@github-actions github-actions bot temporarily deployed to Storybook Preview August 30, 2022 18:48 Inactive
@langermank langermank merged commit 56ea4ab into main Aug 30, 2022
@langermank langermank deleted the overlay-size branch August 30, 2022 19:45
@primer-css primer-css mentioned this pull request Aug 30, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants