Create a logo that is 512x512 square dimension in PNG format without animations. Suggested file size should be 10k or less. Use a lossy image tool (such as pngquant) to reduce file size while keeping the image quality at around 80%.

MAINTAINERS: The logo should be added to the root folder of the repo, and be named logo.png. This must be present on your project’s default branch. This is the logo that will be displayed on the cards in the Project Browser grid and list views.
If you previously added a logo as the first image in the Images field on your project page, please remove it. Project Browser will no longer be using the first image as the logo.

Proposed resolution

https://www.drupal.org/files/issues/2023-09-13/xml_sitemap_2.png

CommentFileSizeAuthor
#6 xml_sitemap_1_64x64.png580 bytesdandolorion
#6 xml_sitemap_2.png3.25 KBdandolorion
#6 xml_sitemap_1.png3.36 KBdandolorion
#3 xml_sitemap.png2.49 KBdandolorion

Comments

chrisfromredfin created an issue. See original summary.

urvashi_vora’s picture

urvashi_vora commented
Status: Needs work » Needs review
StatusFileSize
new xml-sitemap-1.png5.4 KB
new xml-sitemap-2.png6.19 KB

Please review.
Thanks

dandolorion’s picture

dandolorion commented
StatusFileSize
new xml_sitemap.png2.49 KB

Hi there,

Please find my icon attached for your consideration.

Many thanks

Dan

bsnodgrass’s picture

bsnodgrass commented

I like the selection in the proposed resolution.

I confirmed there is currently no logo on the module https://www.drupal.org/project/xmlsitemap

chrisfromredfin’s picture

chrisfromredfin
he/him/his
Primary language English
Location Portland, Maine
commented
Status: Needs review » Needs work

I think this might be able to borrow a little bit more directly from the branding logo for XML Sitemaps altogether: https://www.xml-sitemaps.com/ -- this already does quite a bit, so I'm not super opinionated on it, but maybe we could take more directly from it?

Otherwise, my only opinion is that we might want to make the icon a little bit bigger inside the blue space. There's a lot of blue space surrounding and I think we might want to capitalize on all the available canvas (especially since there's text in the logo) so that it's still readable as it scales down (ex.g. to like 64x64).

dandolorion’s picture

dandolorion commented
StatusFileSize
new xml_sitemap_1.png3.36 KB
new xml_sitemap_2.png3.25 KB
new xml_sitemap_1_64x64.png580 bytes

Hi all,

Please see attached updated version of the icon. I have also attached a 64x64 version to show the text visibility.

thanks

Dandolorion

dandolorion’s picture

dandolorion commented
Status: Needs work » Needs review
bsnodgrass’s picture

bsnodgrass commented
Issue summary: View changes

Thank you! I would opt for #2 with the solid boxes, the smaller example does not show the graphic elements as clear.

I will update the summary with that version. If someone else agrees with me, you can just mark this RTBC

leslieg’s picture

leslieg commented
Status: Needs review » Reviewed & tested by the community

I agree with Bob. +1 for xml_sitemap_2.png

Moving to RTBC. Thanks @Dandolorion

leslieg’s picture

leslieg commented
Issue summary: View changes
poker10’s picture

poker10 commented

This logo was accepted and committed by the maintainer (see #3410948: Project Browser: Add/update logo for XML sitemap), so I think we can mark this as Fixed :)

leslieg’s picture

leslieg commented
Status: Reviewed & tested by the community » Fixed

Maintainer added the suggested Project Browser logo. Added credit and marked this issue as fixed.

Status: Fixed » Closed (fixed)

Automatically closed - issue fixed for 2 weeks with no activity.