gef-classic icon indicating copy to clipboard operation
gef-classic copied to clipboard

Cleanup GEF cursors and remove some of the blockiness

Open ptziegler opened this issue 1 year ago • 4 comments

This creates SVG files for the plug/tree cursors and also the shared arrow icons, resulting in much smoother sides.

Following icons are affected:

  • arrow16
  • arrow24
  • plug-cursor
  • plugnot-cursor
  • tree_add-cursor
  • tree_move-cursor

See https://github.com/eclipse/gef-classic/issues/465

ptziegler avatar Jul 23 '24 13:07 ptziegler

The cursors are a weird one... They are clearly based on the Windows cursors, given that the shaft is slightly tilted, making it point to a different position than the tip of the arrow.

image

And for the plug, I have no idea how to make it look prettier. I'm also not sure if we should even keep it like this. In modern UI, icons tend to be more simplistic, which causes this 3dish icon to look dated. Perhaps it'd be better to replace it with a flat image of a plug?

ptziegler avatar Jul 23 '24 13:07 ptziegler

To add my five cents here: I never liked the connection cursors, and in 4diac IDE we are not using them for a very long time. We resorted to take the most suitable system cursor instead (e.g., crosshair for connection creation). This gives a much more native look and feel. Because any cursor we design, even if done very nice and perfect, will be wrong on any at least 2 of the 3 supported OSes.

azoitl avatar Jul 23 '24 16:07 azoitl

I'd have to double check, but I'm fairly certain we do the same for our product. The cursors just don't look very good. Which is also why I'm hesitant to go above and beyond here.

I tried looking into the possibility of decorating the native cursors so that one could piece together the image data, but I sadly don't believe that's possible.

ptziegler avatar Jul 23 '24 16:07 ptziegler

I think for now it is much better then what we had before. So I would go with this change.

azoitl avatar Jul 25 '24 10:07 azoitl

I finally got around to looking at those cursors again. I think the original plug is just too far gone, so I redesigned it from scratch. I've also adapted the cursor size, so that the "tree" and "plug" cursors are now the same.

ptziegler avatar May 02 '25 04:05 ptziegler

Eclipse SWT will get SVG support with the next release we could then think of directly using the SVGs later.

I'm already working on a PR that I want to push later today for exactly that ;D We just have to be careful to remain backwards compatible.

ptziegler avatar May 02 '25 08:05 ptziegler

Eclipse SWT will get SVG support with the next release we could then think of directly using the SVGs later.

I'm already working on a PR that I want to push later today for exactly that ;D We just have to be careful to remain backwards compatible.

Good point. Got carried away by the exciting new possibilities.

azoitl avatar May 02 '25 08:05 azoitl