fix(BOOK-53): Seo filter block now uses h3 as summary content and adjusted spacing
Approved-by: Bianca Widstam
This commit is contained in:
@@ -38,10 +38,11 @@ export function SeoFilters({ seoFilters, location }: SeoFiltersProps) {
|
|||||||
{ location }
|
{ location }
|
||||||
)}
|
)}
|
||||||
showAsSubtitle
|
showAsSubtitle
|
||||||
|
titleLevel="h3"
|
||||||
>
|
>
|
||||||
<ul className={styles.filterList}>
|
<ul className={styles.filterList}>
|
||||||
{facilityFilters.map(({ filter }) => (
|
{facilityFilters.map(({ filter }) => (
|
||||||
<li key={filter.id}>
|
<li key={filter.id} className={styles.filterItem}>
|
||||||
<Link
|
<Link
|
||||||
href={`${basePath}/${filter.slug}`}
|
href={`${basePath}/${filter.slug}`}
|
||||||
color="Text/Interactive/Secondary"
|
color="Text/Interactive/Secondary"
|
||||||
@@ -61,10 +62,11 @@ export function SeoFilters({ seoFilters, location }: SeoFiltersProps) {
|
|||||||
{ location }
|
{ location }
|
||||||
)}
|
)}
|
||||||
showAsSubtitle
|
showAsSubtitle
|
||||||
|
titleLevel="h3"
|
||||||
>
|
>
|
||||||
<ul className={styles.filterList}>
|
<ul className={styles.filterList}>
|
||||||
{surroundingsFilters.map(({ filter }) => (
|
{surroundingsFilters.map(({ filter }) => (
|
||||||
<li key={filter.id}>
|
<li key={filter.id} className={styles.filterItem}>
|
||||||
<Link
|
<Link
|
||||||
href={`${basePath}/${filter.slug}`}
|
href={`${basePath}/${filter.slug}`}
|
||||||
color="Text/Interactive/Secondary"
|
color="Text/Interactive/Secondary"
|
||||||
|
|||||||
@@ -4,6 +4,10 @@
|
|||||||
gap: var(--Space-x15);
|
gap: var(--Space-x15);
|
||||||
}
|
}
|
||||||
|
|
||||||
|
.filterItem {
|
||||||
|
margin-bottom: var(--Space-x15);
|
||||||
|
}
|
||||||
|
|
||||||
@media screen and (min-width: 1367px) {
|
@media screen and (min-width: 1367px) {
|
||||||
.filterList {
|
.filterList {
|
||||||
column-count: 3;
|
column-count: 3;
|
||||||
|
|||||||
@@ -17,6 +17,7 @@ export interface AccordionItemProps
|
|||||||
extends React.HtmlHTMLAttributes<HTMLDetailsElement>,
|
extends React.HtmlHTMLAttributes<HTMLDetailsElement>,
|
||||||
VariantProps<typeof accordionItemVariants> {
|
VariantProps<typeof accordionItemVariants> {
|
||||||
title: string
|
title: string
|
||||||
|
titleLevel?: 'span' | 'p' | 'h1' | 'h2' | 'h3' | 'h4' | 'h5' | 'h6'
|
||||||
iconName?: IconName
|
iconName?: IconName
|
||||||
icon?: ReactNode
|
icon?: ReactNode
|
||||||
subtitle?: string
|
subtitle?: string
|
||||||
@@ -29,6 +30,7 @@ export default function AccordionItem({
|
|||||||
icon,
|
icon,
|
||||||
iconName,
|
iconName,
|
||||||
title,
|
title,
|
||||||
|
titleLevel = 'p',
|
||||||
type,
|
type,
|
||||||
className,
|
className,
|
||||||
subtitle,
|
subtitle,
|
||||||
@@ -70,6 +72,8 @@ export default function AccordionItem({
|
|||||||
}
|
}
|
||||||
}
|
}
|
||||||
|
|
||||||
|
const TitleLevel = titleLevel
|
||||||
|
|
||||||
return (
|
return (
|
||||||
<li className={accordionItemVariants({ className, type })}>
|
<li className={accordionItemVariants({ className, type })}>
|
||||||
<details ref={detailsRef} onToggle={toggleAccordion}>
|
<details ref={detailsRef} onToggle={toggleAccordion}>
|
||||||
@@ -83,11 +87,11 @@ export default function AccordionItem({
|
|||||||
<div className={styles.title}>
|
<div className={styles.title}>
|
||||||
{subtitle || showAsSubtitle ? (
|
{subtitle || showAsSubtitle ? (
|
||||||
<Typography variant="Title/Subtitle/md">
|
<Typography variant="Title/Subtitle/md">
|
||||||
<p className={styles.title}>{title}</p>
|
<TitleLevel className={styles.title}>{title}</TitleLevel>
|
||||||
</Typography>
|
</Typography>
|
||||||
) : (
|
) : (
|
||||||
<Typography variant="Body/Paragraph/mdBold">
|
<Typography variant="Body/Paragraph/mdBold">
|
||||||
<p className={styles.title}>{title}</p>
|
<TitleLevel className={styles.title}>{title}</TitleLevel>
|
||||||
</Typography>
|
</Typography>
|
||||||
)}
|
)}
|
||||||
{subtitle && (
|
{subtitle && (
|
||||||
|
|||||||
Reference in New Issue
Block a user