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

[PLAY-555] Fixed Body Content disappearing behind Footer for Dialog #2283

Merged
merged 3 commits into from
Feb 2, 2023

Conversation

nidaqg
Copy link
Contributor

@nidaqg nidaqg commented Jan 25, 2023

Screens

Before:

Screenshot 2023-01-25 at 4 01 01 PM

After:

Screenshot 2023-01-25 at 3 54 47 PM

Breaking Changes

No breaking changes

Runway Ticket URL

Runway Story

How to test this

Click on full height variant in 'scrollable' doc example. Scroll to the bottom to see text is no longer covered by footer. Test both Rails and React.

Checklist:

  • LABELS Add a label: enhancement, bug, improvement, new kit, deprecated, or breaking. See Changelog & Labels for details.
  • DEPLOY Please add the Milano label when you are ready for a review.
  • SCREENSHOT Please add a screen shot or two.
  • SPECS Please cover your changes with specs.
  • READ DOCS Please make sure you have read and understand the Playbook Release Process

@nidaqg nidaqg added bug Fixes to issues discovered in Playbook (USED IN CHANGELOG) milano 20 MAX - Deploy this PR to a review environment via Milano labels Jan 25, 2023
@nidaqg nidaqg self-assigned this Jan 25, 2023
@nidaqg nidaqg requested a review from a team as a code owner January 25, 2023 21:02
@app-milano app-milano bot temporarily deployed to pr2283 January 25, 2023 21:13 Inactive
@app-milano app-milano bot temporarily deployed to pr2283 February 2, 2023 14:32 Inactive
@nidaqg nidaqg added the Ready for Release merged to master, ready for a versioned released label Feb 2, 2023
@nidaqg nidaqg merged commit c270f04 into master Feb 2, 2023
@nidaqg nidaqg deleted the Play-555-dialog-sticky-footer-issue branch February 2, 2023 17:59
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Fixes to issues discovered in Playbook (USED IN CHANGELOG) milano 20 MAX - Deploy this PR to a review environment via Milano Ready for Release merged to master, ready for a versioned released
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants