Skip to content

Commit

Permalink
fix: DOM table nesting (microsoft#438)
Browse files Browse the repository at this point in the history
* fix: DOM table nesting

* refactor: use td instead of th and limit padding

Co-authored-by: stew-ro <[email protected]>
  • Loading branch information
alex-krasn and stew-ro authored Jul 21, 2020
1 parent ed919a0 commit f4b4d5d
Showing 1 changed file with 26 additions and 22 deletions.
48 changes: 26 additions & 22 deletions src/react/components/pages/predict/predictPage.tsx
Original file line number Diff line number Diff line change
Expand Up @@ -250,28 +250,32 @@ export default class PredictPage extends React.Component<IPredictPageProps, IPre
<h5 className="model-selection-header">
{strings.predict.selectModelHeader}
</h5>
<tr>
<h6 className="model-selection-info-header" >
<span className="model-selection-info-key">
{strings.predict.modelIDPrefix}
</span>
<span title={mostRecentModel.modelInfo.modelId} className="model-selection-info-value">
{mostRecentModel.modelInfo.modelId}
</span>
</h6>
</tr>
<tr>
{mostRecentModel.modelInfo.modelName &&
<h6 className="model-selection-info-header" >
<span className="model-selection-info-key">
{strings.predict.modelNamePrefix}
</span>
<span title={mostRecentModel.modelInfo.modelName} className="model-selection-info-value">
{mostRecentModel.modelInfo.modelName}
</span>
</h6>
}
</tr>
<table>
<tbody>
<tr>
<td className="model-selection-info-header p-0" >
<span className="model-selection-info-key">
{strings.predict.modelIDPrefix}
</span>
<span title={mostRecentModel.modelInfo.modelId} className="model-selection-info-value">
{mostRecentModel.modelInfo.modelId}
</span>
</td>
</tr>
<tr>
{mostRecentModel.modelInfo.modelName &&
<td className="model-selection-info-header p-0" >
<span className="model-selection-info-key">
{strings.predict.modelNamePrefix}
</span>
<span title={mostRecentModel.modelInfo.modelName} className="model-selection-info-value">
{mostRecentModel.modelInfo.modelName}
</span>
</td>
}
</tr>
</tbody>
</table>
</div>
<DefaultButton
className="keep-button-80px"
Expand Down

0 comments on commit f4b4d5d

Please sign in to comment.