search

eclipse-cdt-cloud / cdt-cloud

Components for web-based C/C++ tools
https://www.eclipse.org/cdt-cloud
Eclipse Public License 2.0
17 stars 0 forks source link

Add logo files and sources #34

Closed planger closed 3 months ago

planger commented 1 year ago

Contributed on behalf of STMicroelectronics

planger commented 1 year ago

Thank you for your feedback! Right, the BLUEPRINT font has indeed some flaws. @sgraband Is it rather easy for you to incorporate Stefan's feedback above?

See also https://github.com/eclipsesource/cdt-cloud-blueprint/pull/3

sgraband commented 1 year ago

Is there a way to get a better version of the Blueprint logo?

Tasking a professional to do it :wink:

Jokes aside, i can definately align the height of the letters. Not sure why this is messed up as i already once aligned them. I can play around a bit with the alignment of the letters to see if it will look better.

Regarding the I and the R i am not sure how much i can improve this. The I is basically just a line with the width of the other letters. So i am not sure how to change this without breaking the font. Maybe the spacing will help? The problem with the R is that i needed to recreate it from another font to try and match the current font, as the original text did not have an R and the font is not detectable by inkscape. (Maybe this is possible in Illustrator). I can try to improve the diagonal line a bit.

sgraband commented 1 year ago

cdtcloud-blueprint

So i updated the things i just mentioned, could you take a look and give me feedback?

sdirix commented 1 year ago

Definitely better :+1:

planger commented 1 year ago

I agree, thanks! Looks much better. Is there a reason why we wouldn't increase the size of the "BLUEPRINT" so it aligns with "CLOUD" above or align the B of BLUEPRINT with the L of CLOUD?

sgraband commented 1 year ago

Current: Current Aligned with C: CAlign Aligned with L: LAlign Same size centered: SameSizeMiddle Same size right align: SameSizeLeft

I tried out some options here. Let me know what you think. I personally still prefer the current solution, but i would also be fine with Aligned with C and Same size centered (Although i find this relatively bulky). Aligning with L i feel like is too symetric for my taste. WDYT?

planger commented 1 year ago

Yeah, you are right, it doesn't really improve. I think I have a slight preference for the "aligned with L" variant or the current. @sdirix WDYT?

sdirix commented 1 year ago

I would keep the current variant where it's not neatly aligned with the L. Seems a bit too tidy aligned with the L and reads like "LOUD BLUEPRINT"

planger commented 1 year ago

@MatthewKhouzam I see that you have added yourself as a reviewer to this PR. Do you want to add any feedback? Thanks!