woocommerce-plugin icon indicating copy to clipboard operation
woocommerce-plugin copied to clipboard

Phase 1: Upgrade JS Checkout to Modular (Part 1/2)

Open thisisayush opened this issue 3 years ago • 4 comments

References #283

Part 1 of JS Modular Checkout:

  • Updates to use General Components instead of custom classes, unless necessary.
  • Uses the new Modular UI Design
  • Font Icons Added

Screenshots:

Lite Mode

JS Checkout

image

No-JS Checkout

image

Strorefront

JS Checkout

image

No-JS Checkout

image

Asrta

JS Checkout

image

No JS Checkout

image

Hestia

JS Checkout

image

No JS Checkout

image

Part 2: Make Refresh via AJAX

To be Done

thisisayush avatar Oct 12 '22 08:10 thisisayush

Note that now we have an additional complexity of changing bitcoin to bitcoin cash in case of BCH checkout What I recommended is to completely remove the bitcoin row and move the 6000USD to the Order# row https://nimb.ws/twfA9Z This will give more vertical space and things will look compact. Feel free to think about this

shivaenigma avatar Oct 13 '22 05:10 shivaenigma

@shivaenigma Right now it's not hardcoded, a styling is applied to the "name" of the crypto chosen: https://github.com/blockonomics/woocommerce-plugin/pull/293/files#diff-8d81ebce00651fad5f0a421bc7d60d514e8ca6bd12cc9fc434ab5e46dab827ecR52

thisisayush avatar Oct 13 '22 10:10 thisisayush

What I recommended is to completely remove the bitcoin row and move the 6000USD to the Order# row https://nimb.ws/twfA9Z This will give more vertical space and things will look compact. Feel free to think about this

@thisisayush let us do this. I will this will make the UI more compact and look better

blockonomics avatar Oct 27 '22 06:10 blockonomics

image

How it looks with Moving the Order ID and Amount in the same row. One thing to look for: Handling large order IDs as mentioned by @DarrenWestwood

thisisayush avatar Oct 28 '22 09:10 thisisayush

Full pull request with all changes: https://github.com/blockonomics/woocommerce-plugin/pull/301

DarrenWestwood avatar Nov 16 '22 09:11 DarrenWestwood