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. If you have a logo as the first image in the Images field on your project page, please remove it. The logo will be displayed on the cards in the Project Browser grid and list views.
I have uploaded a couple of images.
Review images.
| Comment | File | Size | Author |
|---|---|---|---|
| #19 | ckeditor_anchor_links3.png | 9.62 KB | dandolorion |
| #13 | ckeditor_anchor_preview3.png | 104.38 KB | lostcarpark |
| #13 | ckeditor_anchor_links3.png | 31.08 KB | lostcarpark |
| #11 | ckeditor_anchor_preview.png | 112.04 KB | lostcarpark |
| #10 | ckeditor_anchor_links2-fs8.png | 6.45 KB | lostcarpark |
Comments
Comment #2
mrcgomez commentedI am working on this at DrupalCon Pittsburgh.
Comment #3
mrcgomez commentedHere are my proposed images for the logo.
I have put two versions, one transparent and one with a white background.
Comment #4
mrcgomez commentedComment #5
lostcarpark commentedGreat looking logo!
The image dimensions are correct. However the file sizes are much too big. I have run them through pngquant to reduce to around 8kb to meet the requirements.
Of the two, personally I prefer the transparent background version. I did a quick check of how it looks on a dark background, and I think it looks good:
So my vote is for the transparent version.
Comment #6
lostcarpark commentedComment #7
johnpicozziHello - just wondering why the logo doesn't include an anchor? I get the reference to CKEditor and Linking, however feels like it's missing the tie in to anchor links specifically.
Comment #8
lostcarpark commented@johnpicozzi, good call.
I've done some tinkering, and created a logo with the purple CKEditor hexagon, combined with a ships anchor...
However, after completing this, I notice the button the module adds uses a flag symbol. Would that be a better choice?
Comment #9
johnpicozzi@lostcarpark Thanks for the quick update. I'm thinking we keep the links and add the anchor to the middle of the circle. What do you think?
As for the flag, it honestly has never made sense to me why they use a flag. I think the anchor is great and will make sense to people.
Comment #10
lostcarpark commented@johnpicozzi, how about something like this?
I had a quick look at the module repository, and it appears that the anchor button graphic comes from the CKEditor library. We probably don't want Drupal to be inconsistent with other uses of CKEditor, so I think we'd need to try to get it changed there. If you fancy opening an issue on CKEditor, I'd be happy to contribute an image.
Comment #11
lostcarpark commentedI have created a preview of the logo in project browser (Gin theme).
Comment #12
johnpicozziSorry for the delayed response here. I'm thinking we don't want to copy CKEditor with the flag Icon. The purple CKEditor ring is good and widely recognized. I think your original with the links being the focus was good. I think we should add the anchor in the middle of that one.
Comment #13
lostcarpark commented@johnpicozzi, the original version was by @mrcgomez, so I don't want to take credit for that.
Here's another version based on their logo, with the anchor in the middle:
And here's a preview of it in project browser:
Comment #14
johnpicozziYES! That looks good! Moving this to RTBC.
@chrisfromredfin do we need to move this to the project issue queue to be added?
Comment #15
lostcarpark commentedUpdated logo in Proposed Resolution.
Comment #16
chrisfromredfinYes, proposed is my favorite now. @johnpicozzi - Leslie takes it from here to put in maintainer's queue, she acts from RTBC issues. Woo!
Comment #17
mrcgomez commented@lostcarpark Great Job! and Great collaboration to all! Leslie helped me get started at DrupalCon Pittsburgh. #myfirstcontribution
Excited to see the final product in project browser.
Comment #18
leslieg commented@lostcarpark. The file size is 32k. Any chance you can get that closer to 10k?
Comment #19
dandolorion commentedUpdated to 9.6kb
Comment #20
dandolorion commentedComment #21
lostcarpark commentedDan's corrected image in #19 meets the file size requirement.
Comment #22
leslieg commentedAssigned credit and marked as fixed.
Comment #23
leslieg commented