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

CommentFileSizeAuthor
#15 embed-mockup.png78.03 KBlostcarpark
#14 Embed-Project-Logo-With-Border.png2.21 KBurvashi_vora
#12 Screenshot 2023-05-19 at 2.15.21 PM.png98.15 KBchrisfromredfin
#9 Embed-Project-Logo.png9.22 KBurvashi_vora
#4 embedproject.png6.97 KBAkanshaSingh
#3 embedproject.png6.97 KBAkanshaSingh
#2 embedproject.png6.97 KBAkanshaSingh

Comments

leslieg created an issue. See original summary.

AkanshaSingh’s picture

AkanshaSingh commented
Assigned: Unassigned » AkanshaSingh
StatusFileSize
new embedproject.png6.97 KB

An embed link is represented by this icon. This graphical element promotes more user engagement and click-throughs on the platform.

AkanshaSingh’s picture

AkanshaSingh commented
Status: Needs work » Needs review
StatusFileSize
new embedproject.png6.97 KB

An embed link is represented by this icon. This graphical element promotes more user engagement and click-throughs on the platform.

AkanshaSingh’s picture

AkanshaSingh commented
Assigned: AkanshaSingh » Unassigned
StatusFileSize
new embedproject.png6.97 KB
AkanshaSingh’s picture

AkanshaSingh commented
amourow’s picture

amourow
he/him/his
commented
Issue summary: View changes

Adding to the summary.

@AkanshaSingh please remove the rounded corner and we will leave it for CSS.
https://www.drupal.org/docs/contributed-modules/project-browser/module-m...

lostcarpark’s picture

lostcarpark commented

Logo looks great, but as @amourow says, rounded corners should be removed.

amourow’s picture

amourow
he/him/his
commented
Status: Needs review » Needs work
urvashi_vora’s picture

urvashi_vora commented
Status: Needs work » Needs review
StatusFileSize
new Embed-Project-Logo.png9.22 KB

Hi,

According to @lostcarpark's suggestion, I have removed the rounded corners.

Please review.

Thanks

lostcarpark’s picture

lostcarpark commented
Issue summary: View changes
lostcarpark’s picture

lostcarpark commented

Updated logo in issue to #9.

Verified image is a .png file, image dimensions are 512x512 pixels, and file size is under 10K.

+1 from me for #9 to go RTBC.

chrisfromredfin’s picture

chrisfromredfin
he/him/his
Primary language English
Location Portland, Maine
commented
StatusFileSize
new Screenshot 2023-05-19 at 2.15.21 PM.png98.15 KB

I worry about the white band in context. Do you think we should put a border around it or make the dividing line a different color?

chrisfromredfin’s picture

chrisfromredfin
he/him/his
Primary language English
Location Portland, Maine
commented
Issue summary: View changes
Parent issue: #3282852: [Meta] Embed Project updates for Project Browser » #3342533: [Meta] Create a logo for a contrib module
urvashi_vora’s picture

urvashi_vora commented
StatusFileSize
new Embed-Project-Logo-With-Border.png2.21 KB

Hi @chrisfromredfin, according to #12, I have added a slight border around the white band in context.

Now you have an option to compare with logo provided in #9. Also I reduced the size a bit more.

Please have a look.

Thanks

lostcarpark’s picture

lostcarpark commented
StatusFileSize
new embed-mockup.png78.03 KB

Personally, I like the version in #9. I like the way it "breaks the fourth wall" by having the background run through it.

I don't think the very thin border added in #14 will be visible at all when it's reduced to the icon size. If a change is needed, it should be a very clear change, not a token change.

I've mocked 3 possible solutions on the image below:

Embed Mockup

On the left, I've added thin (but visible) borders.

In the middle, I've changed the background of the break (to match the colours of the <>).

On the right I've added a thicker border.

If we were going to add a border, I think the thick one looks better, though it does make the break look a bit like an input box.

I think I prefer the background colour to the line, however.

I'd still vote for the original #9 if it was an election, but thought it worth seeing how options look visually.

lostcarpark’s picture

lostcarpark commented

By the way, well done on getting the file size down to 2.21KB while keeping it looking crisp!

butterwise’s picture

butterwise commented

FWIW, my preference is the third option (far right) in #15. Consider making the brackets (<>) white for simplicity's sake?

catsbap’s picture

catsbap commented

Working on reviewing the logo

catsbap’s picture

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

I have reviewed this and it meets the requirements.

lostcarpark’s picture

lostcarpark commented
Status: Reviewed & tested by the community » Needs work

Sorry, @catsbap, could you clarify what you reviewed.

There are several suggestions for discussion, which probably needs @urvashi_vora to post an adjusted version. I don't think this is ready for RTBC yet.

Thoughts?

anicoto’s picture

anicoto
Primary language Spanish
Location Houston, TX
commented
Status: Needs work » Needs review

Reviewing this issue on mentor contribution room #DrupalconPittsburgh 2023

We like the proposal on https://www.drupal.org/project/project_browser/issues/3283041#comment-15...

The simplest the better!

Borders will add unnecessary complexity.

anicoto’s picture

anicoto
Primary language Spanish
Location Houston, TX
commented
anicoto’s picture

anicoto
Primary language Spanish
Location Houston, TX
commented
Status: Needs review » Reviewed & tested by the community

leslieg credited JorgeOspina.

leslieg’s picture

leslieg commented
leslieg’s picture

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

Gave credit and marked this issue as fixed.

Status: Fixed » Closed (fixed)

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