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(tablekit-modal-dialog): add maxWidth and maxHeight props #33

Merged
merged 2 commits into from
Aug 23, 2021

Conversation

irmir
Copy link
Contributor

@irmir irmir commented Aug 17, 2021

completes #31

📦 Published PR as canary version: Canary Versions

✨ Test out this PR locally via:

npm install @tablecheck/tablekit-input-button@1.0.2-canary.33.b6e8bc0d7e1ee8f160d12db9443d05a9ca0e88cf.0
npm install @tablecheck/tablekit-modal-dialog@1.0.1-canary.33.b6e8bc0d7e1ee8f160d12db9443d05a9ca0e88cf.0
# or 
yarn add @tablecheck/tablekit-input-button@1.0.2-canary.33.b6e8bc0d7e1ee8f160d12db9443d05a9ca0e88cf.0
yarn add @tablecheck/tablekit-modal-dialog@1.0.1-canary.33.b6e8bc0d7e1ee8f160d12db9443d05a9ca0e88cf.0

@irmir irmir added enhancement internal Changes only affect the internal API labels Aug 17, 2021
@github-actions
Copy link

github-actions bot commented Aug 17, 2021

Unit Test Results

    1 files    28 suites   30s ⏱️
191 tests 191 ✔️ 0 💤 0 ❌

Results for commit dd335fe.

♻️ This comment has been updated with latest results.

gazpachu
gazpachu previously approved these changes Aug 18, 2021
packages/modal-dialog/src/styled.ts Outdated Show resolved Hide resolved
@gazpachu
Copy link
Contributor

@irmir what's the status of this PR? I thought this would be ready to merge today

@irmir irmir force-pushed the add-maxWidth-maxHeight-modal-dialog branch 2 times, most recently from fb6fdb0 to cac5d94 Compare August 19, 2021 14:06
@irmir irmir force-pushed the add-maxWidth-maxHeight-modal-dialog branch 2 times, most recently from f4d2bfc to 5f2ee4c Compare August 20, 2021 09:30
@irmir irmir force-pushed the add-maxWidth-maxHeight-modal-dialog branch from 5f2ee4c to 5eea3be Compare August 20, 2021 10:10
@SimeonC SimeonC removed the internal Changes only affect the internal API label Aug 20, 2021
@irmir irmir added internal Changes only affect the internal API patch Increment the patch version when merged and removed internal Changes only affect the internal API labels Aug 20, 2021
@gazpachu gazpachu merged commit 7d06e79 into main Aug 23, 2021
@gazpachu gazpachu deleted the add-maxWidth-maxHeight-modal-dialog branch August 23, 2021 04:07
@tablecheck-public-automation
Copy link
Collaborator

🚀 PR was released in @tablecheck/tablekit-modal-dialog@1.0.1 🚀

@tablecheck-public-automation tablecheck-public-automation added the released This issue/pull request has been released. label Aug 23, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
patch Increment the patch version when merged released This issue/pull request has been released.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants