Skip to content

Conversation

@adam-strzelec
Copy link
Contributor

@adam-strzelec adam-strzelec commented Jan 24, 2025

Description

Staged payment table UI fixes

Testing

  • Make sure, you have installed Staged Payments extension
  • Create staged payment
  • Go through the entire Staged payment flow
  • Compare table appearance for all cases

Diffs

New stuff

Desktop:

Create action:

Screenshot 2025-01-24 at 16 49 22

Performing action

Screenshot 2025-01-24 at 16 49 59

Payment

Screenshot 2025-01-24 at 16 50 35

Claim

Screenshot 2025-01-24 at 16 50 58

Mobile:

Create action:

Screenshot 2025-01-31 at 13 24 01

Performing action

Screenshot 2025-01-31 at 13 25 10

Payment

Screenshot 2025-01-31 at 13 25 57

Claim

Screenshot 2025-01-31 at 13 28 31

Resolves #4112

@adam-strzelec adam-strzelec self-assigned this Jan 24, 2025
@adam-strzelec adam-strzelec requested a review from a team as a code owner January 24, 2025 15:52
Copy link
Contributor

@mmioana mmioana left a 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
Screenshot 2025-01-29 at 09 32 10
Screenshot 2025-01-29 at 09 32 28
Screenshot 2025-01-29 at 09 32 32
Screenshot 2025-01-29 at 09 32 59
Screenshot 2025-01-29 at 09 33 01
Screenshot 2025-01-29 at 09 33 57
Screenshot 2025-01-29 at 09 34 19

On mobile when creating a new Staged payment action
Screenshot 2025-01-29 at 09 35 29

However on mobile once the action is created, you can see an empty column ❌
Screenshot 2025-01-29 at 09 35 47
And once you get to the release stage the pills and Pay now buttons are misaligned ❌
Screenshot 2025-01-29 at 09 35 04

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.

@adam-strzelec adam-strzelec requested a review from mmioana January 31, 2025 12:59
mmioana
mmioana previously requested changes Feb 3, 2025
Copy link
Contributor

@mmioana mmioana left a 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
Screenshot 2025-02-03 at 12 36 19
Screenshot 2025-02-03 at 12 36 27
Screenshot 2025-02-03 at 12 36 46
Screenshot 2025-02-03 at 12 36 51
Screenshot 2025-02-03 at 12 37 15
Screenshot 2025-02-03 at 12 37 22
Screenshot 2025-02-03 at 12 39 05
Screenshot 2025-02-03 at 12 39 33

Then I've tested with a single milestone
Screenshot 2025-02-03 at 12 33 58
Screenshot 2025-02-03 at 12 34 16
Screenshot 2025-02-03 at 12 34 47
Screenshot 2025-02-03 at 12 34 55
Screenshot 2025-02-03 at 12 35 25
Screenshot 2025-02-03 at 12 35 32
Screenshot 2025-02-03 at 12 35 45
Screenshot 2025-02-03 at 12 35 54

Issues

  1. 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 ❌
    Screenshot 2025-02-03 at 12 37 22
    While in Figma
    Screenshot 2025-02-03 at 12 40 00

  2. For a single milestone, on the Payment step there is an empty footer showing up on desktop
    Screenshot 2025-02-03 at 12 35 25

  3. For a single milestone, on the Payment step on mobile the footer shows the total Creds amount but is misaligned. Also the same issue with the Pay now link not being aligned as previously mentioned
    Screenshot 2025-02-03 at 12 35 32

  4. Also after releasing, there is an empty footer showing up on desktop
    Screenshot 2025-02-03 at 12 35 45

  5. And on mobile, the footer has the Creds not aligned
    Screenshot 2025-02-03 at 12 35 54

@adam-strzelec
Copy link
Contributor Author

@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.

@adam-strzelec adam-strzelec requested a review from mmioana February 3, 2025 18:18
@mmioana
Copy link
Contributor

mmioana commented Feb 4, 2025

@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
Maybe @melyndav can help with some information about whether we should show the footer or not if there is a single payment, though I would also assume in this case there is no need for a footer

@melyndav
Copy link

melyndav commented Feb 5, 2025

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!! 🌼🥳

@mmioana
Copy link
Contributor

mmioana commented Feb 6, 2025

Hey @adam-strzelec this PR needs a rebase 👀

@adam-strzelec adam-strzelec force-pushed the fix/16248357-staged-payment-table branch from 2eb27e7 to 7613361 Compare February 11, 2025 12:49
@adam-strzelec
Copy link
Contributor Author

@mmioana done

Copy link
Member

@arrenv arrenv left a 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:

image

image

Desktop looking good:

image

image

@adam-strzelec adam-strzelec merged commit 7538ea5 into master Mar 24, 2025
2 checks passed
@adam-strzelec adam-strzelec deleted the fix/16248357-staged-payment-table branch March 24, 2025 09:47
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Staged Payments Table UI Fixes

5 participants