[Discover with Legacy Table] fix copy issue and open feedbacks in a n…

…ew tab (#5761)

* add a colon (:) after the key
* ensure there's a space between each key-value pair
* allow feedback url to open in a new tab

Signed-off-by: Anan Z <ananzh@amazon.com>

src/plugins/discover/public/application/components/data_grid/data_grid_table_cell_value.tsx

@@ -27,17 +27,19 @@ export function fetchSourceTypeDataCell(
     return <span>{stringify(row[columnId], null, 2)}</span>;
   }
   const formattedRow = idxPattern.formatHit(row);
+  const keys = Object.keys(formattedRow);
 
   return (
     <EuiDescriptionList type="inline" compressed>
-      {Object.keys(formattedRow).map((key) => (
+      {keys.map((key, index) => (
         <Fragment key={key}>
           <EuiDescriptionListTitle className="osdDescriptionListFieldTitle">
-            {key}
+            {key + ':'}
           </EuiDescriptionListTitle>
           <EuiDescriptionListDescription
             dangerouslySetInnerHTML={{ __html: dompurify.sanitize(formattedRow[key]) }}
           />
+          {index !== keys.length - 1 && ' '}
         </Fragment>
       ))}
     </EuiDescriptionList>

src/plugins/discover/public/application/components/top_nav/table_feedbacks_panel.tsx

@@ -33,7 +33,9 @@ export const TableFeedbacksPanel = ({ onClose, onTurnOff }: TableFeedbacksPanelP
         <p>
           Event tables: Documents are now expanded through a flyout. Density, column order, and
           sorting controls have been improved.{' '}
-          <a href="https://survey.opensearch.org">Provide feedbacks</a>.
+          <a href="https://survey.opensearch.org" target="_blank" rel="noopener noreferrer">
+            Provide feedbacks
+          </a>
         </p>
       </EuiText>
     </EuiModalBody>