Problem/Motivation

Existing UI on the parent issue is not standardized

Proposed resolution

Nice UI for POS sidebar

Remaining tasks

User interface changes

API changes

Data model changes

CommentFileSizeAuthor
#3 issue_2909898.patch156 KBrigids

Comments

rakesh.gectcr created an issue. See original summary.

rigids’s picture

Assigned: Unassigned » rigids
rigids’s picture

StatusFileSize
new156 KB

Created PR and Patch attached

smccabe’s picture

Hey @rigids,

The keypad on the payment page does come from the keypad module that is part of the pos module. The problem was that the library was not being attached to the form. I added a line attaching it as well as cleaned up the library file.

We should probably update the keypad library so it will attach automatically, but I've opened a sub-issue for that as it instead related to this issue and also low priority. #2917778: Have keypad automatically attach library

  • smccabe committed b1aabec on 8.x-2.x authored by rigids
    Issue #2909898 by rigids, smccabe: UI for POS sidebar
    
smccabe’s picture

Status: Active » Fixed

Also moved most of the keypad SASS back into the keypad module, I only left the payment specific stuff, although I am suspicious that could be moved as well, although I was going to wait until we use the keypad option more often.

Status: Fixed » Closed (fixed)

Automatically closed - issue fixed for 2 weeks with no activity.