Skip to content

fix(Browser): add initial item locking functionality#364

Open
celikerde wants to merge 1 commit into
11.xfrom
bugfix/current-addon-should-not-be-removed
Open

fix(Browser): add initial item locking functionality#364
celikerde wants to merge 1 commit into
11.xfrom
bugfix/current-addon-should-not-be-removed

Conversation

@celikerde

@celikerde celikerde commented Jun 8, 2026

Copy link
Copy Markdown
Collaborator
  • Introduced a new prop lockInitialItems to prevent removal of pre-existing items in the selection.
  • Implemented logic to capture and manage locked item IDs upon component initialization.
  • Enhanced item selection and clickability checks to respect locked status, ensuring existing items cannot be toggled or removed.

Checklist

  • I have read the CONTRIBUTING document.
  • My code follows the code style of this project.
  • My change requires a change to the documentation.
  • I have updated the documentation accordingly.
  • I have added tests to cover my changes.
  • All new and existing tests passed.

Types of changes

  • feat: A new feature
  • fix: A bug fix
  • docs: Documentation only changes
  • style: Changes that do not affect the meaning of the code (white-space, formatting, etc)
  • refactor: A code change that neither fixes a bug nor adds a feature
  • perf: A code change that improves performance
  • test: Adding missing tests or correcting existing tests
  • build: Changes that affect the build system or external dependencies
  • ci: Changes to our CI configuration files and scripts
  • chore: Other changes that don't modify src or test files
  • revert: Reverts a previous commit

Description

Related Issue

- Introduced a new prop `lockInitialItems` to prevent removal of pre-existing items in the selection.
- Implemented logic to capture and manage locked item IDs upon component initialization.
- Enhanced item selection and clickability checks to respect locked status, ensuring existing items cannot be toggled or removed.
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.

1 participant