Skip to content

Commit

Permalink
Merge pull request #318 from mlibrary/WEBSITE-134-adjust-colors-for-i…
Browse files Browse the repository at this point in the history
…cons-in-locationAside

WEBSITE-164 Adjust colors for icons in locationAside to meet minimum acceptable contrast
  • Loading branch information
SalazarJosh authored Nov 14, 2024
2 parents e9257df + 65a5c32 commit 224eab7
Showing 1 changed file with 6 additions and 4 deletions.
10 changes: 6 additions & 4 deletions src/components/location-aside.js
Original file line number Diff line number Diff line change
Expand Up @@ -10,6 +10,7 @@ import React from 'react';
const LayoutWithIcon = ({
// eslint-disable-next-line react/prop-types
d: data,
color,
palette,
children
}) => {
Expand All @@ -29,7 +30,7 @@ const LayoutWithIcon = ({
alignItems: 'center',
background: `var(--color-${palette}-100)`,
borderRadius: '50%',
color: `var(--color-${palette}-300)`,
color: `var(--color-${palette}-${color})`,
display: 'flex',
height: '2.5rem',
justifyContent: 'center',
Expand All @@ -46,6 +47,7 @@ const LayoutWithIcon = ({

LayoutWithIcon.propTypes = {
children: PropTypes.node,
color: PropTypes.string,
data: PropTypes.string,
palette: PropTypes.string
};
Expand All @@ -65,7 +67,7 @@ export default function LocationAside ({ node }) {
marginBottom: SPACING['3XL']
}}
>
<LayoutWithIcon d={icons.clock} palette='indigo'>
<LayoutWithIcon d={icons.clock} palette='indigo' color='400'>
<Heading
level={2}
size='M'
Expand All @@ -91,7 +93,7 @@ export default function LocationAside ({ node }) {
}
}}
>
<LayoutWithIcon d={icons.address} palette='orange'>
<LayoutWithIcon d={icons.address} palette='orange' color='500'>
<Heading
level={2}
size='M'
Expand All @@ -105,7 +107,7 @@ export default function LocationAside ({ node }) {
<Address node={node} directions={true} kind='full' />
</LayoutWithIcon>

<LayoutWithIcon d={icons.phone} palette='green'>
<LayoutWithIcon d={icons.phone} palette='green' color='500'>
<Heading
level={2}
size='M'
Expand Down

0 comments on commit 224eab7

Please sign in to comment.