[PR #2129] [CLOSED] docs: fix unreadable retrieval pipeline diagram #2157

Closed
opened 2026-02-17 17:23:07 -05:00 by yindo · 0 comments
Owner

📋 Pull Request Information

Original PR: https://github.com/langchain-ai/docs/pull/2129
Author: @YassinNouh21
Created: 1/7/2026
Status: Closed

Base: mainHead: fix/issue-2067-retrieval-diagram


📝 Commits (4)

  • 6bed5d0 docs: fix unreadable retrieval pipeline diagram
  • 76497bf docs: fix unreadable retrieval pipeline diagram
  • 36495ae docs: fix unreadable retrieval pipeline diagram
  • 54bcbf1 docs: fix unreadable retrieval pipeline diagram

📊 Changes

2 files changed (+3 additions, -7 deletions)

View changed files

📝 src/oss/langchain/retrieval.mdx (+3 -0)
📝 src/style.css (+0 -7)

📄 Description

Summary

  • Changed mermaid diagram from horizontal (flowchart LR) to vertical (flowchart TB) layout
  • Added logical subgraphs to organize "Indexing (offline)" and "Retrieval (at query time)" phases
  • The diagram now fits within the viewport and is fully readable

Before

The diagram was using flowchart LR (left-to-right) which made it too wide and got cut off on the right side, making it impossible to read the complete workflow.

After

The diagram uses flowchart TB (top-to-bottom) with labeled subgraphs, making it vertically oriented and fully visible on screen.

Test plan

  • Build completes successfully (make build)
  • Diagram renders correctly in Mintlify preview
  • Diagram is readable on both Chrome and Firefox

Fixes #2067

AI-assisted contribution: This PR was created with assistance from Claude Code.


🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.

## 📋 Pull Request Information **Original PR:** https://github.com/langchain-ai/docs/pull/2129 **Author:** [@YassinNouh21](https://github.com/YassinNouh21) **Created:** 1/7/2026 **Status:** ❌ Closed **Base:** `main` ← **Head:** `fix/issue-2067-retrieval-diagram` --- ### 📝 Commits (4) - [`6bed5d0`](https://github.com/langchain-ai/docs/commit/6bed5d0a58c7b0b909bba9878107e78bf7d962e7) docs: fix unreadable retrieval pipeline diagram - [`76497bf`](https://github.com/langchain-ai/docs/commit/76497bf214089bab8dc83af044f301ff3de08e36) docs: fix unreadable retrieval pipeline diagram - [`36495ae`](https://github.com/langchain-ai/docs/commit/36495ae7670b9975493493c25a5d82b78e294eee) docs: fix unreadable retrieval pipeline diagram - [`54bcbf1`](https://github.com/langchain-ai/docs/commit/54bcbf1a84b10ab7aebaab477b02f73bcc1b7aad) docs: fix unreadable retrieval pipeline diagram ### 📊 Changes **2 files changed** (+3 additions, -7 deletions) <details> <summary>View changed files</summary> 📝 `src/oss/langchain/retrieval.mdx` (+3 -0) 📝 `src/style.css` (+0 -7) </details> ### 📄 Description ## Summary - Changed mermaid diagram from horizontal (`flowchart LR`) to vertical (`flowchart TB`) layout - Added logical subgraphs to organize "Indexing (offline)" and "Retrieval (at query time)" phases - The diagram now fits within the viewport and is fully readable ## Before The diagram was using `flowchart LR` (left-to-right) which made it too wide and got cut off on the right side, making it impossible to read the complete workflow. ## After The diagram uses `flowchart TB` (top-to-bottom) with labeled subgraphs, making it vertically oriented and fully visible on screen. ## Test plan - [x] Build completes successfully (`make build`) - [ ] Diagram renders correctly in Mintlify preview - [ ] Diagram is readable on both Chrome and Firefox Fixes #2067 AI-assisted contribution: This PR was created with assistance from Claude Code. --- <sub>🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.</sub>
yindo added the pull-request label 2026-02-17 17:23:07 -05:00
yindo closed this issue 2026-02-17 17:23:07 -05:00
yindo changed title from [PR #2129] docs: fix unreadable retrieval pipeline diagram to [PR #2129] [CLOSED] docs: fix unreadable retrieval pipeline diagram 2026-06-05 18:16:57 -04:00
Sign in to join this conversation.
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: langchain-ai/docs#2157