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
| Comment | File | Size | Author |
|---|---|---|---|
| #3 | issue_2909898.patch | 156 KB | rigids |
Existing UI on the parent issue is not standardized
Nice UI for POS sidebar
| Comment | File | Size | Author |
|---|---|---|---|
| #3 | issue_2909898.patch | 156 KB | rigids |
Comments
Comment #2
rigids commentedComment #3
rigids commentedCreated PR and Patch attached
Comment #4
smccabe commentedHey @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
Comment #6
smccabe commentedAlso 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.