Skip to content

Add logo#2853

Merged
jaraco merged 6 commits into
pypa:mainfrom
abravalheri:add-logo
Nov 4, 2021
Merged

Add logo#2853
jaraco merged 6 commits into
pypa:mainfrom
abravalheri:add-logo

Conversation

@abravalheri

@abravalheri abravalheri commented Nov 4, 2021

Copy link
Copy Markdown
Contributor

logo

Summary of changes

  • Added a few SVG and PNG files representing some variants of the new logo design
  • Added customisations to furo (sphinx theme):
    • Logo in the sidebar
    • Accent colours from logo
  • Added a banner in the main documentation page (motivated by 0f491dc)
  • Added a description of the logo elements, usage, and inspiration (please feel free to remove anything you might find excessive)
  • Suggestion on how to add this description to the toctree

The banners were create in the size 640x320 to fit GitHub's preview (but the SVG images can be rescaled as necessary)

Closes #2227

Pull Request Checklist

Comment thread docs/images/README.rst
IN THE SOFTWARE.

THE USAGE OF THIS LOGO AND ARTWORK DOES NOT INDICATE ENDORSEMENT BY THE
SETUPTOOLS PROJECT.

@abravalheri abravalheri Nov 4, 2021

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

The main reason why I added this second copyright notice, was to have the last phrase:

  THE USAGE OF THIS LOGO AND ARTWORK DOES NOT INDICATE ENDORSEMENT BY THE
  SETUPTOOLS PROJECT.

I think it make sense... but we can also remove this and just rely on the license note on the root directory.

(Inspired by https://flask.palletsprojects.com/en/1.1.x/license/#artwork-license)

Comment thread docs/index.rst
roadmap
setuptools

.. toctree::

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

This is just a suggestion on how to organise the sidebar in 2 logical sections: one that targets directly the end user and one that is more "meta" regarding the project...

Comment thread docs/index.rst
Development guide <development/index>
Backward compatibility & deprecated practice <deprecated/index>
Changelog <history>
Artwork <images/README>

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

I don't know exactly where to place it, so I added it here, but no strong preference...

@jaraco

jaraco commented Nov 4, 2021

Copy link
Copy Markdown
Member

Beautiful, inspired. I love it.

@jaraco jaraco merged commit f359b8a into pypa:main Nov 4, 2021
@abravalheri abravalheri deleted the add-logo branch November 4, 2021 22:56
@abravalheri abravalheri mentioned this pull request Nov 5, 2021
2 tasks
Comment thread docs/conf.py

# HTML theme
html_theme = 'furo'
html_logo = "images/logo.png"

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

I'd recommend not to add blobs into the repo. Sphinx and any other modern web tool works with SVG just fine. Besides poisoning the Git bare repo size, it can also look ugly when scaled for folks with 200% default browser zoom (I have this, for example) + SVG is more optimized for the network transfer meaning faster page loads.

@abravalheri abravalheri Nov 9, 2021

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

Thanks for the feedback @webknjaz. In the PR I added a few PNGs as requested in the discussion of the original issue.

I can submit a PR removing the binary files and replacing them with the SVGs in the *.rst if you guys think that is the best action forward. (If you prefer, and Github allow, you can even open an issue and assign it to me).

I have one doubt regarding the favicon. I suppose modern browsers can handle SVG favicons well, but I don't know if that will work with sphinx out of the box. I suspect sphinx-favicon can handle that scenario. If that is ok with the mantainers, I can also investigate sphinx-favicon and remove the ".ico" file.


Apparently SVG favicons are not supported in Saffari (https://caniuse.com/link-icon-svg)

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

I'd also prefer relying on vector graphics where possible. Sounds like the favicon may not work, so use your judgment.

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.

Adopt a logo

3 participants