Add mobile styles to the event card #190
Merged
a3thakra
merged 3 commits from adi-mobile-events-card
into main
1 year ago
Loading…
Reference in new issue
There is no content yet.
Delete Branch 'adi-mobile-events-card'
Deleting a branch is permanent. It CANNOT be undone. Continue?
closes #160
closes #154
https://csclub.uwaterloo.ca/~a3thakra/csc/adi-mobile-events-card/
className={[
styles.content,
showDescription ? styles.mobileShowDescriptionContent : "",
].join(" ")}
Would something like this work? (I don't know if it's any cleaner/better than what we already have though...)
it kinda results in a lot of nested ${} which i kinda dont like lol
the only clean way would be to use the classnames library 🤷♂️
export function Image(props: ImgHTMLAttributes<HTMLImageElement>) {
const classes = props.className
? [props.className, styles.image]
: [styles.image];
Similar to line 57 of
components/EventCard.tsx
, not sure if the suggestion there would be helpful here or not.same as above
The mobile EventCard looks a little funny on iPad...
6b88cf308b
into main 1 year ago6b88cf308b
.