Small changes

4 files changed, 6 insertions, 4 deletions

diff --git a/client/js/app/src/app/pages/querybuilder/Components/Buttons/DownloadJSONButton.jsx b/client/js/app/src/app/pages/querybuilder/Components/Buttons/DownloadJSONButton.jsx
index 3997c9c4bcc..ad6da79e709 100644
--- a/client/js/app/src/app/pages/querybuilder/Components/Buttons/DownloadJSONButton.jsx
+++ b/client/js/app/src/app/pages/querybuilder/Components/Buttons/DownloadJSONButton.jsx
@@ -15,7 +15,7 @@ export default function DownloadJSONButton() {
       let transformedResponse = JSON.stringify(
         transformResponse(JSON.parse(response), undefined, '\t')
       );
-      // taken from safakeskin´s answer on SO, link: https://stackoverflow.com/questions/55613438/reactwrite-to-json-file-or-export-download-no-server
+      // copied from safakeskin´s answer on SO, link: https://stackoverflow.com/questions/55613438/reactwrite-to-json-file-or-export-download-no-server
       const fileName = 'vespa-response';
       const blob = new Blob([transformedResponse], {
         type: 'application/json',
@@ -32,6 +32,8 @@ export default function DownloadJSONButton() {
       // clean up "a" element & remove ObjectURL
       document.body.removeChild(link);
       URL.revokeObjectURL(href);
+
+      // open Jaeger in a new tab
       window.open('http://localhost:16686/search', '__blank');
     } else {
       alert('Response was empty');
diff --git a/client/js/app/src/app/pages/querybuilder/Components/Contexts/QueryInputContext.jsx b/client/js/app/src/app/pages/querybuilder/Components/Contexts/QueryInputContext.jsx
index 7126a472d55..bc21ea81d9a 100644
--- a/client/js/app/src/app/pages/querybuilder/Components/Contexts/QueryInputContext.jsx
+++ b/client/js/app/src/app/pages/querybuilder/Components/Contexts/QueryInputContext.jsx
@@ -20,6 +20,7 @@ export const QueryInputProvider = (prop) => {
     },
     searchChain: { name: 'searchChain', type: 'String', hasChildren: false },
     timeout: { name: 'timeout', type: 'Float', hasChildren: false },
+    trace: { name: 'trace', type: 'Parent', hasChildren: true },
     tracelevel: { name: 'tracelevel', type: 'Parent', hasChildren: true },
     traceLevel: { name: 'traceLevel', type: 'Integer', hasChildren: false },
     explainLevel: { name: 'explainLevel', type: 'Integer', hasChildren: false },
@@ -40,7 +41,6 @@ export const QueryInputProvider = (prop) => {
     recall: { name: 'recall', type: 'List', hasChildren: false },
     user: { name: 'user', type: 'String', hasChildren: false },
     metrics: { name: 'metrics', type: 'Parent', hasChildren: true },
-    trace: { name: 'trace', type: 'Parent', hasChildren: true },
   };
 
   // Children of the levelZeroParameters that have child attributes
diff --git a/client/js/app/src/app/pages/querybuilder/Components/Text/QueryDropDownForm.jsx b/client/js/app/src/app/pages/querybuilder/Components/Text/QueryDropDownForm.jsx
index d2de94b2e28..88bd545282a 100644
--- a/client/js/app/src/app/pages/querybuilder/Components/Text/QueryDropDownForm.jsx
+++ b/client/js/app/src/app/pages/querybuilder/Components/Text/QueryDropDownForm.jsx
@@ -1,5 +1,5 @@
+import React, { useContext, useState } from 'react';
 import { QueryInputContext } from '../Contexts/QueryInputContext';
-import React, { useCallback, useContext, useState } from 'react';
 import SimpleDropDownForm from './SimpleDropDownForm';
 
 export default function QueryDropdownForm({
diff --git a/client/js/app/src/app/pages/querybuilder/Components/Text/SimpleDropDownForm.jsx b/client/js/app/src/app/pages/querybuilder/Components/Text/SimpleDropDownForm.jsx
index 40a40e7758b..94c6c01b619 100644
--- a/client/js/app/src/app/pages/querybuilder/Components/Text/SimpleDropDownForm.jsx
+++ b/client/js/app/src/app/pages/querybuilder/Components/Text/SimpleDropDownForm.jsx
@@ -11,7 +11,7 @@ export default function SimpleDropDownForm({
 }) {
   const { selectedItems } = useContext(QueryInputContext);
 
-  //FIXME: using the filtered list to render options results in dropdown not changing the displayed selection to what was actually selected.
+  //TODO: using the filtered list to render options results in dropdown not changing the displayed selection to what was actually selected.
   let filtered = Object.keys(choices).filter(
     (choice) => !selectedItems.includes(choice)
   );