Skip to content

Commit 3fdaf1a

Browse files
Do not render subnav for interactive articles
1 parent 886f7ac commit 3fdaf1a

File tree

2 files changed

+3
-3
lines changed

2 files changed

+3
-3
lines changed

dotcom-rendering/src/layouts/FullPageInteractiveLayout.tsx

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -143,7 +143,7 @@ const Renderer = ({
143143
return <div css={adStyles}>{output}</div>;
144144
};
145145

146-
const NavHeader = ({ article, NAV, format, renderAds }: HeaderProps) => {
146+
const NavHeader = ({ article, NAV, renderAds }: HeaderProps) => {
147147
return (
148148
<section
149149
/* Note, some interactives require this - e.g. https://www.theguardian.com/environment/ng-interactive/2015/jun/05/carbon-bomb-the-coal-boom-choking-china. */
@@ -177,7 +177,7 @@ const NavHeader = ({ article, NAV, format, renderAds }: HeaderProps) => {
177177
discussionApiUrl={article.config.discussionApiUrl}
178178
idApiUrl={article.config.idApiUrl}
179179
contributionsServiceUrl={article.contributionsServiceUrl}
180-
showSubNav={format.theme !== ArticleSpecial.Labs}
180+
showSubNav={false}
181181
showSlimNav={true}
182182
hasPageSkin={false}
183183
hasPageSkinContentSelfConstrain={false}

dotcom-rendering/src/layouts/InteractiveLayout.tsx

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -288,8 +288,8 @@ export const InteractiveLayout = (props: WebProps | AppsProps) => {
288288
discussionApiUrl={article.config.discussionApiUrl}
289289
idApiUrl={article.config.idApiUrl}
290290
contributionsServiceUrl={contributionsServiceUrl}
291-
showSubNav={format.theme !== ArticleSpecial.Labs}
292291
showSlimNav={true}
292+
showSubNav={false}
293293
hasPageSkin={false}
294294
hasPageSkinContentSelfConstrain={false}
295295
pageId={article.pageId}

0 commit comments

Comments
 (0)