Skip to content

Conversation

@jtmaxwell3
Copy link
Collaborator

@jtmaxwell3 jtmaxwell3 commented Jun 6, 2025

This fixes https://jira.sil.org/browse/LT-5408. It adds marks in front of analyses to indicate whether the analysis is human-approved and whether it is parser-approved.


This change is Reviewable

@jasonleenaylor
Copy link
Contributor

Src/LexText/Interlinear/ChooseAnalysisHandler.cs line 478 at r1 (raw file):

			else if (opinion == Opinions.noopinion)
			{
				// Use same foreground and background color to create a blank of the same width as a check mark.

Clever

Copy link
Contributor

@jasonleenaylor jasonleenaylor left a comment

Choose a reason for hiding this comment

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

Could you add a screenshot?
:lgtm:

Reviewed 1 of 1 files at r1, all commit messages.
Reviewable status: :shipit: complete! all files reviewed, all discussions resolved (waiting on @jtmaxwell3)

@jtmaxwell3 jtmaxwell3 enabled auto-merge (squash) June 12, 2025 13:42
@jtmaxwell3 jtmaxwell3 merged commit f462470 into release/9.3 Jun 12, 2025
5 checks passed
@jtmaxwell3 jtmaxwell3 deleted the LT-5408 branch June 12, 2025 13:56
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.

3 participants