SF-3677 Use the verse the user has selected in the New Question dialog

src/SIL.XForge.Scripture/ClientApp/src/app/checking/checking/checking-text/checking-text.component.html

@@ -6,4 +6,5 @@
   (loaded)="onLoaded()"
   [isRightToLeft]="isRightToLeft"
   [fontSize]="fontSize"
+  [highlightSegment]="highlightSegment"
 ></app-text>

src/SIL.XForge.Scripture/ClientApp/src/app/checking/checking/checking-text/checking-text.component.ts

@@ -26,6 +26,7 @@ export class CheckingTextComponent implements AfterViewInit {
 
   private clickSubs: Subscription[] = [];
   private _activeVerse?: VerseRef;
+  private _highlightSegment: boolean = false;
   private _id?: TextDocId;
   private _questionVerses?: VerseRef[];
   private _placeholder?: string;
@@ -69,6 +70,14 @@ export class CheckingTextComponent implements AfterViewInit {
     return this._activeVerse;
   }
 
+  @Input() set highlightSegment(value: boolean) {
+    this._highlightSegment = value;
+  }
+
+  get highlightSegment(): boolean {
+    return this._highlightSegment;
+  }
+
   @Input() set id(textDocId: TextDocId | undefined) {
     if (textDocId) {
       this._id = textDocId;

src/SIL.XForge.Scripture/ClientApp/src/app/checking/checking/checking.component.html

@@ -135,6 +135,7 @@ <h2 matSubheader>{{ t("filter_questions") }}</h2>
                     [class.hidden]="hideChapterText"
                     [projectDoc]="projectDoc"
                     [resizableContainer]="splitter"
+                    [highlightSegment]="canCreateQuestions"
                   ></app-checking-text>
                 </div>
               </div>

src/SIL.XForge.Scripture/ClientApp/src/app/checking/checking/checking.component.ts

@@ -47,7 +47,7 @@ import { PermissionsService } from '../../core/permissions.service';
 import { SFProjectService } from '../../core/sf-project.service';
 import { BookChapterChooserComponent } from '../../shared/book-chapter-chooser/book-chapter-chooser.component';
 import { ShareButtonComponent } from '../../shared/share/share-button.component';
-import { getVerseStrFromSegmentRef } from '../../shared/verse-utils';
+import { getVerseRefFromSegmentRef, getVerseStrFromSegmentRef } from '../../shared/verse-utils';
 import { ChapterAudioDialogData } from '../chapter-audio-dialog/chapter-audio-dialog.component';
 import { ChapterAudioDialogService } from '../chapter-audio-dialog/chapter-audio-dialog.service';
 import { BookChapter, CheckingUtils, isQuestionScope, QuestionScope } from '../checking.utils';
@@ -1054,12 +1054,21 @@ export class CheckingComponent extends DataLoadingComponent implements OnInit, A
     this._scriptureAudioPlayer?.pause();
     this.answersPanel?.questionComponent?.stopAudio();
 
+    // Get the segment to set the scripture reference
+    let verseRef: VerseRef | undefined = undefined;
+    const segmentRef: string | undefined = this.scripturePanel?.textComponent.segmentRef;
+    if (segmentRef != null && this.book != null) {
+      verseRef = getVerseRefFromSegmentRef(this.book, segmentRef);
+    }
+
+    verseRef ??= new VerseRef(this.book ?? 0, this.chapter ?? 1, 1);
+
     const data: QuestionDialogData = {
       questionDoc: undefined,
       projectDoc: this.projectDoc,
       textsByBookId: this.textsByBookId,
       projectId: this.projectDoc.id,
-      defaultVerse: new VerseRef(this.book ?? 0, this.chapter ?? 1, 1),
+      defaultVerse: verseRef,
       isRightToLeft: this.projectDoc.data?.isRightToLeft
     };
     const newQuestion: QuestionDoc | undefined = await this.questionDialogService.questionDialog(data);
@@ -1715,6 +1724,11 @@ export class CheckingComponent extends DataLoadingComponent implements OnInit, A
       }
     }
 
+    // Clear the segment so the add question dialog will not select an incorrect verse
+    this.scripturePanel?.textComponent.setSegment('');
+    // Clear the selection so the editor won't select the wrong segment
+    this.scripturePanel?.textComponent.editor?.setSelection(null);
+    // Perform the navigation
     void this.router.navigate(['projects', projectId, 'checking', ...bookChapterPathTokens], {
       ...navigationExtras,
       queryParams: { scope: scope },