docs: Replace agent-and-renderer.png by a Mermaid diagram.#1464
Conversation
|
@polina-c is there any reason why the Glossary page is not linked from the left-hand side menu of the page? I think it'd be a good fit under the "Concepts" section? (I can modify the |
![gemini-code-assist[bot]](https://avatars.githubusercontent.com/in/956858?s=60&v=4){kind=small}
There was a problem hiding this comment.
Code Review
This pull request replaces a static image in the glossary documentation with a Mermaid sequence diagram to illustrate the A2UI protocol flow. The reviewer suggested refining the diagram's loop name and adding an explicit note for data updates to better align with the textual description.
No reasons. Feel free to link it in this pr. Thank you for improvements! |
|
I'll rebase this and add the link. Thanks for the review @polina-c! |
ditman
force-pushed
the
remove-agent-and-renderer-png
branch
from
c37be1d to
3474293
Compare
ditman
force-pushed
the
remove-agent-and-renderer-png
branch
from
937504e to
efe869f
Compare
…ect#1464) * docs: Replace agent-and-renderer.png by a Mermaid diagram. * docs: Add Glossary menu entry under Concepts
2 participants
Description
There's a diagram in the glossary that doesn't look that great:
I replaced it by the following Mermaid diagram:
(Mermaid is already used in the "Renderer Stack" section of the glossary)
Pre-launch Checklist
If you need help, consider asking for advice on the discussion board.