[Fix] Fix broken term definition link - ensure a text quote selector …

…is used when available Remove unused VocabularyFileDetail.tsx file, update caniuse
kbss-cvut · Feb 13, 2025 · dca1f58 · dca1f58
1 parent d12b056
commit dca1f58
Show file tree

Hide file tree

Showing 5 changed files with 18 additions and 41 deletions.
diff --git a/package-lock.json b/package-lock.json
diff --git a/src/component/annotator/AnnotatorTermsSelector.tsx b/src/component/annotator/AnnotatorTermsSelector.tsx
@@ -46,7 +46,11 @@ const AnnotatorTermsSelector: React.FC<AnnotatorTermsSelectorProps> = ({
 
   React.useEffect(() => {
     if (autoFocus) {
-      setTimeout(() => treeSelect.current.focus(), 100);
+      setTimeout(() => {
+        if (treeSelect.current !== null) {
+          treeSelect.current.focus();
+        }
+      }, 100);
     }
   }, [autoFocus, treeSelect]);
 

diff --git a/src/component/file/FileContentDetail.tsx b/src/component/file/FileContentDetail.tsx
@@ -104,7 +104,7 @@ export class FileContentDetail extends React.Component<
   }
 
   private onUpdate = (newFileContent: string) => {
-    trackPromise(
+    return trackPromise(
       this.props.saveFileContent(
         {
           fragment: this.props.iri.fragment,

diff --git a/src/component/term/TermDefinitionSourceLink.tsx b/src/component/term/TermDefinitionSourceLink.tsx
@@ -37,10 +37,12 @@ export const TermDefinitionSourceLink: React.FC<
     if (!file || !file.owner) {
       return;
     }
-    // assert target.selectors.length === 1
+    const selector = Utils.sanitizeArray(target.selectors).find((s) =>
+      Utils.sanitizeArray(s.types).includes(VocabularyUtils.TEXT_QUOTE_SELECTOR)
+    );
     dispatch(
       pushRoutingPayload(Routes.annotateFile, {
-        selector: Utils.sanitizeArray(target.selectors)[0],
+        selector,
       })
     );
     const ownerIri = VocabularyUtils.create(file.owner.iri!);

diff --git a/src/component/vocabulary/VocabularyFileDetail.tsx b/src/component/vocabulary/VocabularyFileDetail.tsx