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

Cross position in subscription modal fix #1936

Merged
merged 7 commits into from
Feb 14, 2022

Conversation

tanmoyAtb
Copy link
Contributor

We had a small bug on the subscription modal with some screen sizes. Here's a screenshot.

Screenshot 2022-02-11 at 8 48 28 PM

This PR fixes the problem.

@render
Copy link

render bot commented Feb 11, 2022

@render
Copy link

render bot commented Feb 11, 2022

@render
Copy link

render bot commented Feb 11, 2022

@github-actions github-actions bot added the Type: Bug Fix Added to PRs if they are addressing a bug label Feb 11, 2022
@tanmoyAtb tanmoyAtb requested review from Tbaut, FSM1 and asnaith February 11, 2022 14:54
@tanmoyAtb tanmoyAtb enabled auto-merge (squash) February 11, 2022 14:55
Copy link
Collaborator

@Tbaut Tbaut left a comment

Choose a reason for hiding this comment

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

Nice, while we're here, I noticed there's the same css used in PayInvoiceModal.tsx with the same bug, fixed by the same 480 instead of undefined, can you add it there too?

@FSM1
Copy link
Contributor

FSM1 commented Feb 11, 2022

This change does mess with the padding that the modal usually has:
Before:
image

After:
image

@asnaith
Copy link
Contributor

asnaith commented Feb 11, 2022

Looks good 🔥

I will re-test when crypto payment modal gets the same fix applied

@tanmoyAtb
Copy link
Contributor Author

All right, this should be good now. 👍

@tanmoyAtb tanmoyAtb requested review from Tbaut and asnaith February 14, 2022 04:28
@Tbaut
Copy link
Collaborator

Tbaut commented Feb 14, 2022

I'll let @FSM1 confirm it's looking good

Copy link
Contributor

@FSM1 FSM1 left a comment

Choose a reason for hiding this comment

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

Works well 🎉

@tanmoyAtb tanmoyAtb merged commit f27606c into dev Feb 14, 2022
@tanmoyAtb tanmoyAtb deleted the fix/cross-button-in-subscription-plan-modal branch February 14, 2022 14:06
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Type: Bug Fix Added to PRs if they are addressing a bug
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants