Skip to content

Commit

Permalink
🐛 Address style issues on assessment summary / questionnaire view (#1706
Browse files Browse the repository at this point in the history
)

Resolves https://issues.redhat.com/browse/MTA-1880
- Adds spacing for vertical tabs to align with question table
- Adds spacing for expanded section to meet PF standards as outlined in
above issue comments from UXD
- Adds Explanation term for explanation previously not labeled.
<img width="1638" alt="Screenshot 2024-02-27 at 2 48 46 PM"
src="https://github.com/konveyor/tackle2-ui/assets/11218376/a5d915d9-7977-48cd-ac4c-5c3ca1cdc909">

---------

Signed-off-by: Ian Bolton <ibolton@redhat.com>
Co-authored-by: Scott Dickerson <sdickers@redhat.com>
Signed-off-by: Cherry Picker <noreply@github.com>
  • Loading branch information
2 people authored and web-flow committed Feb 28, 2024
1 parent 0dcfa7f commit b37d340
Show file tree
Hide file tree
Showing 6 changed files with 15 additions and 17 deletions.
1 change: 1 addition & 0 deletions client/public/locales/en/translation.json
Original file line number Diff line number Diff line change
Expand Up @@ -321,6 +321,7 @@
"email": "Email",
"error": "Error",
"errorReport": "Error report",
"explanation": "Explanation",
"exclude": "Exclude",
"exportToIssue": "Export to Issue Manager",
"facts": "Facts",
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -166,6 +166,7 @@ const QuestionnaireSummary: React.FC<QuestionnaireSummaryProps> = ({
isVertical
aria-label="Tabs for summaryData sections"
role="region"
className="tabs-vertical-container__tabs"
>
{[
<Tab
Expand Down
13 changes: 8 additions & 5 deletions client/src/app/components/questions-table/questions-table.tsx
Original file line number Diff line number Diff line change
Expand Up @@ -178,11 +178,14 @@ const QuestionsTable: React.FC<{
className={spacing.pyLg}
>
<ExpandableRowContent>
{question.explanation}
<AnswerTable
hideAnswerKey={hideAnswerKey}
answers={question.answers}
/>
<div className={spacing.mlMd}>
<strong>{t("terms.explanation")}: &nbsp;</strong>
<span>{question.explanation}</span>
<AnswerTable
hideAnswerKey={hideAnswerKey}
answers={question.answers}
/>
</div>
</ExpandableRowContent>
</Td>
</Tr>
Expand Down
Original file line number Diff line number Diff line change
@@ -1,5 +1,10 @@
.tabs-vertical-container {
display: flex;
margin-top: 1.5em;
}

.tabs-vertical-container__tabs {
margin-top: 1.5em;
}

.tabs-vertical-container .pf-v5-c-tabs {
Expand Down

This file was deleted.

Original file line number Diff line number Diff line change
@@ -1,6 +1,5 @@
import React from "react";
import { useParams } from "react-router-dom";
import "./assessment-summary-page.css";
import QuestionnaireSummary, {
SummaryType,
} from "@app/components/questionnaire-summary/questionnaire-summary";
Expand Down

0 comments on commit b37d340

Please sign in to comment.