What brought you to change this?

thanks.

Sorry, I tried but couldn't fix that build issue in Enventor side. In my opinion, efl version up is necessary. It failed due to efl 1.20.5 package itself including wrong eo headers.

Thanks for your report, I will bump up soon.

It would be better if you mentioned the problem situation. Plus, it would be better if you described what does this patch improved for.

perfect!

sounds it makes to me.

This is a nice feature improvement!

please check a comment.

please check a comment.

this is not a good solution for opensource world.
It sounds this palette needs to provide a function to align items.
How about deciding the alignment acoording to the colorselector alignment values?
ex)
elm_colorselector_palette_align_set(cs, 0.5, 0);

@jpeg how about accepting and close this?

Were there any issues ?
What brought you to add this patch?
Could you please describe an actual scenario more detail?
Why/Who needs to call this api for what?
Scroller already supports loop mode. Why does it need to call this additionally?

Ping?

What if user wants an other position?
Please take a look at the elm_genlist_item_show(), elm_toolbar_item_show()
They have additional arguments: item position (NONE, IN, TOP, MIDDLE, BOTTOM ..)
I prefer hovesel item reserves the item show positioning like them.

I don't sure how this patch guarantee the update of the map rendering if the map is same...

please check a comment.

Please check a comment and give your opinion.

@raster @cedric even cairo backend hasn't been proved. What does mean of sticking Cairo now? As subranthu said, even Cairo backend has crahses in using APIs. I believe no one use/test cairo backend but native is much used and tested.

In D4310#72195, @minkyu wrote:

@Hermet
I saw 39 line in elc_hoversel.h.
That area is for elm_object common APIs.
But APIs that I added are elm_widget APIs.
Is it correct to intend you?

please check a comment.

and don't miss @since there.

Could you please add the new supported APi name in elc_hoversel.h?
please see 39 line in elc_hoversel.h

the code is duplicated so it's unnecessary.

I don't think this is correct.
find_proc() doesn't suppose the item is selected always.
but replace_proc() does.
So this call should be in replace_proc() instead of it.

Considered again...
But this change is meaningless..

I agree on this korean translation.