feat(layout): Refractoring

[AugustinMauroy]

Description

• Remove handwritten ToC, and use metabar
• Clean and merge some layout to simplify our logic.

Related Issues

It's one solution for #6922

Check List

• I have read the Contributing Guidelines and made commit messages that follow the guideline.
• I have run npm run format to ensure the code follows the style guide.
• I have run npm run test to check if all tests are passing.
• I have run npx turbo build to check if the website builds without errors.

[vercel]

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name	Status	Preview	Updated (UTC)
nodejs-org	✅ Ready (Inspect)	Visit Preview	Oct 9, 2024 5:59pm

[ovflowd]

Is this still a draft?

[AugustinMauroy]

Is this still a draft?

Yes and no. Basically, the component and file names are clearly not the right ones. But I have no idea.
But from a code point of view nothing will change in my opinion

[bmuenzenmeyer]

Is the only thing blocking this the naming?

[AugustinMauroy]

Yeah 😅 I'm not fan about how layout is named

[mikeesto]

ArticlePageLayout?

[bmuenzenmeyer]

I'm working on this addition to the COLLABORATOR_GUIDE. I was hoping it would help us identify the SimplePage uniqueness, and perhaps simplify our layouts - we have a LOT

Layouts

There are several Layouts available within the Website. Here's a comparison of their features:

Layout	Includes Layout...	WithNavBar	WithSidebar	WithFooter	WithMetaBar	WithBreadcrumbs	Comment / Custom Element(s)
Default	Article	✅	✅	✅	❌	❌
About	Article	✅	✅	✅	✅	✅
SimplePage	Article	✅	✅	❌	❌	❌
Learn	Article	✅	❌	✅	✅	✅	WithSidebarCrossLinks
Post	Content	✅	❌	✅	✅	❌	WithBlogCrossLinks
Blog		✅	❌	✅	❌	❌	WithBlogCategories
Download		✅	❌	✅	❌	❌	WithDownloadCategories
Search		✅	❌	✅	❌	❌
Centered		✅	❌	❌	❌	❌
Base		❌	❌	❌	❌	❌	NotificationProvider and NavigationStateProvider
Home	Centered	❌	❌	❌	❌	❌	GlowingBackdrop
Article		❌	❌	❌	❌	❌	CSS Only
Content		❌	❌	❌	❌	❌	CSS Only

[AugustinMauroy]

Centered and home can be merge; and content with post ?

[ovflowd]

Centered and home can be merge; and content with post ?

If it works, it works. Do it.

[github-actions]

Unit Test Coverage Report

Lines	Statements	Branches	Functions
	90.47% (589/651)	76.29% (177/232)	94.35% (117/124)

Unit Test Report

Tests	Skipped	Failures	Errors	Time
131	0 💤	0 ❌	0 🔥	5.329s ⏱️

[canerakdas]

This PR solves more than one problem (#7062, #6922) and simplifies the layouts, do you have time to resolve the conflict @AugustinMauroy 🙇

I have availability this week and I can help too if you want 👀

[AugustinMauroy]

@canerakdas it's not overcomplicated just need to open spotify 😁

[github-actions]

Lighthouse Results

URL	Performance	Accessibility	Best Practices	SEO	Report
/en	🟢 99	🟢 100	🟢 100	🟢 91	🔗
/en/about	🟢 99	🟢 100	🟢 100	🟢 91	🔗
/en/about/previous-releases	🟢 100	🟢 100	🟢 100	🟢 92	🔗
/en/download	🟢 100	🟢 100	🟢 100	🟢 91	🔗
/en/blog	🟢 99	🟢 100	🟢 100	🟢 92	🔗

[canerakdas]

LGTM 🎵 I checked the layouts in general for desktop and mobile resolution. Thank you @AugustinMauroy