corysanin/tubestation
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

Bug 1892772 - Prevent about welcome split title container from rendering when no title or subtitle is present r=omc-reviewers,aminomancer

Browse Source 
Differential Revision: https://phabricator.services.mozilla.com/D208269
This commit is contained in:
Jason Prickett
2024-04-24 20:07:35 +00:00
parent a1519fab0f
commit 3ab800b0c7
3 changed files with 59 additions and 26 deletions
Download Patch File Download Diff File Expand all files Collapse all files

55
browser/components/aboutwelcome/content-src/components/MultiStageProtonScreen.jsx
View File

@@ -570,32 +570,39 @@ export class ProtonScreen extends React.PureComponent {
</div>
) : null}
<div className="main-content-inner">
<div className={`welcome-text ${content.title_style || ""}`}>
{content.title ? this.renderTitle(content) : null}
<div
className="main-content-inner"
style={{
justifyContent: content.split_content_justify_content,
}}
>
{content.title || content.subtitle ? (
<div className={`welcome-text ${content.title_style || ""}`}>
{content.title ? this.renderTitle(content) : null}
{content.subtitle ? (
<Localized text={content.subtitle}>
<h2
data-l10n-args={JSON.stringify({
"addon-name": this.props.addonName,
...this.props.appAndSystemLocaleInfo?.displayNames,
})}
aria-flowto={
this.props.messageId?.includes("FEATURE_TOUR")
? "steps"
: ""
}
{content.subtitle ? (
<Localized text={content.subtitle}>
<h2
data-l10n-args={JSON.stringify({
"addon-name": this.props.addonName,
...this.props.appAndSystemLocaleInfo?.displayNames,
})}
aria-flowto={
this.props.messageId?.includes("FEATURE_TOUR")
? "steps"
: ""
}
/>
</Localized>
) : null}
{content.cta_paragraph ? (
<CTAParagraph
content={content.cta_paragraph}
handleAction={this.props.handleAction}
/>
</Localized>
) : null}
{content.cta_paragraph ? (
<CTAParagraph
content={content.cta_paragraph}
handleAction={this.props.handleAction}
/>
) : null}
</div>
) : null}
</div>
) : null}
{content.video_container ? (
<OnboardingVideo
content={content.video_container}