Skip to content

docs: center diagram images in native model client dev note#483

Merged
nabinchha merged 1 commit intomainfrom
nmulepati/fix/model-client-dev-note-display
Mar 31, 2026
Merged

docs: center diagram images in native model client dev note#483
nabinchha merged 1 commit intomainfrom
nmulepati/fix/model-client-dev-note-display

Conversation

@nabinchha
Copy link
Copy Markdown
Contributor

📋 Summary

Centers the four diagram images in the native model client dev note so they display properly on wide monitors instead of being left-aligned.

🔄 Changes

🔧 Changed

  • Wrapped diagram images in <div style="text-align: center;" markdown> containers in owning-the-model-stack.md (native-model-client-layers, aimd-concurrency-over-time, throttle-keying, retry-boundary)
  • Hero image left unchanged (full-width by design)

🤖 Generated with AI

Made with Cursor

Wrap non-hero images in text-align:center divs so they
display centered on wide monitors instead of left-aligned.

Made-with: Cursor
@nabinchha nabinchha requested a review from a team as a code owner March 31, 2026 22:02
@greptile-apps
Copy link
Copy Markdown
Contributor

greptile-apps bot commented Mar 31, 2026

Greptile Summary

This PR centers four diagram images in the native model client dev note by wrapping them in <div style="text-align: center;" markdown> containers, preventing them from appearing left-aligned on wide monitors. The hero image is intentionally left unchanged as a full-width element.

  • The md_in_html markdown extension is enabled in mkdocs.yml (line 178), which correctly supports the markdown attribute on div tags to process Markdown content within HTML blocks.
  • The blank lines inside each div ensure images are parsed as block-level Markdown elements, which is required by the md_in_html extension.
  • All four wrapped images already carry { style=\"max-width:75%; height:auto\" } attributes, so text-align: center will center the inline image element within its paragraph as expected.
  • The pattern is consistent with the project's use of centered divs in deep-research-trajectories.md (which uses display: flex; justify-content: center; — a stylistically equivalent approach).

No functional or correctness issues found.

Confidence Score: 5/5

Safe to merge — pure documentation formatting change with no logic, code, or config modifications.

The change is a straightforward docs-only update. The md_in_html extension is confirmed enabled in mkdocs.yml, the markdown attribute syntax is correct, blank lines inside the divs are properly placed, and the pattern mirrors what other dev notes in this project already do. No P0/P1 issues found.

No files require special attention.

Important Files Changed

Filename Overview
docs/devnotes/posts/owning-the-model-stack.md Wraps four architecture diagram images in <div style="text-align: center;" markdown> containers for centered display; hero image intentionally left unchanged; uses correct md_in_html syntax with proper blank lines inside divs

Flowchart

%%{init: {'theme': 'neutral'}}%%
flowchart TD
    A["Markdown source\n![img](path){ style=max-width:75% }"] --> B["md_in_html extension\n(mkdocs.yml line 178)"]
    B --> C{"Wrapped in\ndiv markdown?"}
    C -- "Before this PR\n(no wrapper)" --> D["Browser renders img\nleft-aligned on wide screens"]
    C -- "After this PR\ndiv style=text-align:center markdown" --> E["text-align:center\napplied to parent div"]
    E --> F["Browser centers inline img\ninside paragraph block"]
    F --> G["Image centered on wide monitors ✓"]
    D -.->|"hero image\n(intentionally unchanged)"| H["Full-width hero\nleft-aligned by design"]
Loading

Reviews (1): Last reviewed commit: "docs: center diagram images in native mo..." | Re-trigger Greptile

@nabinchha nabinchha merged commit 7a05a83 into main Mar 31, 2026
47 checks passed
@nabinchha nabinchha deleted the nmulepati/fix/model-client-dev-note-display branch March 31, 2026 22:05
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