Skip to content

fix(s09_memory): point read/write arrow at memory files box#339

Merged
Gui-Yue merged 1 commit into
shareAI-lab:mainfrom
hardness1020:fix/memory-subsystems-arrow-direction
Jun 2, 2026
Merged

fix(s09_memory): point read/write arrow at memory files box#339
Gui-Yue merged 1 commit into
shareAI-lab:mainfrom
hardness1020:fix/memory-subsystems-arrow-direction

Conversation

@hardness1020
Copy link
Copy Markdown
Contributor

@hardness1020 hardness1020 commented Jun 2, 2026

What

In the s09_memory subsystems diagram, the read/write arrow connecting Storage to the Memory Files box curved at its end and rendered horizontally on top of the dashed border. Visually it read as lying along the dashed line rather than pointing to it.

This changes the path from a curve (M 112 138 L 112 174 Q 112 180 118 180) to a straight vertical segment (M 112 138 L 112 180) that terminates at the box edge, so the arrowhead points cleanly at the box. This now matches the adjacent write and overwrite arrows.

Files

Applied to all three variants of the diagram:

  • s09_memory/images/memory-subsystems.svg (default / zh)
  • s09_memory/images/memory-subsystems.en.svg
  • s09_memory/images/memory-subsystems.ja.svg

Before / After

  • Before: arrow curves and runs along the top of the dashed box border.
  • After: arrow points straight down at the box edge, consistent with the other two arrows.

@vercel
Copy link
Copy Markdown

vercel Bot commented Jun 2, 2026

@hardness1020 is attempting to deploy a commit to the crazyboym's projects Team on Vercel.

A member of the Team first needs to authorize it.

Make the Storage to Memory Files arrow a straight vertical segment
ending at the box edge, instead of curving onto the dashed border.
Applies to the en, ja, and zh (default) diagram variants.
@hardness1020 hardness1020 force-pushed the fix/memory-subsystems-arrow-direction branch from 1e1d477 to aef9e67 Compare June 2, 2026 00:49
@Gui-Yue
Copy link
Copy Markdown
Collaborator

Gui-Yue commented Jun 2, 2026

@hardness1020
make sense, i like it.

@Gui-Yue Gui-Yue merged commit 91682fa into shareAI-lab:main Jun 2, 2026
3 of 5 checks passed
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