Skip to content

fix: pos screen ui ux #47680

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

Merged
merged 13 commits into from
May 23, 2025
Merged

fix: pos screen ui ux #47680

merged 13 commits into from
May 23, 2025

Conversation

diptanilsaha
Copy link
Member

@diptanilsaha diptanilsaha commented May 22, 2025

Changes include:

  • Trigger the submission of the invoice when clicking the 'Save' button after filling out the mandatory fields in the Additional Information Dialog. This eliminates the need for an additional 'Complete Order' click.
    Demo:
additional-info-dialog.mov
  • Moved the Menu Items to Action buttons.

Before:
pos-menu

After:
pos-action-buttons

  • Replaced text buttons for Fullscreen and Minimize Screen with Icon buttons.
  • Replaced 'Save as Draft' menu Item with 'New Invoice' button. On clicking 'New Invoice', POS will load a new Invoice. If some items are added to the cart, POS will ask for confirmation to save or discard the changes.

image

Users can configure to always save or discard changes to an invoice before loading a new one from the POS Profile.

image

  • Hide Item Selector Panel if Items Display is visible.

Before:
image

After:
image

  • Added hovering effect on the Numpad
numpad-hovering.mov
  • Rearranged the filter fields in the same row on Recent Order

image

  • Rearranged the Invoice Name and Customer Name on Order List Items on Recent Order

image

  • Fixed the issue with Recent Order toggling after invoice submission. Earlier, it used to open up the Item Selector and Item cart instead of the Summary of the Invoice
  • If the user clicks New Invoice when the Recent Order List is toggled on, POS will toggle off the Recent Order List and open the Item Selector and Item Cart.

@github-actions github-actions bot added the needs-tests This PR needs automated unit-tests. label May 22, 2025
@diptanilsaha diptanilsaha marked this pull request as ready for review May 22, 2025 08:43
@diptanilsaha diptanilsaha requested a review from nabinhait May 22, 2025 08:46
@diptanilsaha diptanilsaha marked this pull request as draft May 22, 2025 10:23
@diptanilsaha diptanilsaha marked this pull request as ready for review May 22, 2025 11:19
@diptanilsaha diptanilsaha requested a review from nabinhait May 23, 2025 06:52
@nabinhait nabinhait merged commit 8911568 into frappe:develop May 23, 2025
18 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
needs-tests This PR needs automated unit-tests.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants