Describe your bug or feature request.

We’ve discussed holistic changes to the “Add order item” modal workflow, which I think we’d summarize:

Do not float the “Add new order item” button to the right of the search bar. It should also just be labeled “Next”.

If possible via simple JS, let’s allow an “enter” keypress while in the autocomplete textfield to click the “Next” button, so I can just type in my SKU, select it, and hit enter to proceed.

Use a “continue” color (e.g., green) to identify partial steps toward full element submission.

For the next phase, where the quantity textfield and override unit price elements are shown, the button should be labeled “Add” and should also use the green continue color.

The “Remove” operations button for a staged order item should be fixed to just use the secondary button styling. This is not data destructive, just zeroing out staged data, so it does not need red.

When you edit a staged order item, the button text to stage changes should be labeled “Update”.

The “Add items” button should be “Save” and ideally would be disabled until a save would actually result in the creation of new order items. Otherwise, the admin should just cancel.

Switch to a regular outlined button for “Cancel.”

Issue fork commerce-3568785

Command icon Show commands

Start within a Git clone of the project using the version control instructions.

Or, if you do not have SSH keys set up on git.drupalcode.org:

Comments

tbkot created an issue. See original summary.

jsacksick’s picture

Status: Active » Needs work

Let's not depend on JQuery if possible. Core is moving away from JQuery, so as our recent JS.

  • jsacksick committed 98361c97 on 3.x authored by tbkot
    feat: #3568785 Clarify the order of operations in the add order item...
jsacksick’s picture

Status: Needs work » Fixed

Now that this issue is closed, review the contribution record.

As a contributor, attribute any organization that helped you, or if you volunteered your own time.

Maintainers, credit people who helped resolve this issue.

majmunbog made their first commit to this issue’s fork.

  • jsacksick committed a2cc7ebb on 3.x authored by majmunbog
    feat: #3568785 heming and styling updates, message text update, no-print...

Status: Fixed » Closed (fixed)

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