Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Compress Architecture Diagram #339

Open
wants to merge 2 commits into
base: main
Choose a base branch
from
Open

Compress Architecture Diagram #339

wants to merge 2 commits into from

Conversation

SteveLasker
Copy link
Collaborator

From @mcr,
https://mailarchive.ietf.org/arch/msg/scitt/sOGI7xcUaOJx5Zag8uqsygooLrc/

The diagram is very tall and skiny. The SVG is lovely mind.
Maybe use more of the horizontal space, then "Signed-Statement->Transparency
Statement" wouldn't have to weave around.

Which saves some vertical scrolling and a wee bit of compute and network bytes:
image

There's no change in content or flow, just a repositioning of the boxes.

@SteveLasker
Compress Architecture Diagram
4d4851b 
Signed-off-by: steve lasker <[email protected]>
@SteveLasker
balanced spacing
33cab1c 
Signed-off-by: steve lasker <[email protected]>
mcr
mcr approved these changes Dec 31, 2024
View reviewed changes
Copy link

@mcr mcr left a comment

Choose a reason for hiding this comment

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

have you tried "asciio"?

JAG-UK
JAG-UK approved these changes Jan 7, 2025
View reviewed changes
Copy link
Contributor

@JAG-UK JAG-UK left a comment

Choose a reason for hiding this comment

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

LGTM

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@aj-stein aj-stein aj-stein approved these changes

@JAG-UK JAG-UK JAG-UK approved these changes

@henkbirkholz henkbirkholz henkbirkholz approved these changes

@achamayou achamayou achamayou approved these changes

@mcr mcr mcr approved these changes

@ad-l ad-l Awaiting requested review from ad-l ad-l is a code owner

@fournet fournet Awaiting requested review from fournet fournet is a code owner

@yogeshbdeshpande yogeshbdeshpande Awaiting requested review from yogeshbdeshpande yogeshbdeshpande is a code owner

@OR13 OR13 Awaiting requested review from OR13 OR13 is a code owner

@aj-stein-nist aj-stein-nist Awaiting requested review from aj-stein-nist aj-stein-nist is a code owner

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
Draft 11 - SECDIR Review IETF 122
Development

Successfully merging this pull request may close these issues.
6 participants
@SteveLasker @mcr @achamayou @henkbirkholz @JAG-UK @aj-stein