Skip to content

Fix slow Iris concepts diagrams and oversized service-key screenshots - #4141

Merged
mich-elle-luna merged 1 commit into
mainfrom
iris-image-fix
Sep 28, 2026
Merged

mich-elle-luna merged 1 commit into
mainfrom
iris-image-fix

Conversation

@mich-elle-luna

@mich-elle-luna mich-elle-luna commented Sep 28, 2026 •

Copy link
Copy Markdown
Collaborator
  • Replace client-side Mermaid diagrams on the five Iris concepts pages with pre-rendered static SVGs (matching the site's mermaid theme colors), removing the runtime fetch of the Mermaid library from the jsdelivr CDN and the client-side layout pass on every page load.

  • Halve several oversized "service key generated" and "admin/API key" screenshots across the LangCache, Context Retriever, and Agent Memory operate pages.


Note

Low Risk
Documentation, static assets, and presentational CSS/JS only; no runtime product or security behavior changes.

Overview
This PR speeds up Iris concept pages and aligns interactive diagram styling with the new static art.

Concept diagrams: Five context-engine pages no longer embed Mermaid blocks that pulled the library from a CDN and laid out diagrams in the browser. Each flow is now a pre-rendered SVG under static/images/ai/context-engine/, referenced with the image shortcode (same content, fixed sizing and alt text).

Context map look: Interactive context map nodes no longer use a flat redis-red-500 stroke. context-map.js defines #context-map-border-gradient, and index.css uses that gradient on default, hover, selected, and focus-visible states so borders match the Mermaid “neo” diagrams.

Operate screenshots: Service-key and admin/API-key dialog screenshots on LangCache, Agent Memory, and Context Retriever guides are narrowed (roughly 80% → ~37.5–40%) so they don’t dominate the page.

Reviewed by Cursor Bugbot for commit 7643540. Bugbot is set up for automated code reviews on this repo. Configure here.

- Replace client-side Mermaid diagrams on the five Iris concepts pages
  with pre-rendered static SVGs (matching the site's mermaid theme
  colors), removing the runtime fetch of the Mermaid library from the
  jsdelivr CDN and the client-side layout pass on every page load.
- Restyle the request-flow page's context-map node borders to use the
  same red-to-grey gradient as the mermaid-derived "neo" look diagrams,
  instead of the flat brand redis-red-500, so they read as one system.
- Halve several oversized "service key generated" and "admin/API key"
  screenshots across the LangCache, Context Retriever, and Agent Memory
  operate pages.
@mich-elle-luna
mich-elle-luna requested a review from a team September 28, 2026 19:25
@mich-elle-luna mich-elle-luna self-assigned this Sep 28, 2026
@github-actions

Copy link
Copy Markdown
Contributor

🧠 Redis Memory

Found 5 related items from repository history (5 new this commit):

Memory updated at 7643540

@cursor cursor 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.

Cursor Bugbot has reviewed your changes using high effort and found 1 potential issue.

Fix All in Cursor

❌ Bugbot Autofix is OFF. To automatically fix reported issues with cloud agents, have a team admin enable autofix in the Cursor dashboard.

Reviewed by Cursor Bugbot for commit 7643540. Configure here.

Comment thread assets/css/index.css
/* Red-to-grey diagonal gradient (defined in context-map.js), matching the
node border on the mermaid-derived diagrams elsewhere on these pages
(mermaid's "neo" look), not the flat brand redis-red-500. */
stroke: url(#context-map-border-gradient);

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Context map borders fail in Firefox

Medium Severity

The new stroke: url(#context-map-border-gradient) rules sit in the external index.css stylesheet, while the gradient is defined on the inline context-map SVG. Firefox resolves fragment URLs in CSS against the stylesheet, so the paint server is not found and node outlines do not paint. White fills then blend into the page, leaving only labels and edges.

Additional Locations (2)
Fix in Cursor Fix in Web

Reviewed by Cursor Bugbot for commit 7643540. Configure here.

@mich-elle-luna
mich-elle-luna merged commit 4307e46 into main Sep 28, 2026
100 checks passed
@mich-elle-luna
mich-elle-luna deleted the iris-image-fix branch September 28, 2026 20:46
EliShteinman added a commit to EliShteinman/docs that referenced this pull request Sep 29, 2026
Replaces the Iris concepts Mermaid diagrams with static SVGs and shrinks oversized service-key screenshots.
EliShteinman added a commit to EliShteinman/docs that referenced this pull request Sep 29, 2026
Replaces the Iris concepts Mermaid diagrams with static SVGs and shrinks oversized service-key screenshots.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants