Skip to content

Commit 2d6f419

Browse files
committed
use Card and not MuiCard
1 parent efe80f1 commit 2d6f419

File tree

1 file changed

+41
-45
lines changed

1 file changed

+41
-45
lines changed

frontends/mit-open/src/pages/UnitsListingPage/UnitsListingPage.tsx

Lines changed: 41 additions & 45 deletions
Original file line numberDiff line numberDiff line change
@@ -4,10 +4,8 @@ import {
44
} from "api/hooks/learningResources"
55
import {
66
Banner,
7-
MuiCard,
8-
CardContent,
7+
Card,
98
Container,
10-
Link,
119
Skeleton,
1210
Typography,
1311
styled,
@@ -147,23 +145,15 @@ const GridContainer = styled.div(({ theme }) => ({
147145
},
148146
}))
149147

150-
const UnitCardContainer = styled(MuiCard)({
148+
const UnitCardContainer = styled.div({
151149
display: "flex",
152150
flexDirection: "column",
153151
alignItems: "center",
152+
padding: "16px",
153+
height: "100%",
154154
})
155155

156-
const UnitCardLink = styled(Link)({
157-
display: "flex",
158-
flexDirection: "column",
159-
alignItems: "center",
160-
flexGrow: 1,
161-
"&:hover": {
162-
textDecoration: "none",
163-
},
164-
})
165-
166-
const UnitCardContent = styled(CardContent)({
156+
const UnitCardContent = styled.div({
167157
display: "flex",
168158
flexDirection: "column",
169159
flexGrow: 1,
@@ -316,41 +306,47 @@ const UnitCard: React.FC<UnitCardProps> = (props) => {
316306
return channelDetailQuery.isLoading ? (
317307
<UnitCardLoading />
318308
) : (
319-
<UnitCardContainer>
320-
<UnitCardLink href={unitUrl}>
321-
<UnitCardContent>
322-
<LogoContainer>
323-
<UnitLogo src={logo} alt={unit.name} />
324-
</LogoContainer>
325-
<ValuePropContainer>
326-
<ValuePropText>{unit.value_prop}</ValuePropText>
327-
</ValuePropContainer>
328-
<CountsTextContainer>
329-
<CountsText data-testid={`course-count-${unit.code}`}>
330-
{courseCount > 0 ? `Courses: ${courseCount}` : ""}
331-
</CountsText>
332-
<CountsText data-testid={`program-count-${unit.code}`}>
333-
{programCount > 0 ? `Programs: ${programCount}` : ""}
334-
</CountsText>
335-
</CountsTextContainer>
336-
</UnitCardContent>
337-
</UnitCardLink>
338-
</UnitCardContainer>
309+
<Card link={true} href={unitUrl}>
310+
<Card.Content>
311+
<UnitCardContainer>
312+
<UnitCardContent>
313+
<LogoContainer>
314+
<UnitLogo src={logo} alt={unit.name} />
315+
</LogoContainer>
316+
<ValuePropContainer>
317+
<ValuePropText>{unit.value_prop}</ValuePropText>
318+
</ValuePropContainer>
319+
<CountsTextContainer>
320+
<CountsText data-testid={`course-count-${unit.code}`}>
321+
{courseCount > 0 ? `Courses: ${courseCount}` : ""}
322+
</CountsText>
323+
<CountsText data-testid={`program-count-${unit.code}`}>
324+
{programCount > 0 ? `Programs: ${programCount}` : ""}
325+
</CountsText>
326+
</CountsTextContainer>
327+
</UnitCardContent>
328+
</UnitCardContainer>
329+
</Card.Content>
330+
</Card>
339331
)
340332
}
341333

342334
const UnitCardLoading = () => {
343335
return (
344-
<UnitCardContainer>
345-
<UnitCardContent>
346-
<LogoContainer>
347-
<Skeleton variant="rectangular" width={500} height={50} />
348-
</LogoContainer>
349-
<ValuePropContainer>
350-
<Skeleton variant="text" width={500} height={200} />
351-
</ValuePropContainer>
352-
</UnitCardContent>
353-
</UnitCardContainer>
336+
<Card>
337+
<Card.Content>
338+
<UnitCardContainer>
339+
<UnitCardContent>
340+
<LogoContainer>
341+
<Skeleton variant="rectangular" width={500} height={50} />
342+
</LogoContainer>
343+
<ValuePropContainer>
344+
<Skeleton variant="text" width={500} height={200} />
345+
</ValuePropContainer>
346+
</UnitCardContent>
347+
</UnitCardContainer>
348+
</Card.Content>
349+
</Card>
354350
)
355351
}
356352

0 commit comments

Comments
 (0)