Merged in fix/tripadvisor-rating (pull request #2193)

fix(SW-2892): don't show Tripadvisor rating if undefined

* fix: don't show rating if undefined


Approved-by: Bianca Widstam
Approved-by: Christian Andolf
Approved-by: Erik Tiekstra
This commit is contained in:
Matilda Landström
2025-05-22 15:05:25 +00:00
parent e6963cd110
commit 76cf33a199
6 changed files with 29 additions and 25 deletions

View File

@@ -7,16 +7,16 @@ import Chip from "@/components/TempDesignSystem/Chip"
import styles from "./dialogImage.module.css" import styles from "./dialogImage.module.css"
interface DialogImageProps { interface DialogImageProps {
image: string | undefined image?: string
altText: string | undefined altText?: string
ratings: number rating?: number
imageError: boolean imageError: boolean
setImageError: (error: boolean) => void setImageError: (error: boolean) => void
} }
export default function DialogImage({ export default function DialogImage({
image, image,
altText, altText,
ratings, rating,
imageError, imageError,
setImageError, setImageError,
}: DialogImageProps) { }: DialogImageProps) {
@@ -32,14 +32,16 @@ export default function DialogImage({
onError={() => setImageError(true)} onError={() => setImageError(true)}
/> />
)} )}
<div className={styles.tripAdvisor}> {rating ? (
<Chip className={styles.tripAdvisor}> <div className={styles.tripAdvisor}>
<TripadvisorIcon color="Icon/Interactive/Default" /> <Chip className={styles.tripAdvisor}>
<Typography variant="Tag/sm"> <TripadvisorIcon color="Icon/Interactive/Default" />
<p>{ratings}</p> <Typography variant="Tag/sm">
</Typography> <span>{rating}</span>
</Chip> </Typography>
</div> </Chip>
</div>
) : null}
</div> </div>
) )
} }

View File

@@ -68,7 +68,7 @@ export default function HotelMapCard({
<DialogImage <DialogImage
image={image.src} image={image.src}
altText={image.alt} altText={image.alt}
ratings={tripadvisorRating || 0} rating={tripadvisorRating}
imageError={imageError} imageError={imageError}
setImageError={setImageError} setImageError={setImageError}
/> />

View File

@@ -12,7 +12,7 @@ import type { HotelCardDialogImageProps } from "@/types/components/hotelReservat
export default function HotelCardDialogImage({ export default function HotelCardDialogImage({
firstImage, firstImage,
altText, altText,
ratings, rating,
imageError, imageError,
setImageError, setImageError,
position, position,
@@ -31,12 +31,14 @@ export default function HotelCardDialogImage({
onError={() => setImageError(true)} onError={() => setImageError(true)}
/> />
)} )}
<div className={styles.tripAdvisor}> {rating ? (
<Chip className={styles.tripAdvisor}> <div className={styles.tripAdvisor}>
<TripadvisorIcon color="Icon/Interactive/Default" /> <Chip className={styles.tripAdvisor}>
{ratings} <TripadvisorIcon color="Icon/Interactive/Default" />
</Chip> {rating}
</div> </Chip>
</div>
) : null}
</div> </div>
) )
} }

View File

@@ -62,7 +62,7 @@ export default function ListingHotelCardDialog({
<HotelCardDialogImage <HotelCardDialogImage
firstImage={firstImage} firstImage={firstImage}
altText={altText} altText={altText}
ratings={ratings || 0} rating={ratings}
imageError={imageError} imageError={imageError}
setImageError={setImageError} setImageError={setImageError}
position="top" position="top"

View File

@@ -61,7 +61,7 @@ export default function StandaloneHotelCardDialog({
<HotelCardDialogImage <HotelCardDialogImage
firstImage={firstImage} firstImage={firstImage}
altText={altText} altText={altText}
ratings={ratings || 0} rating={ratings}
imageError={imageError} imageError={imageError}
setImageError={setImageError} setImageError={setImageError}
position="left" position="left"

View File

@@ -62,9 +62,9 @@ export interface HotelCardDialogProps {
} }
export interface HotelCardDialogImageProps { export interface HotelCardDialogImageProps {
firstImage: string | undefined firstImage?: string
altText: string | undefined altText?: string
ratings: number rating?: number | null
imageError: boolean imageError: boolean
setImageError: (error: boolean) => void setImageError: (error: boolean) => void
position: "top" | "left" position: "top" | "left"