Skip to content

Commit 8fadc33

Browse files
authored
fix(plugin-seo): titles being displayed twice (#8310)
1 parent c6519ab commit 8fadc33

File tree

6 files changed

+31
-9
lines changed

6 files changed

+31
-9
lines changed

packages/plugin-seo/src/fields/MetaDescription/MetaDescriptionComponent.tsx

Lines changed: 7 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -100,7 +100,13 @@ export const MetaDescriptionComponent: React.FC<MetaDescriptionProps> = (props)
100100
}}
101101
>
102102
<div className="plugin-seo__field">
103-
<FieldLabel field={null} Label={Label} label={label} {...(labelProps || {})} />
103+
<FieldLabel
104+
field={null}
105+
Label={Label}
106+
label={label}
107+
required={required}
108+
{...(labelProps || {})}
109+
/>
104110
{hasGenerateDescriptionFn && (
105111
<React.Fragment>
106112
&nbsp; &mdash; &nbsp;
@@ -151,7 +157,6 @@ export const MetaDescriptionComponent: React.FC<MetaDescriptionProps> = (props)
151157
Component: null,
152158
RenderedComponent: errorMessage,
153159
}}
154-
label={label}
155160
onChange={setValue}
156161
path={pathFromContext}
157162
required={required}

packages/plugin-seo/src/fields/MetaImage/MetaImageComponent.tsx

Lines changed: 7 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -102,7 +102,13 @@ export const MetaImageComponent: React.FC<MetaImageProps> = (props) => {
102102
}}
103103
>
104104
<div className="plugin-seo__field">
105-
<FieldLabel field={null} Label={Label} label={label} {...(labelProps || {})} />
105+
<FieldLabel
106+
field={null}
107+
Label={Label}
108+
label={label}
109+
required={required}
110+
{...(labelProps || {})}
111+
/>
106112
{hasGenerateImageFn && (
107113
<React.Fragment>
108114
&nbsp; &mdash; &nbsp;
@@ -151,7 +157,6 @@ export const MetaImageComponent: React.FC<MetaImageProps> = (props) => {
151157
RenderedComponent: errorMessage,
152158
}}
153159
filterOptions={field.filterOptions}
154-
label={undefined}
155160
onChange={(incomingImage) => {
156161
if (incomingImage !== null) {
157162
if (typeof incomingImage === 'object') {

packages/plugin-seo/src/fields/MetaTitle/MetaTitleComponent.tsx

Lines changed: 8 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -37,11 +37,11 @@ export const MetaTitleComponent: React.FC<MetaTitleProps> = (props) => {
3737
label,
3838
required,
3939
},
40+
field: fieldFromProps,
4041
hasGenerateTitleFn,
4142
labelProps,
4243
} = props || {}
4344
const { path: pathFromContext } = useFieldProps()
44-
4545
const { t } = useTranslation<PluginSEOTranslations, PluginSEOTranslationKeys>()
4646

4747
const field: FieldType<string> = useField({
@@ -98,7 +98,13 @@ export const MetaTitleComponent: React.FC<MetaTitleProps> = (props) => {
9898
}}
9999
>
100100
<div className="plugin-seo__field">
101-
<FieldLabel field={null} Label={Label} label={label} {...(labelProps || {})} />
101+
<FieldLabel
102+
field={fieldFromProps}
103+
Label={Label}
104+
label={label}
105+
required={required}
106+
{...(labelProps || {})}
107+
/>
102108
{hasGenerateTitleFn && (
103109
<React.Fragment>
104110
&nbsp; &mdash; &nbsp;
@@ -150,7 +156,6 @@ export const MetaTitleComponent: React.FC<MetaTitleProps> = (props) => {
150156
Component: null,
151157
RenderedComponent: errorMessage,
152158
}}
153-
label={label}
154159
onChange={setValue}
155160
path={pathFromContext}
156161
required={required}

packages/plugin-seo/src/fields/Preview/PreviewComponent.tsx

Lines changed: 5 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -78,7 +78,11 @@ export const PreviewComponent: React.FC<PreviewProps> = (props) => {
7878
}, [fields, href, locale, docInfo, hasGenerateURLFn, getData])
7979

8080
return (
81-
<div>
81+
<div
82+
style={{
83+
marginBottom: '20px',
84+
}}
85+
>
8286
<div>{t('plugin-seo:preview')}</div>
8387
<div
8488
style={{

packages/plugin-seo/src/index.tsx

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -28,7 +28,7 @@ export const seoPlugin =
2828
OverviewField({}),
2929
MetaTitleField({
3030
hasGenerateFn: typeof pluginConfig?.generateTitle === 'function',
31-
overrides: pluginConfig?.fieldOverrides?.title as unknown as TextField,
31+
overrides: pluginConfig?.fieldOverrides?.title,
3232
}),
3333
MetaDescriptionField({
3434
hasGenerateFn: typeof pluginConfig?.generateDescription === 'function',

test/plugin-seo/collections/PagesWithImportedFields.ts

Lines changed: 3 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -65,6 +65,9 @@ export const PagesWithImportedFields: CollectionConfig = {
6565
fields: [
6666
MetaTitleField({
6767
hasGenerateFn: true,
68+
overrides: {
69+
required: true,
70+
},
6871
}),
6972
PreviewField({
7073
hasGenerateFn: true,

0 commit comments

Comments
 (0)