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
- Download the current logo from the project page.
- Resize the image to 512x512 pixels.
- Check the image file size is under 10K. If not use a utility such as pngquant to compress it.
- Upload and share in a comment for review.
- Click on "Create fork" in this issue to create an issue fork.
- 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). - Click "Open merge request" to create a merge request to the default branch.
Remaining tasks
- Merge Request
Issue fork keycloak-3398014
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
Comment #2
bramdriesenComment #4
roshni27 commentedPlease review.
Comment #5
bramdriesenResizing 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.
Comment #6
roshni27 commentedOk. I will update
Comment #7
roshni27 commentedplease review
Comment #8
bramdriesenThanks!
Comment #10
bramdriesen