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.

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

Trim embedded doc image assets. #2368

Merged
merged 3 commits into from Feb 16, 2024

Conversation

jsirois
Copy link
Member

@jsirois jsirois commented Feb 15, 2024

Re-use the Pex icon png for the favicon and delete pex.ico. Browser
support for png favicons is both wide and deep at this point and
removing the ~46KB .ico file trims ~20KB from the binary distributions
we release.

Also re-name pex-full* assets to pex-logo* to reflect their role and
tidy up the SVG source, adding its license to the metadata. There is
the SVG source and then the logo and icon derived from it. This should
be a bit more clear now.

Re-use the Pex icon png for the favicon and delete `pex.ico`. Browser
support for png favicons is both wide and deep at this point and
removing the ~46KB `.ico` file trims ~20KB from the binary distributions
we release.

Also re-name pex-full* assets to pex-logo* to reflect their role and
tidy up the SVG source. There is the SVG source and then the logo and
icon derived from it. This should be a bit more clear now.
@jsirois jsirois requested review from benjyw and huonw February 15, 2024 18:19
@jsirois
Copy link
Member Author

jsirois commented Feb 15, 2024

The doc-site for this branch is deployed and viewable here: https://jsirois.github.io/pex/

@@ -106,5 +106,5 @@
simplepdf_vars = {
"primary": "#ffcc00",
"cover": "black",
"cover-bg": "url(pex-cover.png) no-repeat center",
"cover-bg": "url(pex-icon.png) no-repeat center",
Copy link
Member Author

Choose a reason for hiding this comment

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

Sorry about that, https://jsirois.github.io/pex/_static/pex.pdf was missing the cover page artwork. Now re-deployed and looking correct.

Comment on lines +108 to +112
# Confusing! The white gets applied to the page number color (which appears inside a rusty red
# box) and the back cover text color. Neither white nor black work well for the back page text
# color since the text intersects the thick black bottom arc of the Pex P-egg. The yolk color
# works well enough in both these spots though.
"white": "#ffee00",
Copy link
Member Author

Choose a reason for hiding this comment

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

Alright, that's it from me. Sorry for the thrash.

The PDF looks good enough. Site re-deployed with this change and the PDF link is the PDF icon on the bottom bar: https://jsirois.github.io/pex/

Copy link
Collaborator

@benjyw benjyw left a comment

Choose a reason for hiding this comment

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

Is the logo an homage to egg files, .egg-info etc?

@jsirois
Copy link
Member Author

jsirois commented Feb 16, 2024

Is the logo an homage to egg files, .egg-info etc?

Yes, and an upside down egg looks like a P if you squint. Most directly though, eggs perhaps introduced the idea of a zipapp to the wider world with eggsecutables in 2006: pypa/setuptools@ae347c1
Documented today for posterity here: https://setuptools.pypa.io/en/latest/deprecated/functionalities.html

@jsirois jsirois merged commit ca58b71 into pex-tool:main Feb 16, 2024
26 checks passed
@jsirois jsirois deleted the embedded-docs/trim-assets branch February 16, 2024 01:43
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.

None yet

2 participants