You mean this?

The problem with your icons is that none of these is in hicolor. Which means, when we are dealing with the fallback to what we have in the default theme, we are on the fly checking for hicolor. However, it might not be 100% the case, as i do not know if we are going into fallback handling or not, as i do not know your config. Want to share it ? :)

I'm afraid this is as far as my knowledge of Linux icons and Efreet reach. Unless @bu5hm4n or @zmike want to pitch in, this is not a terribly urgent matter.

True, my EFL was out of date!

The fact that the indicator widget (the little dots at the top) is not centered tells me that your EFL is not up to date.
And this reminds me that to fix the missing icons D11381 needs to land first, doh!

Oh, okay, found the neovim issue: the file is actually called "nvim.desktop", not "neovim.desktop", that's why. My eyes just didn't catch it at first.

About the purpose: understood, gonna update the main post.

The purpose of this sample is to showcase the Efl.Ui.Spotlight widget, therefore, I am not interested in making it waterproof.
However, during its development it uncovered some icon management issues in EFL, and that is good.

Oh, sorry, I had not realized these patches existed and I already pushed some of the fixes. Anyway, landed the rest now. Thanks!

@segfaultxavi it's done in D9915 and D9916. I think all examples that it don't use UI is updated.

Thanks! I'll update the examples that do not need UI to work.

The stringshare check has been fixed with the last string changes but the lock still remains.

The problem is with the marshalling of array<stringshare>, which is the value expected by CommandLine.SetCommandArray. It is being generated as Eina.Array<System.String>. While we don't expect C# users to deal with stringshares directly, they should be marshalled correctly instead of just being passed to C.

A guess into the blue, is it possible that in efl_csharp_application.cs:102 the command line args are not stringshares, but ecore expects stringshares ? (This was changed at some point, and c# was likely not adjusted)

That code was merged into the Hello World GUI tutorial.

Aaaaaaaaand you are right @lauromoura, I did not port your fix correctly.
I will update my diff and close this ticket as invalid.

I included your fix (passing win to the constructor of the timer object) to my diff already :)

Could you check my branch devs/lauromoura/fix-csharp-life on the examples repo?

Will do!

Please tag future changes to tools/examples.git with the examples project