- Issue created by @chrisfromredfin
- 🇺🇸United States mrcgomez
I am working on this at DrupalCon Pittsburgh.
- 🇺🇸United States mrcgomez
Here are my proposed images for the logo.
I have put two versions, one transparent and one with a white background. - Status changed to Needs review
over 1 year ago 7:50pm 7 June 2023 - 🇮🇪Ireland lostcarpark
Great 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.
- 🇺🇸United States johnpicozzi Providence, RI
Hello - 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.
- 🇮🇪Ireland lostcarpark
@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?
- 🇺🇸United States johnpicozzi Providence, RI
@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.
- 🇮🇪Ireland lostcarpark
@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.
- 🇮🇪Ireland lostcarpark
I have created a preview of the logo in project browser (Gin theme).
- 🇺🇸United States johnpicozzi Providence, RI
Sorry 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.
- 🇮🇪Ireland lostcarpark
@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:
- Status changed to RTBC
over 1 year ago 1:36am 27 June 2023 - 🇺🇸United States johnpicozzi Providence, RI
YES! That looks good! Moving this to RTBC.
@chrisfromredfin do we need to move this to the project issue queue to be added?
- 🇺🇸United States chrisfromredfin Portland, Maine
Yes, proposed is my favorite now. @johnpicozzi - Leslie takes it from here to put in maintainer's queue, she acts from RTBC issues. Woo!
- 🇺🇸United States mrcgomez
@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. - Status changed to Needs work
about 2 months ago 9:20pm 25 September 2024 - 🇺🇸United States leslieg
@lostcarpark. The file size is 32k. Any chance you can get that closer to 10k?
- 🇮🇪Ireland lostcarpark
Dan's corrected image in #19 meets the file size requirement.
Automatically closed - issue fixed for 2 weeks with no activity.