eclipse-embed-cdt / eclipse-plugins

The Eclipse Embedded CDT plug-ins for Arm & RISC-V C/C++ developers (formerly known as the GNU MCU Eclipse plug-ins). Includes the archive of previous plug-ins versions, as Releases.
http://eclipse-embed-cdt.github.io/
Eclipse Public License 2.0
555 stars 130 forks source link

Documents image trademark #442

Closed jonahgraham closed 3 years ago

jonahgraham commented 3 years ago

The there is this image pdf icon replicated a few times in embed-cdt plugins. Do we have permission to use Acrobat logo here?

The icon appears in the Documents view (other places too?):

image

ilg-ul commented 3 years ago

Do you have any suggestion how to address this?

jonahgraham commented 3 years ago

Quick solution is to just create a simple PDF icon, or if there is suitably licensed one out there we can create a CQ to reuse it.

If you want I can handle this, there may be an icon in Eclipse icon repo already that is suitable.

ilg-ul commented 3 years ago

Sure, if you want, please handle it.

jonahgraham commented 3 years ago

Step 1 - here are the icons - converted from SVG to PNG with eclipse image renderer SVG created with Inkscape based on a standard blank document (I used the document from ant_buildfile.svg)

zip of SVG + generated png

The Hi-DPI image looks ok, the low res one is less than perfect, but ok for a first try.

Step 2 - I will do a PR with these images replacing the existing ones later.

ilg-ul commented 3 years ago

Ok, thank you.

jonahgraham commented 3 years ago

Step 2 - I will do a PR with these images replacing the existing ones later.

Done. See PR #447

ilg-ul commented 3 years ago

Thank you, I merged it.

Regarding HI-DPI, I never tried the plug-ins on such display. Do they display properly?

jonahgraham commented 3 years ago

My current setup is not hidpi, I have access to one if I need to. I just followed the standard practice of the @2x which automagically does the correct thing in most cases. If a @2x image isn't found, then Eclipse will scale up a normal image.

Your Mac isn't hidpi by default? I thought most macs were.

ilg-ul commented 3 years ago

My Mac is, but the monitor is only 1920x1200.

jonahgraham commented 3 years ago

There is some stuff here - I think you can force scaling, but I haven't played with it much.

This issue is done now. Thanks for merging the change.