lightning-browser-extension icon indicating copy to clipboard operation
lightning-browser-extension copied to clipboard

UX/UI: align "Send" screens

Open escapedcat opened this issue 3 years ago • 2 comments

Is your feature request related to a problem?

Currently all "Send" screens look similar but different. This is partly related to different info that is available to be displayed and partly to code-inconsistencies.

Describe the solution you'd like

A general solution which can be used for all these screens. This will hopefully:

  • align the UI/UX
  • reduce code because we can re-use the same "component" for all screens

Related to:

  • https://github.com/fiatjaf/lnurl-rfc/blob/luds/09.md

Are you working on this?

No

Screenshots

  1. Keysend (payment directly to a node)
    • Start (non-prompt)
    • Success (non-prompt only)
  2. Withdraw
    • Start (prompt)
    • Start (non-prompt)
    • Success (non-prompt only)
  3. Channel
    • Start (prompt)
    • Start (non-prompt)
    • Success (non-prompt only) - nodeId can be rather long - k1 is "random or non-random string to identify the user's LN WALLET when using the callback URL"
  4. Auth
    • Start (prompt)
    • Start (non-prompt)
    • Success (non-prompt only)
  5. Confirm payment
    • Start (prompt)
    • Start (non-prompt)
    • Success (non-prompt only)
  6. LNURLPay
    • Start (prompt)
    • Start (non-prompt) - -
    • Success (non-prompt only)

escapedcat avatar Sep 01 '22 10:09 escapedcat

Relates to #1197

escapedcat avatar Sep 08 '22 09:09 escapedcat

Current draft ideas

  • No image for actions pasted directly via "Send"
  • Header same size but with no image
  • Keysend "long" key will get cut off via "..."
  • For Popup/Tab success/error screens use desgins from related ticket:
    • image (without the budget info)
    • image

Designs can be found here:

Figma Designs (make sure to select "for Dev" in the left navigation): https://www.figma.com/file/HXIxAFG9OUCcr2EJMJ6JMm/1-Click-Pay?node-id=321%3A5678

One more point:

Add header to LNURLPay screen: "Send"

escapedcat avatar Sep 08 '22 09:09 escapedcat