GeezerVison is showing github.com — Docs: new brand look and diagrams by tristencross · Pull Request #7841 · feldera/feldera
Conversation
tristencross
commented
Nov 4, 2026
•
edited
Loading
Uh oh!
There was an error while loading. Please reload this page.
Brings the docs into the new Feldera brand.
- New typography, colours and code blocks (DM Sans, Geist Mono, brand palette).
- All diagrams redrawn in the brand diagram style. Content and layout are unchanged.
- Old logos and magenta buttons updated in a few screenshots.
- Readable "On this page" menu on narrow screens, and a working navbar logo.
Sorry, something went wrong.
Uh oh!
There was an error while loading. Please reload this page.
Karakatiza666 reacted with eyes emoji
* docs: tutorial part 3 supply chain diagram in the brand diagram style
* docs: part 3 diagram, darker Moss on the Feldera output so all arrows read the same weight
* docs: TPC-H schema diagram on batch part 1 in the brand diagram style
* docs: TPC-H schema, PARTSUPP's composite key joins and forks into LINEITEM as in the original; hyphenated rows and centred names
* docs: batch part 2 architecture diagram in the Warp Speed diagram style
* docs: batch part 2 diagram, table-to-view arrows no longer cross
* docs: batch part 3 architecture and Kafka offset diagrams in the Warp Speed diagram style
* docs: Kafka offset diagram, bird centred over LINEITEM and a shorter Feldera frame
…tions in the brand diagram style
…original layout with right-angle edges
…s source"
This reverts commit b54c179.
…ms in the brand diagram style
…) in the brand diagram style
|
|
title: "Feldera Documentation",
|
|
|
tagline: "Powerup your Data Lake with Incremental View Maintenance",
|
|
|
favicon: "img/favicon.ico",
|
|
|
favicon: "img/favicon.svg",
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
bug (medium): there's no static/img/favicon.svg in this PR or in the tree; only favicon.ico exists, and this PR updates it. After a build, build/index.html has <link rel="icon" href="/img/favicon.svg"> and that file returns 404, so the site shows no favicon. Either add the SVG or keep pointing at the .ico:
|
favicon: "img/favicon.svg", |
|
favicon: "img/favicon.ico", |
Sorry, something went wrong.
Uh oh!
There was an error while loading. Please reload this page.
|
|
// see custom.css --feldera-code-bg); worst case is now 6.10:1.
|
|
|
// Code block theme: the website's dark code window. The colours come from
|
|
|
// felderaDarkTheme (website2/src/utils/shiki-theme.mjs), mapped to Prism
|
|
|
// token types. The ground is Onyx (website2/docs/design-system.md, section 9).
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
tech-debt (medium): this public repo has no website2/... paths. The same goes for the references in src/css/custom.css (lines 3 and 361). Contributors can't follow them. Inline the values or rationale, or drop the pointers.
Sorry, something went wrong.
Uh oh!
There was an error while loading. Please reload this page.
|
This PR changes only the docs site: a new brand theme, fonts and code theme, plus redrawn diagrams. No product code changes, so the test gates don't apply.
What I ran
| Check |
Result |
yarn install --frozen-lockfile (docs.feldera.com) |
✅ |
yarn build |
✅ builds. The broken-anchor warnings also appear without this PR |
pre-commit run --files <changed> |
✅ |
Favicon in the built index.html |
❌ points to /img/favicon.svg, which doesn't exist (see inline) |
Findings
- licensing (medium): the new
@fontsource-variable/dm-sans and @fontsource-variable/geist-mono packages are OFL-1.1, not MIT or Apache-2.0. OFL is fine for web fonts on a docs site, but under our dependency rule it needs an explicit sign-off.
- tech-debt (medium): the PR leaves orphaned assets:
real_time_apps/spreadsheet-architecture-parts.svg, model_scoring/model-scoring-architecture.{svg,dot} and static/img/logo-color-{dark,light}.svg. Nothing references them any more. A commit in this PR removes the model-scoring .svg/.dot and the next one reverts it, so please decide whether to keep them and squash that pair.
- commits (medium): several subject lines are cut off with
…. There's also a remove/revert pair, and a commit that removes the source a diagram is generated from would need a reason in its message. Please squash and reword before merge.
|
Sorry, something went wrong.
Uh oh!
There was an error while loading. Please reload this page.
|
Looks off: too blue and thin. The color may be fine though. And too much space above
Vanilla scrollbar. let's come up with something nicer

Try styling the Python SDK pages

|
Sorry, something went wrong.
Uh oh!
There was an error while loading. Please reload this page.
Karakatiza666
left a comment
•
edited
Loading
Uh oh!
There was an error while loading. Please reload this page.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
The styling code became messier. Will have to be addressed in later PRs
Sorry, something went wrong.
Uh oh!
There was an error while loading. Please reload this page.
Uh oh!
There was an error while loading. Please reload this page.
Uh oh!
There was an error while loading. Please reload this page.
Assignees
No one assigned
Milestone
No milestone
Development
Successfully merging this pull request may close these issues.
Uh oh!
There was an error while loading. Please reload this page.
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
|