Filtering palette causes images to zoom

[akravets]

We are using some classes from Triquetrum palette in our Graphiti editor. An issue was discovered that when you filter the tree using the filter and clear out the filter box manually (not with the erase icon on the right, but character by character), the icons for the groups become very big. This does not happen with all strings, but seems to always happen when first letter is an 'f'. Also, this bug doesn't not occur in Linux, only in Windows. Workaround is to restart the editor. I started looking at the palette code, but decide to post this in case this is a known bug or perhaps someone can point me to the right place where to look for a cause. Thanks.

Here is screen recording of the behavior

[erwindl0]

I am not able to reproduce this problem on my Windows 10 PC.

For some reason your screen recording is blocked/unavailable for me (in Belgium).
Can you provide it in another way?

thanks
erwin

[cxbrooks]

I'm not able to view the video from California.

[akravets]

Apologies, please try it now - video

[erwindl0]

Ok, I can see the video now. Weird stuff happening in there that I don’t get in my vanilla Triquetrum… Is it possible to share some more info on your palette implementation : * Which Triquetrum version are you using? * Do you use the Triquetrum extension points to declare palette entries and/or an implementation of org.eclipse.triquetrum.workflow.editor.palette.spi.PaletteEntryProvider ? * Or did you hook into org.eclipse.triquetrum.workflow.editor.palette.ui.PaletteTreeViewer in a different way? * Any customizations of palette-related code in Triquetrum? * What kind of icons do you use? It would help if you would be able to share related parts of your code? Regards erwin From: akravets [mailto:notifications@github.com] Sent: Monday, October 30, 2017 12:33 PM To: eclipse/triquetrum <triquetrum@noreply.github.com> Cc: erwin de ley <erwindl0@gmail.com>; Comment <comment@noreply.github.com> Subject: Re: [eclipse/triquetrum] Filtering palette causes images to zoom (#297) Apologies, please try it now - video <https://www.youtube.com/watch?v=r-IKT-SB9UA&feature=youtu.be> — You are receiving this because you commented. Reply to this email directly, <#297 (comment)> view it on GitHub, or <https://github.com/notifications/unsubscribe-auth/AGKZ4S1q1UToagt5Ss-JyNwL0aLWknXdks5sxbPOgaJpZM4QJQL-> mute the thread.

[akravets]

@erwindl0 I will get you this info shortly.

[akravets]

We did not use plugin as a whole nor were any extension points used, only key classes that contribute to palette functionality were adapted. We used DefaultPaletteBehavior to hook up your PaletteViewerProvider. We did this way because in our PaletteBehavior we read palette metadata from XML.

Below is one of the images we are using, which might be culprit in this bug IMO.

Image