Steps to Reproduce

  1. Add a component that includes a multi-value prop of type Integer or Number.
  2. Enter multiple values in the prop field (e.g., 1, 2, 3, 4).
  3. Attempt to reorder the values (e.g., drag and move 4 to the first position).
  4. Observe the order after the reordering action is completed.

Actual Result
The reordered values do not persist. The list resets back to its original order automatically.

Expected Result
The reordered values should persist in the updated order after the user rearranges them.

CommentFileSizeAuthor
#11 Numner-integer functioning.gif407.45 KBvipin.mittal18

Issue fork canvas-3582219

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

vipin.mittal18 created an issue. See original summary.

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

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

chandu7929’s picture

Status: Needs work » Needs review
narendrar’s picture

Assigned: chandu7929 » Unassigned
tim.plunkett’s picture

Status: Needs review » Needs work
Issue tags: +Needs tests

Existing test coverage should have caught this but it didn't, Naren suggests adding more wait calls.

narendrar’s picture

Status: Needs work » Needs review
Issue tags: -Needs tests
Related issues: +#3577946: Remove Canvas Dev Mode flag for Multi-Value Props support

Existing multivalue entity fields tests are expanded here. But component tests for Canvas page to cover multivalue props are getting added as a part of #3577946: Remove Canvas Dev Mode flag for Multi-Value Props support and I verified with @Utkarsh_33 that adding this fix resolved issue related to drag/drop for Text/Number/Integer fields which were not working without this fix earlier.

vipin.mittal18’s picture

Status: Needs review » Reviewed & tested by the community
StatusFileSize
new407.45 KB

MR !827 fixes the reordering issue on Integer Number and Link (even not part of this ticket) prop types.

reordering

tim.plunkett’s picture

Status: Reviewed & tested by the community » Fixed

Merged, thanks!

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.

Status: Fixed » Closed (fixed)

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