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

Comments

leslieg created an issue. See original summary.

leslieg’s picture

Status: Active » Needs work
urvashi_vora’s picture

Assigned: Unassigned » urvashi_vora
urvashi_vora’s picture

Assigned: urvashi_vora » Unassigned
Status: Needs work » Needs review
StatusFileSize
new4.33 KB
new3.29 KB
new4.09 KB
new7.77 KB
new4.12 KB

Hi,
I would like to suggest a few designs. Please have a look.

Thanks

urvashi_vora’s picture

leslieg’s picture

Issue summary: View changes
leslieg’s picture

Status: Needs review » Reviewed & tested by the community

Entity-Reference-Revision-Logo-5.png is a ng, 512x512 and less than 10k

berdir’s picture

Issue summary: View changes
StatusFileSize
new24.92 KB

This is a hard one to have a logo for and I think it's not that relevant because people will install this just because other modules depend on it, like Paragraphs or Commerce.

I like the style of version 5 best as well. That said, the logo doesn't really make sense to me. the magnifying glass implies search related features while the arrow in both direction implies bidirection. While it actually kind of does that, it's not really the central part of its functionallity. What about removing the magnifying glass, making the arrow directional and then visual revisions/versions a bit so we can point to a specific version? I'm imagining roughly something like the following for versions, but with the element used in the current logo:

Thoughts?

Also, shouldn't we keep a source/editable version of this file in the repository as well, like an SVG? So that we can update it in the future? Also not sure how issue credits work, as @leslieg uploaded this in the project, shouldn't we ensure the person who created it gets credits there as well?

And last comment, it would be very helpful for maintainers if you could already open merge request with the file, then you can easily ensure that it will have the correct filename and so on. You can actually do that through the, for ERR, I first requested a issue fork to be opened, then clicking there brings me to https://git.drupalcode.org/issue/entity_reference_revisions-3362560/-/tr..., where a file can be uploaded and committed with the "+" button and then UPload image.

lostcarpark’s picture

I agree it's a tricky concept to encompass in a logo.

Here's a possible idea...

Form with field connecting to another form, and smaller feinter versions of the form in the background

Hopefully it conveys the idea of a of an entity referencing another, and previous versions stacked behind.

I've attached as a PNG file, shrunk to be under 10kb. I've also included the SVG file so it can be added to the repo. I had to add .txt to the file extension so it could upload.

lostcarpark’s picture

Status: Reviewed & tested by the community » Needs review

I wonder does this issue need further review. There seem to have been several options proposed, but no actual discussion on them. Not sure was there any reasoning in selecting the logo in the Proposed Resolution.

Setting back to Needs Review. Please move back to RTBC if this is settled.

lostcarpark’s picture

I do actually like @urvashi_vora's logo in the Proposed Resolution, but there is a lot of whitespace around the edges. I feel it needs the logo part blown up to fill the space.

kristen pol’s picture

@lostcarpark Would you rather we update the one in the summary or switch to your proposal? I like the summary one but agree it needs less whitespace.

leslieg’s picture

Status: Needs review » Reviewed & tested by the community

I am moving this back to RTBC since the Proposed resolution was already provided to the maintainer. They have a link to this ticket of they want to review other options

leslieg’s picture

Status: Reviewed & tested by the community » Fixed

Gave credit and marked as fixed

berdir’s picture

Status: Fixed » Needs work

I already gave feedback in #9, IMHO the current logo does not represent the purpose of this project.

chrisfromredfin’s picture

Status: Needs work » Fixed

Fixed here as far as PB is concerned, please continue work in #3362560: Update logo for compatibility with Project Browser

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.

Status: Fixed » Closed (fixed)

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