Skip to content

fix: Tag protection now works correctly in GridTextEditor#2

Open
evitolo wants to merge 1 commit intoev-textfrom
claude/review-changes-mkl71um3gv6cvvz5-WMeBE
Open

fix: Tag protection now works correctly in GridTextEditor#2
evitolo wants to merge 1 commit intoev-textfrom
claude/review-changes-mkl71um3gv6cvvz5-WMeBE

Conversation

@evitolo
Copy link

@evitolo evitolo commented Jan 19, 2026

Fixed bug where tag protection setting was not working because the logic was added to EditableGridTextEditor but the delegate creates GridTextEditor instances for target cells.

Changes:

  • Added tag protection attributes to GridTextEditor.init
  • Copied tag protection methods (_extract_tags, _tags_are_intact, _on_text_changed_with_tag_protection) to GridTextEditor
  • Added focusInEvent/focusOutEvent handlers to GridTextEditor
  • Ensured delegate is recreated when protection setting changes

The tag protection feature now properly prevents tag modification when enabled.

Fixed bug where tag protection setting was not working because the logic
was added to EditableGridTextEditor but the delegate creates GridTextEditor
instances for target cells.

Changes:
- Added tag protection attributes to GridTextEditor.__init__
- Copied tag protection methods (_extract_tags, _tags_are_intact,
  _on_text_changed_with_tag_protection) to GridTextEditor
- Added focusInEvent/focusOutEvent handlers to GridTextEditor
- Ensured delegate is recreated when protection setting changes

The tag protection feature now properly prevents tag modification when enabled.
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.

2 participants