Skip to content

Fix LT-22606: Word Cat has tan background when bundle selected#999

Merged
jtmaxwell3 merged 3 commits into
mainfrom
LT-22606
Jul 8, 2026
Merged

Fix LT-22606: Word Cat has tan background when bundle selected#999
jtmaxwell3 merged 3 commits into
mainfrom
LT-22606

Conversation

@jtmaxwell3

@jtmaxwell3 jtmaxwell3 commented Jul 8, 2026

Copy link
Copy Markdown
Collaborator

This fixes https://jira.sil.org/browse/LT-22606. Don't guess the word cat or word gloss if the analysis was approved by the user since it is confusing to have a mixture of approved and guessed data. This changes the fix to https://jira.sil.org/browse/LT-22577.


This change is Reviewable

@github-actions

github-actions Bot commented Jul 8, 2026

Copy link
Copy Markdown

NUnit Tests

    1 files  ±0      1 suites  ±0   10m 8s ⏱️ +38s
4 302 tests +3  4 229 ✅ +3  73 💤 ±0  0 ❌ ±0 
4 311 runs  +3  4 238 ✅ +3  73 💤 ±0  0 ❌ ±0 

Results for commit 5d5844f. ± Comparison against base commit b7c9ce5.

♻️ This comment has been updated with latest results.

jtmaxwell3 and others added 2 commits July 8, 2026 10:29
Not guessing the word cat for user-approved analyses restores the
pre-LT-20848 tab behavior: the Nihimbilira test data is an approved
analysis with no category, so tabbing to the word cat line lands on
the plain dropdown icon (ktagWordPosIcon) again. Revert the two
assertions that LT-20848 had updated for the auto-filled category.

Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>

@jasonleenaylor jasonleenaylor left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

:lgtm:

@jasonleenaylor reviewed 4 files and all commit messages, and made 1 comment.
Reviewable status: :shipit: complete! all files reviewed, all discussions resolved (waiting on jtmaxwell3).

@jtmaxwell3 jtmaxwell3 merged commit c874439 into main Jul 8, 2026
6 of 7 checks passed
@jtmaxwell3 jtmaxwell3 deleted the LT-22606 branch July 8, 2026 22:15
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