Skip to content

[WIP] Improve include/exclude list value management in Add/Edit modals - #3263

Draft
zackcl wants to merge 1 commit into
devfrom
feature/3256-list-value-management-modal
Draft

[WIP] Improve include/exclude list value management in Add/Edit modals#3263
zackcl wants to merge 1 commit into
devfrom
feature/3256-list-value-management-modal

Conversation

@zackcl

@zackcl zackcl commented Aug 3, 2026

Copy link
Copy Markdown
Collaborator

Resolves #3256

Summary

Implements the modal-based value management design proposed in #3256.

  • Replaces the chip-based editor with a searchable, row-based table
  • Supports adding, editing, deleting, importing, and exporting values
  • Keeps changes staged until the user selects Create or Save
  • Uses the same behavior across Add/Edit Include List and Exclude List modals
  • Preserves existing values when editing partially loaded lists

Key behavior

  • Values can be entered manually, pasted, or imported from CSV
  • Imported values are placed in the Values field for review
  • Selecting Add (+) or pressing Enter commits pending values to the table
  • Search is always available and the table header displays the total count
  • Export includes the complete list regardless of the active search
  • The table displays up to four rows before becoming scrollable

Intentional differences from #3256

Based on implementation review and UX iteration:

  • The Description field was removed to provide more space for value management
  • The value count was moved to the Value table header
  • Enter or Add (+), rather than a comma, commits pending values
  • CSV imports no longer provide Add/Replace actions or modify the table immediately
  • Search remains visible for lists of all sizes

Screenshot

Screenshot 2026-08-03 at 5 37 34 PM

Screen recording

larger.list.modal.mov

@zackcl
zackcl requested review from bcb37 and danoswaltCL August 3, 2026 21:38
@zackcl zackcl self-assigned this Aug 3, 2026
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.

Improve include/exclude list value management in the Add/Edit List modal

1 participant