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

Reexport Core and Security diagram PNGs for smaller file size #942

Merged
merged 1 commit into from
Dec 12, 2024
Merged

Reexport Core and Security diagram PNGs for smaller file size #942

merged 1 commit into from
Dec 12, 2024

Conversation

bact
Copy link
Collaborator

@bact bact commented Dec 11, 2024 •
edited
Loading

  • No content or image dimension changes. Just removed draw.io editable vector metadata from the PNGs
  • Core size: 532 KB -> 245 KB
  • Security size: 434 KB -> 157 KB
  • Will help faster website loading

@bact
Reexport Core and Security diagram PNGs
724c4ba 
- Old Core size: 532 KB; New Core size: 245 KB
- Old Security size: 434 KB; New Security size: 157 KB

Signed-off-by: Arthit Suriyawongkul <arthit@gmail.com>
@bact bact added Profile:Core Core Profile and related matters Profile:Security Security Profile and related matters labels Dec 11, 2024
@bact bact mentioned this pull request Dec 11, 2024
Merged
kestewart
kestewart approved these changes Dec 12, 2024
View reviewed changes
Copy link
Contributor

@kestewart kestewart left a comment

Choose a reason for hiding this comment

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

Thanks for improving the rendering time!

@goneall goneall merged commit a745f63 into spdx:main Dec 12, 2024
1 check passed
@bact bact deleted the reexport-core-diagram branch December 12, 2024 23:26
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@kestewart kestewart kestewart approved these changes

@goneall goneall goneall approved these changes

@zvr zvr Awaiting requested review from zvr

@sbarnum sbarnum Awaiting requested review from sbarnum

Assignees
No one assigned
Labels
Profile:Core Core Profile and related matters Profile:Security Security Profile and related matters
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@bact @goneall @kestewart