Skip to content

Commit

Permalink
Features/psp 7494 (#4068)
Browse files Browse the repository at this point in the history
* Updated the deposts page

* Fixed stylings for buttons and margin for collapsable section

* updated snaps

* PR comment updates.

---------

Co-authored-by: FuriousLlama <[email protected]>
Co-authored-by: Smith <[email protected]>
Co-authored-by: Alejandro Sanchez <[email protected]>
  • Loading branch information
4 people committed May 31, 2024
1 parent 48e7a8c commit 3a59370
Show file tree
Hide file tree
Showing 39 changed files with 181 additions and 460 deletions.
3 changes: 2 additions & 1 deletion source/frontend/src/assets/scss/_variables.module.scss
Original file line number Diff line number Diff line change
Expand Up @@ -10,8 +10,9 @@
highlightBackgroundColor: $highlight-background-color;
// buttons
borderOutlineColor: $border-outline-color;

// Colors
pimsGreen80: $pims-green-80;
warningBackgroundColor: $warning-background-color;
numberBackgroundColor: $number-background-color;
dangerBackgroundColor: $danger-background-color;
Expand Down
2 changes: 1 addition & 1 deletion source/frontend/src/components/common/Section/Section.tsx
Original file line number Diff line number Diff line change
Expand Up @@ -42,7 +42,7 @@ export const Section: React.FC<
<Row className="no-gutters">
<Col>{header}</Col>
{isCollapsable && (
<Col xs="1">
<Col xs="auto" className="pl-8">
{isCollapsed && (
<ArrowDropDownIcon
title={`expand-${title ?? 'section'}`}
Expand Down
3 changes: 2 additions & 1 deletion source/frontend/src/components/common/SectionListHeader.tsx
Original file line number Diff line number Diff line change
Expand Up @@ -12,6 +12,7 @@ export interface ISectionListHeaderProps {
addButtonIcon?: JSX.Element;
onAdd?: () => void;
claims: Claims[];
'data-testId'?: string;
}

export const SectionListHeader: React.FunctionComponent<
Expand All @@ -27,7 +28,7 @@ export const SectionListHeader: React.FunctionComponent<
</Col>
<Col xs="auto" className="my-1">
{hasClaim(props.claims) && (
<StyledSectionAddButton onClick={onClick}>
<StyledSectionAddButton onClick={onClick} data-testid={props['data-testId']}>
{props.addButtonIcon}
&nbsp;{props.addButtonText ?? 'Add'}
</StyledSectionAddButton>
Expand Down
3 changes: 3 additions & 0 deletions source/frontend/src/components/common/styles.ts
Original file line number Diff line number Diff line change
Expand Up @@ -106,6 +106,9 @@ export const StyledAddButton = styled(Button)`
&.btn.btn-primary:active {
background-color: ${props => props.theme.bcTokens.iconsColorSuccess};
}
&.btn.btn-primary:hover {
background-color: ${props => props.theme.css.pimsGreen80};
}
`;

export const StyledSectionAddButton = styled(StyledAddButton)`
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -688,7 +688,7 @@ exports[`CreateOrganizationForm > renders as expected 1`] = `
Mailing Address
</div>
<div
class="col-1"
class="pl-8 col-auto"
>
<svg
class="c10"
Expand Down Expand Up @@ -942,7 +942,7 @@ exports[`CreateOrganizationForm > renders as expected 1`] = `
Property Address
</div>
<div
class="col-1"
class="pl-8 col-auto"
>
<svg
class="c10"
Expand Down Expand Up @@ -1196,7 +1196,7 @@ exports[`CreateOrganizationForm > renders as expected 1`] = `
Billing Address
</div>
<div
class="col-1"
class="pl-8 col-auto"
>
<svg
class="c10"
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -853,7 +853,7 @@ exports[`CreatePersonForm > renders as expected 1`] = `
</div>
</div>
<div
class="col-1"
class="pl-8 col-auto"
>
<svg
class="c12"
Expand Down Expand Up @@ -1107,7 +1107,7 @@ exports[`CreatePersonForm > renders as expected 1`] = `
Property Address
</div>
<div
class="col-1"
class="pl-8 col-auto"
>
<svg
class="c12"
Expand Down Expand Up @@ -1361,7 +1361,7 @@ exports[`CreatePersonForm > renders as expected 1`] = `
Billing Address
</div>
<div
class="col-1"
class="pl-8 col-auto"
>
<svg
class="c12"
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -1067,7 +1067,7 @@ exports[`CreateContactContainer component > should render as expected 1`] = `
</div>
</div>
<div
class="col-1"
class="pl-8 col-auto"
>
<svg
class="c20"
Expand Down Expand Up @@ -1321,7 +1321,7 @@ exports[`CreateContactContainer component > should render as expected 1`] = `
Property Address
</div>
<div
class="col-1"
class="pl-8 col-auto"
>
<svg
class="c20"
Expand Down Expand Up @@ -1575,7 +1575,7 @@ exports[`CreateContactContainer component > should render as expected 1`] = `
Billing Address
</div>
<div
class="col-1"
class="pl-8 col-auto"
>
<svg
class="c20"
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -806,7 +806,7 @@ exports[`UpdateOrganizationForm > renders as expected 1`] = `
Mailing Address
</div>
<div
class="col-1"
class="pl-8 col-auto"
>
<svg
class="c10"
Expand Down Expand Up @@ -1060,7 +1060,7 @@ exports[`UpdateOrganizationForm > renders as expected 1`] = `
Property Address
</div>
<div
class="col-1"
class="pl-8 col-auto"
>
<svg
class="c10"
Expand Down Expand Up @@ -1314,7 +1314,7 @@ exports[`UpdateOrganizationForm > renders as expected 1`] = `
Billing Address
</div>
<div
class="col-1"
class="pl-8 col-auto"
>
<svg
class="c10"
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -926,7 +926,7 @@ exports[`UpdatePersonForm > renders as expected 1`] = `
</div>
</div>
<div
class="col-1"
class="pl-8 col-auto"
>
<svg
class="c13"
Expand Down Expand Up @@ -1180,7 +1180,7 @@ exports[`UpdatePersonForm > renders as expected 1`] = `
Property Address
</div>
<div
class="col-1"
class="pl-8 col-auto"
>
<svg
class="c13"
Expand Down Expand Up @@ -1434,7 +1434,7 @@ exports[`UpdatePersonForm > renders as expected 1`] = `
Billing Address
</div>
<div
class="col-1"
class="pl-8 col-auto"
>
<svg
class="c13"
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -1070,7 +1070,7 @@ exports[`UpdateContactContainer > renders as expected 1`] = `
</div>
</div>
<div
class="col-1"
class="pl-8 col-auto"
>
<svg
class="c19"
Expand Down Expand Up @@ -1324,7 +1324,7 @@ exports[`UpdateContactContainer > renders as expected 1`] = `
Property Address
</div>
<div
class="col-1"
class="pl-8 col-auto"
>
<svg
class="c19"
Expand Down Expand Up @@ -1578,7 +1578,7 @@ exports[`UpdateContactContainer > renders as expected 1`] = `
Billing Address
</div>
<div
class="col-1"
class="pl-8 col-auto"
>
<svg
class="c19"
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -263,7 +263,7 @@ exports[`Document List View > renders as expected 1`] = `
</div>
</div>
<div
class="col-1"
class="pl-8 col-auto"
>
<svg
class="c3"
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -2330,7 +2330,7 @@ exports[`AddLeaseContainer component > renders as expected 1`] = `
Consultation
</div>
<div
class="col-1"
class="pl-8 col-auto"
>
<svg
class="c25"
Expand Down
Loading

0 comments on commit 3a59370

Please sign in to comment.