Skip to content

Settings: Sortable multi-selection fixes#2092

Open
filipvnencak wants to merge 2 commits into
developfrom
2004-settings-sortable-multiselect-fixes
Open

Settings: Sortable multi-selection fixes#2092
filipvnencak wants to merge 2 commits into
developfrom
2004-settings-sortable-multiselect-fixes

Conversation

@filipvnencak

Copy link
Copy Markdown
Collaborator

Description of changes

Moves the ordered-multiselect reordering into a "Set order" dialog next to the dropdown. The dropdown now lists selected items in their chosen order.

Technical details

  • New OrderSelectionDialog.tsx holds the drag-to-reorder list; header reads "Selection order".
  • OrderedListWidget.tsx reduced to dropdown + "Set order" button; dropdown options sorted selected-first via orderedOptions memo.
  • SelectWidget.tsx: gate on widget === 'sortable_multiselect' only; dropped the applications_profiles id regex fallback.

Additional context

@filipvnencak filipvnencak self-assigned this Jun 22, 2026
@filipvnencak filipvnencak linked an issue Jun 22, 2026 that may be closed by this pull request
4 tasks
@Innders Innders self-requested a review June 22, 2026 13:03

@Innders Innders left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

  • widget undefined on schema
  • dialog max height should be 80-90vh to expand for lots of items.
Image

Comment on lines 135 to 136
const isSortableMultiselect = widget === 'sortable_multiselect'

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

widget always seems to be undefined and does not exist on props.schema. Consult @martastain on how to do this.

Copy link
Copy Markdown
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Which version of ayon-applications do you have??

@filipvnencak filipvnencak requested a review from Innders June 23, 2026 06:27
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Settings: Sortable multi-selection fixes

2 participants