slsa-framework / slsa

Supply-chain Levels for Software Artifacts
https://slsa.dev
Other
1.48k stars 212 forks source link

impl: refactor diagram in figma (no visual change) #1055

Closed MarkLodato closed 1 month ago

MarkLodato commented 1 month ago

Refactor how the diagrams are implemented in Figma so that it's easier to edit.

There's no visual change. Two SVGs are completely unchanged, and two are modified but only have lines reordered AFAICT. The two modified SVGs seem to be a pixel off, for a reason I cannot comprehend.

Explanation: Previously the diagrams were built "up", starting with the simplest and successively adding more detail. This was a pain to edit, because you'd first have to find which diagram was the "master" for the change you wanted to make. Now there's only one "master" with all of the detail, and the derived diagrams just hide detail that's not wanted. This is much easier to edit (and understand!)

Also delete some unused diagrams that were in the figma file but never exported to SVG.

Signed-off-by: Mark Lodato lodato@google.com

netlify[bot] commented 1 month ago

Deploy Preview for slsa ready!

Name Link
Latest commit 08d089d87e69111a1d31acad2a279efe9a60b6a5
Latest deploy log https://app.netlify.com/sites/slsa/deploys/66424b0e9a44a1000891f79e
Deploy Preview https://deploy-preview-1055--slsa.netlify.app
Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site configuration.