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.

Link to module

https://www.drupal.org/project/search_api_attachments

Proposed Resolution (in attachments)

Proposal 1: a more complex drop showing the advanced character

logo proposal
Proposal 2: a flat logo visualizing complex search

CommentFileSizeAuthor
#22 search_api_attachments_preview_js.png47.96 KBlostcarpark
#22 search_api_attachments_js.png7.54 KBlostcarpark
#18 search_api_attachments_preview_uv2_dark.png46.07 KBlostcarpark
#17 search-api-attachment-logo-retouch.png6.12 KBurvashi_vora
#16 search_api_attachments_comparison.png170.33 KBlostcarpark
#15 search-api-attachment-logo-retouch.png8.56 KBurvashi_vora
#12 search-api-attachment-logo.png9.13 KBurvashi_vora
#10 search_api_attachments_preview_uv.png47.54 KBlostcarpark
#8 search-api-attachment-logo.png8.78 KBurvashi_vora
search_api_attachments_logo_proposal2-min.png18.81 KBstefvanlooveren
search_api_attachments_logo_proposal1-min.png23.95 KBstefvanlooveren

Comments

stefvanlooveren created an issue. See original summary.

stefvanlooveren’s picture

stefvanlooveren commented
Issue summary: View changes
stefvanlooveren’s picture

stefvanlooveren commented
Parent issue: » #3342533: [Meta] Create a logo for a contrib module
leslieg’s picture

leslieg commented
Status: Active » Needs review
Issue tags: +Novice
leslieg’s picture

leslieg commented

Tagged as Novice
We, the mentoring team, are triaging issues for first time contributors at DrupalCon Lille and I think this is a good issue for the contribution day.
We are reserving this issue so please don't work on this issue if you are not at DrupalCon Lille. You can continue the work when the event is over.

amourow’s picture

amourow
he/him/his
commented

I'm reviewing the logo on the contrib day of Lille 2023

amourow’s picture

amourow
he/him/his
commented
Status: Needs review » Needs work

The proposal 2 looks like a better candidate to move forward.
We need to scale up the image to 512px and reduce the file size further. Change the gradient shadow should reduce physical size of the image.

urvashi_vora’s picture

urvashi_vora commented
Status: Needs work » Needs review
StatusFileSize
new search-api-attachment-logo.png8.78 KB

I tried re-designing it.
Please review. Thanks

lostcarpark’s picture

lostcarpark commented
Issue summary: View changes

Added link to module

lostcarpark’s picture

lostcarpark commented
StatusFileSize
new search_api_attachments_preview_uv.png47.54 KB

Nice work, urvashi_vora.

I have prepared a preview:

Urvashi_vora logo

I think this captures the essence of the original proposal, but meets the image and file size requirements.

Just one thing. The image is currently on a white background. I think the background for this logo would be better transparent.

lostcarpark’s picture

lostcarpark commented
Status: Needs review » Needs work

Moving back to needs work. I think if the image from #8 was redone with a transparent background, it would be perfect.

urvashi_vora’s picture

urvashi_vora commented
Status: Needs work » Needs review
StatusFileSize
new search-api-attachment-logo.png9.13 KB

Hi @lostcarpark,

Please review.

Thanks

lostcarpark’s picture

lostcarpark commented

Thanks @urvashi_vora,

I think all that's needed is to make the white background from #8 with transparency. The logo in #12 adds a thick white band, which makes the actual logo quite a bit smaller.

Could you try again?

urvashi_vora’s picture

urvashi_vora commented

That white band was actually present in #8. Let me see what I can do to remove it.

urvashi_vora’s picture

urvashi_vora commented
StatusFileSize
new search-api-attachment-logo-retouch.png8.56 KB

Hi @lostcarpark,

That's all I was able to do to remove the white band.

Please review.

Thanks

lostcarpark’s picture

lostcarpark commented
StatusFileSize
new search_api_attachments_comparison.png170.33 KB

The logo in #15 is an improvement, but it's still a little shrunk from #8.

I've put together a side by side comparison.

The first 3 panels show #8, #12 and #15 side by side.

Panel 4 shows #8 and #12 superimposed (with opacity turned down so you can see both).

Panel 5 shows #8 and #15 superimposed. You can see the outline of #8 around the lower half of #15.

Logo comparison.

#15 is still slightly smaller than #8 was. The border around the edges seems bigger than it needs to be.

If it could be made just a little bigger, it would be perfect.

Thanks again!

urvashi_vora’s picture

urvashi_vora commented
StatusFileSize
new search-api-attachment-logo-retouch.png6.12 KB

Hi @lostcarpark,

Please review. I have tried to remove the background from the #8 logo. And also reduced the size of image.

Thanks

lostcarpark’s picture

lostcarpark commented
StatusFileSize
new search_api_attachments_preview_uv2_dark.png46.07 KB

#17 looks great to me.

I could do a new preview, but it would look exactly like #10.

So here's a preview showing how it currently looks in Gin theme, dark mode. There are some problems, but they aren't related to how it shows the transparency of the logo on the dark background.

Urvashi_vora logo on dark background

I think it looks great on both dark and light background (the end of the magnifier handle does get a bit lost, but I don't think that detracts from the logo). I've confirmed it is 512x512 pixels, and the file size is under 10K.

I'm happy to recommend for RTBC.

urvashi_vora’s picture

urvashi_vora commented

Thanks @lostcarpark.

bramdriesen’s picture

bramdriesen
he/him
Primary language Dutch
Location Belgium 🇧🇪🇪🇺
commented
Status: Needs review » Needs work

I think the logo in 2 looks a lot more modern as proposal 1.

However, due to the many edits on the PNG file over and over instead of source file (psd or whatever), the image has lost quite a lot of quality (you can clearly see it you open the first one next to the last one and zoom in a bit). Also, the glass in the magnifier glass itself is off centre, which my brain doesn't like 😅 it's quite noticeable in the preview.

lostcarpark’s picture

lostcarpark commented

Good point @BramDriesen.

@urvashi_vora, what did you use to draw the image. Is the original in a vector format?

lostcarpark’s picture

lostcarpark commented
StatusFileSize
new search_api_attachments_js.png7.54 KB
new search_api_attachments_preview_js.png47.96 KB

I've redrawn @urvashi_vora's version as a vector, exported as PNG, and compressed to under 10K.

Redrawn logo

Here's a preview of this version.

Preview of redrawn logo

@BramDriesen, I think the off-centre part is meant to be a reflection in the magnifier glass. I've tweaked it to be a little more centred. I could adjust further if it still bothers you. I also added a bit of a highlight to the handle to give it a little more visibility on dark backgrounds.

Let me know what you think of this one.

urvashi_vora’s picture

urvashi_vora commented
Status: Needs work » Needs review

@lostcarpark, I used canva to redraw the proposed logo, as we were not having the original logo, so it was not possible to remove the shadow. Thus, I recreated it.

However, the logo you provided, looks good to me.

As of now, setting it to Needs Review.

bramdriesen’s picture

bramdriesen
he/him
Primary language Dutch
Location Belgium 🇧🇪🇪🇺
commented

Looks better and a lot less noticeable! Thanks 🙏 RTBC+1

bramdriesen’s picture

bramdriesen
he/him
Primary language Dutch
Location Belgium 🇧🇪🇪🇺
commented
Issue summary: View changes
Status: Needs review » Reviewed & tested by the community
lostcarpark’s picture

lostcarpark commented

Thanks everyone!

amourow’s picture

amourow
he/him/his
commented

Great work! @lotscarpark

chrisfromredfin’s picture

chrisfromredfin
he/him/his
Primary language English
Location Portland, Maine
commented
Status: Reviewed & tested by the community » Fixed
Related issues: +#3627290: Update logo for compatibility with Project Browser and Drupal.org

filed upstream issue with suggestions
https://www.drupal.org/project/search_api_attachments/issues/3627290

Now that this issue is closed, review the contribution record.

As a contributor, attribute any organization that helped you, or if you volunteered your own time.

Maintainers, credit people who helped resolve this issue.