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.

Please review the module page, and check for any existing logo before proceeding:

https://www.drupal.org/project/ckeditor_wiris

Proposed resolution

When the issue moves to RTBC, please add the accepted logo here.

Comments

lostcarpark created an issue. See original summary.

roshni27’s picture

Status: Active » Needs review
StatusFileSize
new4.18 KB

please review

bramdriesen’s picture

Status: Needs review » Needs work

I would suggest to stick (or at least closer) to the original logo of WIRIS: https://ckeditor.com/cke4/addon/ckeditorwiris

Also, I really, really don't like comic sans 🤣

roshni27’s picture

Status: Needs work » Needs review
StatusFileSize
new7.61 KB
new8.61 KB
new7.95 KB
new8.44 KB

@BramDriesen Thanks. please review.

christophweber’s picture

I think version 4 or 3 can work, in that order. Technically the icons all conform, and once a final version is decided this issue can be marked as RTBC.
@roshni27 It is your decision.

urvashi_vora’s picture

All the logos, I feel are too blur. Can we have better quality?

urvashi_vora’s picture

StatusFileSize
new9.48 KB
new6.18 KB
new4.14 KB

I would like to suggest a few logos.

Please review.

Thanks

lostcarpark’s picture

I agree the Wiris logo is too blurred in @roshni27's versions. Nice ideas if they could be cleaned up.

I like @urvashi_vora's ones.

However, I wonder should they incorporate the CKEditor logo?

What would you think of the W inside the purple and green hexagon?

lostcarpark’s picture

Status: Needs review » Needs work
urvashi_vora’s picture

I can give it a try @lostcarpark

urvashi_vora’s picture

Status: Needs work » Needs review
StatusFileSize
new8.55 KB
new8.55 KB

Hi @lostcarpark,

Please review.

lostcarpark’s picture

StatusFileSize
new1.62 KB

That looks great, though you've uploaded the same logo twice.

Unfortunately, the CKEditor logo appears somewhat pixilated.

Are you able to import vector images into your graphics software? I've attached the CKEditor logo as a SVG (you'll need to remove the .txt extension). I think I must have downloaded it from CKEditor's website. If you could replace the CKEditor part with the vector version, I think it would look great.

lostcarpark’s picture

Status: Needs review » Needs work
urvashi_vora’s picture

Status: Needs work » Needs review
StatusFileSize
new6.07 KB
new5.52 KB

Hi @lostcarpark,

Please review.

Thanks

bramdriesen’s picture

Status: Needs review » Fixed

I'm triaging issues for the project browser initiative. This can be set to fixed as the logo has been committed, listing this on slack so a maintainer here can double check issue credits.

https://git.drupalcode.org/project/ckeditor_wiris/-/blob/2.x/logo.png?re...

antonina pavlenko’s picture

Status: Fixed » Needs review
StatusFileSize
new6.84 KB

Hi,
I gave it a try, too, please review it.
Thanks

lostcarpark’s picture

There's not a lot of difference between the latest logos from @urvashi_vora and @Antonina Pavlenko.

wiris-logo-2.png from #14 would seem to match the W in the favicon on the Wiris site, and the CKEditor logo, so I'll give my +1 to that one.

I would appreciate another opinion or two before going RTBC.

antonina pavlenko’s picture

StatusFileSize
new4.75 KB
new4.72 KB

I was trying to make the logo sharp and close to the original logos, keeping the same layout.
There's another try from me.
Please review

bramdriesen’s picture

Status: Needs review » Fixed
bramdriesen’s picture

Not sure why my comment in #15 was ignored as this was committed already.

I did however updated it to the one in #16 because it looked sharper and the "W" was a tad bigger making it more visible in the small thumbnail.

lostcarpark’s picture

Sorry, I missed that it had already been committed.

Status: Fixed » Closed (fixed)

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

leslieg’s picture