Problem/Motivation

In order for the module to display properly and following the recommendations we need to add the logo to the repository itself. We already have a logo so it's just a matter of resizing and compressing it.

See meta issue for more information: #3342533: [Meta] Create a logo for a contrib module

In a nutshell:

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

  1. Download the current logo from the project page.
  2. Resize the image to 512x512 pixels.
  3. Check the image file size is under 10K. If not use a utility such as pngquant to compress it.
  4. Upload and share in a comment for review.
  5. Click on "Create fork" in this issue to create an issue fork.
  6. Follow the issue branch link to view in GitLab, then click the "+" icon next to the root folder, and select "upload file". Upload the logo file (make sure it is named logo.png).
  7. Click "Open merge request" to create a merge request to the default branch.

Remaining tasks

  • Merge Request
CommentFileSizeAuthor
#7 logo1.png9.7 KBroshni27
#4 Logo.png9.53 KBroshni27

Issue fork keycloak-3398014

Command icon Show commands

Start within a Git clone of the project using the version control instructions.

Or, if you do not have SSH keys set up on git.drupalcode.org:

Comments

BramDriesen created an issue. See original summary.

bramdriesen’s picture

roshni27 made their first commit to this issue’s fork.

roshni27’s picture

Status: Active » Needs review
StatusFileSize
new9.53 KB

Please review.

bramdriesen’s picture

Status: Needs review » Needs work

Resizing looks good, however I think we need to add a white (or other colour) background to the image. As the black "OpenID" text will be lost when using dark mode.

roshni27’s picture

Ok. I will update

roshni27’s picture

Status: Needs work » Needs review
StatusFileSize
new9.7 KB

please review

bramdriesen’s picture

Status: Needs review » Reviewed & tested by the community

Thanks!

  • BramDriesen committed b1d47dda on 2.2.x
    Issue #3398014 by roshni27, BramDriesen: Project Browser: Add logo for "...
bramdriesen’s picture

Status: Reviewed & tested by the community » Fixed

Status: Fixed » Closed (fixed)

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