Skip to content

Commit 7bcaac9

Browse files
authored
Merge pull request #258 from zigzagdev/fix/add-no-referrer-policy-to-heritage-images
fix: add referrerPolicy="no-referrer" to heritage image tags
2 parents 8ae6523 + a28ae9d commit 7bcaac9

5 files changed

Lines changed: 4 additions & 2 deletions

File tree

client/src/app/features/top/cards/HeritageCard.tsx

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -67,6 +67,7 @@ export function HeritageCard({ item, onClickItem }: HeritageCardProps) {
6767
<img
6868
src={item.thumbnailUrl}
6969
alt={item.thumbnailUrl ?? title}
70+
referrerPolicy="no-referrer"
7071
loading="lazy"
7172
className="h-56 w-full object-cover sm:h-64 lg:h-72"
7273
/>

client/src/app/features/top/components/HeritageSearchForm.tsx

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -101,7 +101,7 @@ export function HeritageSearchForm({
101101
aria-label="Region"
102102
>
103103
{regionOptions.map((opt, i) => (
104-
<option key={`${opt || "all"}-${i}`} value={opt}>
104+
<option key={`${opt || "All"}-${i}`} value={opt}>
105105
{opt ? opt : "All"}
106106
</option>
107107
))}

client/src/app/features/top/components/heritage-detail/HeritageGallery.tsx

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -56,6 +56,7 @@ export function HeritageGallery({ images, previewCount = 6, onOpenGallery, onSel
5656
<img
5757
src={img.url}
5858
alt={img.alt ?? ""}
59+
referrerPolicy="no-referrer"
5960
loading="lazy"
6061
className="aspect-[4/3] w-full object-cover transition-transform duration-300 group-hover:scale-[1.02]"
6162
/>

client/src/app/features/top/components/heritage-detail/HeritageHero.tsx

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -34,6 +34,7 @@ export function HeritageHero({ item }: Props) {
3434
<img
3535
src={primaryImage.url}
3636
alt={primaryImage.alt ?? ""}
37+
referrerPolicy="no-referrer"
3738
loading="lazy"
3839
className="h-72 w-full object-cover md:h-[600px]"
3940
/>

client/src/app/features/top/containers/top-page-container.tsx

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -149,7 +149,6 @@ export default function TopPageContainer(): React.ReactElement {
149149
</>
150150
);
151151
}
152-
153152
return (
154153
<TopPage
155154
header={

0 commit comments

Comments
 (0)