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.
Module Link
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.
| Comment | File | Size | Author |
|---|---|---|---|
| #18 | CKEditorWIRIS_2.png | 4.72 KB | antonina pavlenko |
| #18 | CKEditorWIRIS_1.png | 4.75 KB | antonina pavlenko |
| #16 | CKEditorWIRIS.png | 6.84 KB | antonina pavlenko |
| #14 | wiris-logo-2.png | 5.52 KB | urvashi_vora |
| #14 | wiris-logo-1.png | 6.07 KB | urvashi_vora |
Comments
Comment #2
roshni27 commentedplease review
Comment #3
bramdriesenI 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 🤣
Comment #4
roshni27 commented@BramDriesen Thanks. please review.
Comment #5
christophweber commentedI 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.
Comment #6
urvashi_vora commentedAll the logos, I feel are too blur. Can we have better quality?
Comment #7
urvashi_vora commentedI would like to suggest a few logos.
Please review.
Thanks
Comment #8
lostcarpark commentedI 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?
Comment #9
lostcarpark commentedComment #10
urvashi_vora commentedI can give it a try @lostcarpark
Comment #11
urvashi_vora commentedHi @lostcarpark,
Please review.
Comment #12
lostcarpark commentedThat 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.
Comment #13
lostcarpark commentedComment #14
urvashi_vora commentedHi @lostcarpark,
Please review.
Thanks
Comment #15
bramdriesenI'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...
Comment #16
antonina pavlenko commentedHi,
I gave it a try, too, please review it.
Thanks
Comment #17
lostcarpark commentedThere'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.
Comment #18
antonina pavlenko commentedI 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
Comment #19
bramdriesenComment #20
bramdriesenNot 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.
Comment #21
lostcarpark commentedSorry, I missed that it had already been committed.
Comment #23
leslieg commented