-
Notifications
You must be signed in to change notification settings - Fork 14
fix: staged payment table content position #4181
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
Conversation
mmioana
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Hey @adam-strzelec really great start on tackling this issue! 🥇
On desktop the content remains in the same position







On mobile when creating a new Staged payment action

However on mobile once the action is created, you can see an empty column ❌

And once you get to the release stage the pills and Pay now buttons are misaligned ❌

You can find the designs here
Also @adam-strzelec please add in the testing steps that the Staged payments extension needs to be installed, as it might not be very clear.
mmioana
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Hey @adam-strzelec thank you for your updates! I'll leave the issues I've spotted at the end of the review 😄
I've tested one staged payment with multiple milestones








Then I've tested with a single milestone








Issues
-
The issue I have spotted when having multiple milestones is before releasing the payments as the links on mobile are still misaligned for each milestone row ❌

While inFigma

-
For a single milestone, on the
Paymentstep there is an empty footer showing up ondesktop

-
For a single milestone, on the
Paymentstep onmobilethe footer shows the totalCredsamount but is misaligned. Also the same issue with thePay nowlink not being aligned as previously mentioned

-
Also after releasing, there is an empty footer showing up on
desktop

|
@mmioana I haven't found a case for a single milestone in Figma, but I assume that when we have only one milestone, the footer shouldn't be displayed, just like it is when I create an action. |
Hey @adam-strzelec. You're right, I couldn't find it in the design system either https://www.figma.com/design/l1dOM5qiQYwF0ElvKDqqjg/Design-System---Colony-v3?node-id=1782-172609&t=640u5BCeqcHJKkDR-0 |
|
Hey guys @mmioana @adam-strzelec! For single table rows on mobile for Staged Payments, we can leave out the footer. This keeps it consistent with the Advanced Payment. Thanks for checking!! 🌼🥳 |
|
Hey @adam-strzelec this PR needs a rebase 👀 |
2eb27e7 to
7613361
Compare
|
@mmioana done |
arrenv
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@adam-strzelec All looks great, thank you for the updates.
Mobile looking good:
Desktop looking good:





Description
Staged payment table UI fixes
Testing
Staged PaymentsextensionDiffs
New stuff ✨
Desktop:
Create action:
Performing action
Payment
Claim
Mobile:
Create action:
Performing action
Payment
Claim
Resolves #4112