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

fix: staged payment table content position #4181

Open
wants to merge 3 commits into
base: master
Choose a base branch
from

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
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 👀

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
3 participants