Nextjs readme by danielmarv · Pull Request #273 · OpenElements/open-elements-website

danielmarv · 2026-02-08T22:00:28Z

This pull request adds a new localized Impressum page to the application, supporting both German and English content. The page dynamically generates metadata based on the user's locale and renders appropriate legal information for each language.

New Impressum page implementation:

Added a new ImpressumPage component in src/app/[locale]/impressum/page.tsx that displays legal information in either German or English, depending on the locale. (src/app/[locale]/impressum/page.tsxR1-R92)
Implemented the generateMetadata function to provide localized metadata (title, description, OpenGraph tags) for SEO and social sharing. (src/app/[locale]/impressum/page.tsxR1-R92)

Localization and content rendering:

Dynamically selects and renders locale-specific HTML content for the Impressum, including address, contact, and legal details. (src/app/[locale]/impressum/page.tsxR1-R92)
Uses the next/image component to display a localized underline illustration below the page heading. (src/app/[locale]/impressum/page.tsxR1-R92)
Applies responsive and styled layout using Tailwind CSS utility classes for consistent appearance across devices. (src/app/[locale]/impressum/page.tsxR1-R92)

Copilot

Pull request overview

Adds a new /impressum route under the localized app router segment, intended to display locale-specific legal information and provide localized SEO metadata.

Changes:

Added src/app/[locale]/impressum/page.tsx implementing the Impressum page for DE/EN.
Implemented generateMetadata for the new page, including Open Graph metadata.
Rendered DE/EN Impressum content (currently via an HTML string + dangerouslySetInnerHTML).

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Copilot · 2026-02-08T22:03:28Z

src/app/[locale]/impressum/page.tsx

+export async function generateMetadata({ params }: ImpressumPageProps): Promise<Metadata> {
+  const { locale } = await params
+
+  const title = locale === 'de' ? 'Impressum - Open Elements' : 'Impressum - Open Elements'
+  const description = locale === 'de' ? 'Impressum' : 'Impressum'
+


generateMetadata uses locale conditionals, but both branches return identical title and description, so metadata isn’t actually localized (beyond openGraph.locale). This also makes the meta description extremely short, which is inconsistent with the repo’s SEO expectations (see tests/e2e/seo.spec.ts where descriptions are expected to be meaningful). Consider providing distinct EN/DE title/description strings (and a longer description) or removing the unnecessary conditional if localization isn’t intended here.

Copilot · 2026-02-08T22:03:29Z

src/app/[locale]/impressum/page.tsx

+            <h1 className="text-center h1">Impressum</h1>
+            <Image


The page heading is hard-coded to "Impressum" for all locales. For the English locale the rest of the content is in English, so the H1 should likely be localized as well (e.g., "Imprint" / "Legal notice") to avoid a mixed-language page.

Copilot · 2026-02-08T22:03:29Z

src/app/[locale]/impressum/page.tsx

+          <div
+            className="text-blue prose prose-a:text-purple-700 prose-code:bg-yellow prose-blockquote:border-l-0 prose-blockquote:bg-green-100 prose-blockquote:not-italic prose-blockquote:px-8 prose-blockquote:py-3 prose-blockquote:rounded-3xl relative"
+            style={{ maxWidth: '100%' }}
+            dangerouslySetInnerHTML={{ __html: contentHtml }}
+          />


dangerouslySetInnerHTML is being used to render static, hard-coded content. Since this isn’t coming from markdown or a CMS, rendering as JSX (or at least storing the content as React nodes) would be safer and easier to maintain, and avoids normalizing dangerouslySetInnerHTML usage for simple content.

Copilot · 2026-02-08T22:03:29Z

src/app/[locale]/impressum/page.tsx

+export async function generateMetadata({ params }: ImpressumPageProps): Promise<Metadata> {
+  const { locale } = await params
+
+  const title = locale === 'de' ? 'Impressum - Open Elements' : 'Impressum - Open Elements'
+  const description = locale === 'de' ? 'Impressum' : 'Impressum'
+
+  return {
+    title,
+    description,
+    openGraph: {
+      type: 'website',
+      title,
+      description,
+      siteName: 'Open Elements',
+      locale: locale === 'de' ? 'de_DE' : 'en_US',
+    },
+  }
+}


There’s Playwright coverage for SEO across a fixed set of routes (tests/e2e/seo.spec.ts), but the new impressum route isn’t included, so regressions in title/description/OpenGraph tags won’t be caught. Consider extending the existing SEO (and/or navigation) E2E tests to include /impressum for both locales.

Signed-off-by: Daniel Ntege <danientege785@gmail.com>

…ript loading Signed-off-by: Daniel Ntege <danientege785@gmail.com>

…lean up layout files Signed-off-by: Daniel Ntege <danientege785@gmail.com>

Signed-off-by: Daniel Ntege <danientege785@gmail.com>

…ocale Signed-off-by: Daniel Ntege <danientege785@gmail.com>

Signed-off-by: Jessie Ssebuliba <jessiessebuliba@gmail.com> Signed-off-by: Daniel Ntege <danientege785@gmail.com>

- Added green-arrow.svg with a detailed arrow design and gradients. - Introduced plane-sm.svg featuring a stylized plane with dashed lines. - Created plane.svg showcasing a larger plane illustration with gradient strokes. - Added round11.svg containing two circles with varying opacities. - Introduced sky-round.svg featuring a large circle with a light blue fill. Signed-off-by: Daniel Ntege <danientege785@gmail.com>

…logs - Created German and English markdown files for updates v0.1.7 and v0.1.8. - Implemented new components for displaying updates, including collapsible sections and truncated views. - Added functionality to parse markdown files and extract update information. - Introduced a new data structure for updates, including contributors and assets. - Enhanced the updates page with metadata generation and localization support. Signed-off-by: Daniel Ntege <danientege785@gmail.com>

Signed-off-by: Jessie Ssebuliba <jessiessebuliba@gmail.com> Signed-off-by: Daniel Ntege <danientege785@gmail.com>

- Added green-arrow.svg with a detailed arrow design and gradients. - Introduced plane-sm.svg featuring a stylized plane with dashed lines. - Created plane.svg showcasing a larger plane illustration with gradient strokes. - Added round11.svg containing two circles with varying opacities. - Introduced sky-round.svg featuring a large circle with a light blue fill. Signed-off-by: Daniel Ntege <danientege785@gmail.com>

…logs - Created German and English markdown files for updates v0.1.7 and v0.1.8. - Implemented new components for displaying updates, including collapsible sections and truncated views. - Added functionality to parse markdown files and extract update information. - Introduced a new data structure for updates, including contributors and assets. - Enhanced the updates page with metadata generation and localization support. Signed-off-by: Daniel Ntege <danientege785@gmail.com>

Signed-off-by: Daniel Ntege <danientege785@gmail.com>

…ript loading Signed-off-by: Daniel Ntege <danientege785@gmail.com>

…lean up layout files Signed-off-by: Daniel Ntege <danientege785@gmail.com>

Signed-off-by: Daniel Ntege <danientege785@gmail.com>

…ocale Signed-off-by: Daniel Ntege <danientege785@gmail.com>

Signed-off-by: Jessie Ssebuliba <jessiessebuliba@gmail.com> Signed-off-by: Daniel Ntege <danientege785@gmail.com>

- Added green-arrow.svg with a detailed arrow design and gradients. - Introduced plane-sm.svg featuring a stylized plane with dashed lines. - Created plane.svg showcasing a larger plane illustration with gradient strokes. - Added round11.svg containing two circles with varying opacities. - Introduced sky-round.svg featuring a large circle with a light blue fill. Signed-off-by: Daniel Ntege <danientege785@gmail.com>

…logs - Created German and English markdown files for updates v0.1.7 and v0.1.8. - Implemented new components for displaying updates, including collapsible sections and truncated views. - Added functionality to parse markdown files and extract update information. - Introduced a new data structure for updates, including contributors and assets. - Enhanced the updates page with metadata generation and localization support. Signed-off-by: Daniel Ntege <danientege785@gmail.com>

Signed-off-by: Jessie Ssebuliba <jessiessebuliba@gmail.com> Signed-off-by: Daniel Ntege <danientege785@gmail.com>

- Added green-arrow.svg with a detailed arrow design and gradients. - Introduced plane-sm.svg featuring a stylized plane with dashed lines. - Created plane.svg showcasing a larger plane illustration with gradient strokes. - Added round11.svg containing two circles with varying opacities. - Introduced sky-round.svg featuring a large circle with a light blue fill. Signed-off-by: Daniel Ntege <danientege785@gmail.com>

…logs - Created German and English markdown files for updates v0.1.7 and v0.1.8. - Implemented new components for displaying updates, including collapsible sections and truncated views. - Added functionality to parse markdown files and extract update information. - Introduced a new data structure for updates, including contributors and assets. - Enhanced the updates page with metadata generation and localization support. Signed-off-by: Daniel Ntege <danientege785@gmail.com>

Signed-off-by: Daniel Ntege <danientege785@gmail.com>

…elements-website into nextjs-readme

danielmarv self-assigned this Feb 8, 2026

Copilot AI review requested due to automatic review settings February 8, 2026 22:00

danielmarv added this to Migration to NextJS Feb 8, 2026

github-project-automation bot moved this to Backlog in Migration to NextJS Feb 8, 2026

Copilot started reviewing on behalf of danielmarv February 8, 2026 22:00 View session

Copilot AI reviewed Feb 8, 2026

View reviewed changes

danielmarv and others added 14 commits February 9, 2026 01:07

feat: test automation

dc02f43

Signed-off-by: Daniel Ntege <danientege785@gmail.com>

feat: test automation

20b7d03

Signed-off-by: Daniel Ntege <danientege785@gmail.com>

feat: test automation

49db125

Signed-off-by: Daniel Ntege <danientege785@gmail.com>

feat: update pre-commit hook to check for pnpm before running lint

b5aaf1f

Signed-off-by: Daniel Ntege <danientege785@gmail.com>

feat: update pre-commit hook, fix lint command, and enhance layout sc…

e0e6b2c

…ript loading Signed-off-by: Daniel Ntege <danientege785@gmail.com>

refactor: remove commented-out code in About and Contact pages, and c…

9c85631

…lean up layout files Signed-off-by: Daniel Ntege <danientege785@gmail.com>

docs: update README to clarify project structure and deployment process

390310f

Signed-off-by: Daniel Ntege <danientege785@gmail.com>

feat: add Impressum page with dynamic metadata and content based on l…

0a007c3

…ocale Signed-off-by: Daniel Ntege <danientege785@gmail.com>

language switcher should not point to the home page

a304c86

Signed-off-by: Jessie Ssebuliba <jessiessebuliba@gmail.com> Signed-off-by: Daniel Ntege <danientege785@gmail.com>

language switcher should not point to the home page

6175112

Signed-off-by: Jessie Ssebuliba <jessiessebuliba@gmail.com> Signed-off-by: Daniel Ntege <danientege785@gmail.com>

danielmarv force-pushed the nextjs-readme branch from fcf8566 to 70f5b02 Compare February 8, 2026 22:10

Merge branch 'next-js-migration' into nextjs-readme

d811d54

danielmarv mentioned this pull request Feb 8, 2026

Provide OSS-Updates Page #244

Open

danielmarv added 7 commits February 9, 2026 01:14

fix: remove duplicate import of Script in layout.tsx

5b41d3a

feat: test automation

3ebc17a

Signed-off-by: Daniel Ntege <danientege785@gmail.com>

feat: test automation

d37cb5a

Signed-off-by: Daniel Ntege <danientege785@gmail.com>

feat: test automation

1c974b3

Signed-off-by: Daniel Ntege <danientege785@gmail.com>

feat: update pre-commit hook to check for pnpm before running lint

5fe0c6f

Signed-off-by: Daniel Ntege <danientege785@gmail.com>

feat: update pre-commit hook, fix lint command, and enhance layout sc…

8bea3e4

…ript loading Signed-off-by: Daniel Ntege <danientege785@gmail.com>

refactor: remove commented-out code in About and Contact pages, and c…

789ea51

…lean up layout files Signed-off-by: Daniel Ntege <danientege785@gmail.com>

danielmarv and others added 11 commits February 9, 2026 01:18

docs: update README to clarify project structure and deployment process

4201d6d

Signed-off-by: Daniel Ntege <danientege785@gmail.com>

feat: add Impressum page with dynamic metadata and content based on l…

dc90289

…ocale Signed-off-by: Daniel Ntege <danientege785@gmail.com>

language switcher should not point to the home page

1ef675f

Signed-off-by: Jessie Ssebuliba <jessiessebuliba@gmail.com> Signed-off-by: Daniel Ntege <danientege785@gmail.com>

language switcher should not point to the home page

bf6aa75

Signed-off-by: Jessie Ssebuliba <jessiessebuliba@gmail.com> Signed-off-by: Daniel Ntege <danientege785@gmail.com>

fix: remove duplicate import of Script in layout.tsx

b870383

Signed-off-by: Daniel Ntege <danientege785@gmail.com>

fix: add missing import of Script in layout.tsx

129712a

Merge branch 'nextjs-readme' of https://github.com/OpenElements/open-…

864d55d

…elements-website into nextjs-readme

danielmarv merged commit 479ffff into next-js-migration Feb 8, 2026
1 of 2 checks passed

github-project-automation bot moved this from Backlog to Done in Migration to NextJS Feb 8, 2026

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

Nextjs readme#273

Nextjs readme#273
danielmarv merged 33 commits intonext-js-migrationfrom
nextjs-readme

danielmarv commented Feb 8, 2026

Uh oh!

Copilot AI left a comment

Uh oh!

Copilot AI Feb 8, 2026

Uh oh!

Copilot AI Feb 8, 2026

Uh oh!

Copilot AI Feb 8, 2026

Uh oh!

Copilot AI Feb 8, 2026

Uh oh!

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

2 participants

Conversation

danielmarv commented Feb 8, 2026

Uh oh!

Copilot AI left a comment

Choose a reason for hiding this comment

Pull request overview

Uh oh!

Copilot AI Feb 8, 2026

Choose a reason for hiding this comment

Uh oh!

Copilot AI Feb 8, 2026

Choose a reason for hiding this comment

Uh oh!

Copilot AI Feb 8, 2026

Choose a reason for hiding this comment

Uh oh!

Copilot AI Feb 8, 2026

Choose a reason for hiding this comment

Uh oh!

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

2 participants