Revert "[SG-43144]: Tables in code excerpts should use headers for line numbers (#43389)"

This reverts commit 5e41e12.

Author: ggilmore

client/branded/src/components/panel/views/__snapshots__/HierarchicalLocationsView.test.tsx.snap

@@ -5,7 +5,7 @@ export function generateSourcegraphCodeTable(lines: string[]): string {
     const code = lines
         .map(
             (line, index) => `<tr>
-                <th class="line" data-line="${index + 1}"></th>
+                <td class="line" data-line="${index + 1}"></td>
                 <td class="code">${line}</td>
             </tr>`
         )

client/codeintellify/src/testutils/sourcegraph/generate.ts

@@ -50,7 +50,7 @@
     border-collapse: collapse;
 }
 
-.sourcegraph-testcase code.code th.line {
+.sourcegraph-testcase code.code td.line {
     text-align: right;
     padding-left: 0.5rem;
     min-width: 2rem;
@@ -61,7 +61,7 @@
     vertical-align: top;
 }
 
-.sourcegraph-testcase th.line::before {
+.sourcegraph-testcase td.line::before {
     content: attr(data-line);
 }
 

client/codeintellify/src/testutils/sourcegraph/styles.css

@@ -10,7 +10,6 @@
     :global(.line) {
         min-width: 1.5rem;
         text-align: right;
-        font-weight: inherit;
         user-select: none;
 
         &::before {

client/search-ui/src/components/CodeExcerpt.module.scss

@@ -76,7 +76,7 @@ const domFunctions: DOMFunctions = {
         return row.cells[1]
     },
     getCodeElementFromLineNumber: (codeView: HTMLElement, line: number): HTMLTableCellElement | null => {
-        const lineElement = codeView.querySelector(`th[data-line="${line}"]`)
+        const lineElement = codeView.querySelector(`td[data-line="${line}"]`)
         if (!lineElement) {
             return null
         }
@@ -330,7 +330,7 @@ export const CodeExcerpt: React.FunctionComponent<Props> = ({
                         <tbody>
                             {range(startLine, endLine).map(index => (
                                 <tr key={index}>
-                                    <th className="line" data-line={index + 1} />
+                                    <td className="line" data-line={index + 1} />
                                     {/* create empty space to fill viewport (as if the blob content were already fetched, otherwise we'll overfetch) */}
                                     <td className="code"> </td>
                                 </tr>

client/search-ui/src/components/CodeExcerpt.tsx

@@ -32,7 +32,7 @@
     table {
         margin-bottom: 0 !important; /* Override docsite Markdown table CSS. A currently open PR removes that CSS and lets us remove this override. */
     }
-    th:global(.line) {
+    td:global(.line) {
         display: none;
     }
     td:global(.code) > span:first-child {