Skip to content

feat(home): Enhance home section with new illustrations and translations - #833

Merged
aXenDeveloper merged 3 commits into
canaryfrom
feat(home)--Enhance-home-section-with-new-illustrations-and-translations
Sep 30, 2026
Merged

aXenDeveloper merged 3 commits into
canaryfrom
feat(home)--Enhance-home-section-with-new-illustrations-and-translations

Conversation

@aXenDeveloper

Copy link
Copy Markdown
Owner

Improving Documentation

Description

What?

Why?

@github-actions github-actions Bot added the 💡 Feature A new feature label Sep 29, 2026

@chatgpt-codex-connector chatgpt-codex-connector Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

💡 Codex Review

Here are some automated review suggestions for this pull request.

Reviewed commit: 09edce6d80

ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".

Comment on lines +10 to +14
await context.queryClient.query({
...intlQueryOptions({
locale: context.locale,
namespaces: HOME_NAMESPACES,
}),

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

P1 Badge Move the home query out of the route file

Keep this file topology-only by exposing a home route loader/query contract from the TanStack feature module instead of constructing intlQueryOptions here and importing an app-local namespace list. The current arrangement couples the host route to the screen's query and namespace implementation, contrary to the repository's route-module boundary.

AGENTS.md reference: AGENTS.md:L26-L28

Useful? React with 👍 / 👎.

Comment on lines +131 to +133
<ScreenFrame
priority
screen={{ ...SCREENS.dashboard, alt: t('alt') }}

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

P2 Badge Avoid eagerly fetching both themed preview images

On every home-page load, priority is forwarded by ScreenFrame to both its light and dark <img> elements, making both screenshots eager and high-priority even though CSS hides one of them. The unused theme asset therefore competes with the visible LCP image; render the variants through mutually exclusive media sources or reserve eager priority for only the active variant.

Useful? React with 👍 / 👎.

@chatgpt-codex-connector chatgpt-codex-connector Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

💡 Codex Review

Here are some automated review suggestions for this pull request.

Reviewed commit: e6057bbc6e

ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".

{
blurb: 'A fast, typed API with OpenAPI docs served at /api/swagger.',
category: 'API',
color: '#e36002',

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

P1 Badge Reduce the integration palette to 3–5 colors

The new integration cards assign 11 distinct hex brand colors across this list, before counting the semantic foreground/background colors. This exceeds the repository’s required 3–5-color palette and makes the section visually inconsistent with the documented design system; consolidate these accents into a small shared palette.

AGENTS.md reference: AGENTS.md:L64-L64

Useful? React with 👍 / 👎.

@aXenDeveloper
aXenDeveloper merged commit 7cba00e into canary Sep 30, 2026
5 checks passed
@aXenDeveloper
aXenDeveloper deleted the feat(home)--Enhance-home-section-with-new-illustrations-and-translations branch September 30, 2026 13:46
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

💡 Feature A new feature

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant