Skip to content

Fix LT-22508: text not enabled; text deletion issues#866

Merged
jtmaxwell3 merged 4 commits intomainfrom
LT-22508
May 7, 2026
Merged

Fix LT-22508: text not enabled; text deletion issues#866
jtmaxwell3 merged 4 commits intomainfrom
LT-22508

Conversation

@jtmaxwell3
Copy link
Copy Markdown
Collaborator

@jtmaxwell3 jtmaxwell3 commented May 6, 2026

This fixes https://jira.sil.org/browse/LT-22508. I don't think that this was a regression. The latest change to InterestingTextsDecorator was April 7th. I tried a version from March 31st and the bug was still there. The penultimate change was in 2017.

I added a PropChanged to clear the cache of InterestingTextsList when needed.


This change is Reviewable

@github-actions
Copy link
Copy Markdown

github-actions Bot commented May 6, 2026

NUnit Tests

    1 files  ±0      1 suites  ±0   6m 6s ⏱️ -52s
4 102 tests ±0  4 031 ✅ ±0  71 💤 ±0  0 ❌ ±0 
4 111 runs  ±0  4 040 ✅ ±0  71 💤 ±0  0 ❌ ±0 

Results for commit c14c279. ± Comparison against base commit d4fd1ec.

Copy link
Copy Markdown
Contributor

@mark-sil mark-sil left a comment

Choose a reason for hiding this comment

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

@mark-sil reviewed 3 files and all commit messages.
Reviewable status: :shipit: complete! all files reviewed, all discussions resolved (waiting on jtmaxwell3).

@jtmaxwell3 jtmaxwell3 merged commit 8c7aee1 into main May 7, 2026
7 checks passed
@jtmaxwell3 jtmaxwell3 deleted the LT-22508 branch May 7, 2026 16:35
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