Skip to content

Conversation

@gtca
Copy link
Contributor

@gtca gtca commented Nov 13, 2025

This is the first draft for the media kit page.

Please share any major feedback you might have, and let's merge it and iterate on it!

@gtca gtca requested review from Zethson and Copilot November 13, 2025 00:32
@netlify
Copy link

netlify bot commented Nov 13, 2025

Deploy Preview for jade-cajeta-1bcca0 ready!

Name Link
🔨 Latest commit f34d660
🔍 Latest deploy log https://app.netlify.com/projects/jade-cajeta-1bcca0/deploys/691a3d4d3eca46000842308f
😎 Deploy Preview https://deploy-preview-186--jade-cajeta-1bcca0.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 project configuration.

Copy link

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull Request Overview

This pull request adds a media kit page to the scverse website by introducing logo assets and updating navigation elements to link to the new page.

Key Changes:

  • Added logo image assets in SVG and PNG formats (scverse logos and library-specific logos)
  • Updated header navigation to include "Media kit" link in the About dropdown
  • Updated footer navigation to include "Media kit" link in the Pages section

Reviewed Changes

Copilot reviewed 5 out of 17 changed files in this pull request and generated no comments.

Show a summary per file
File Description
static/img/logo/scverse_symbol.svg SVG version of scverse symbol logo
static/img/logo/scverse_symbol.png PNG version of scverse symbol logo
static/img/logo/scverse.svg Full scverse logo with text in SVG format
static/img/logo/scverse.png Full scverse logo with text in PNG format
static/img/libs/spatialdata_horizontal.png Library logo asset for spatialdata
static/img/libs/muon.png Library logo asset for muon
layouts/partials/header.html Added "Media kit" link to About dropdown menu
layouts/partials/footer.html Added "Media kit" link to footer Pages section

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

@grst
Copy link
Contributor

grst commented Nov 13, 2025

Thanks for adding this :)

@grst
Copy link
Contributor

grst commented Nov 13, 2025

A more general question is if we should get rid of the gradients from slide templates/examples. I think they have been critizied by multiple people.

@flying-sheep
Copy link
Member

Can we start using the version of the logo that I fixed instead of this?

image

@gtca
Copy link
Contributor Author

gtca commented Nov 16, 2025

Thanks for checking the PR, @grst and @flying-sheep!

Shouldn't both logos include the (R)?

I added the version with ®️ to use in legal contexts. In general though, I don't think we should use it as a default option.

Is PT sans font freely available? Is it even relevant as I think in the logo text is converted to a path

I think it's freely available.

scirpy has a full logo (https://github.com/scverse/scirpy/blob/main/docs/img/scirpy_logo.svg) and the icon (already listed). I believe the same is true for scanpy?

I added it for scirpy for now.

More generally, some work is needed across packages to make sure both versions exist and are available as svg files but I don't want to make that blocking.

Can we start using the version of the logo that I fixed instead of this?

I took a look at it, and it seems svgs for different versions (®, with text, icon only) still need this fix to be incorporated. This is true in general and not just about the media kit. I would put it in the same bucket as the logo improvements for the packages. And thanks again for fixing it!

@gtca gtca merged commit 3347123 into main Nov 16, 2025
6 checks passed
@gtca gtca deleted the mediakit branch November 16, 2025 21:24
@Zethson Zethson changed the title Media kit draft Media kit Nov 16, 2025
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.

4 participants