In D11781#224554, @jsuya wrote:

The code before modification refers to the location information of the object to be searched. However, the modified code does not consider the location information. This can make to compatibility issues.

Added this ticket for reference.

Noo... i just found a bug in this widget. :P

This patch does fix this issue. 253680f754f1165ff8b6397d378ecba4af8acc50 -> efl patch.

I shall try fixing this based on @raster's hints.

I have added following patches into master branch.

This is fixed now. we can close it.

rebuilt efl/e and i could reproduce this.

i am rebuilding efl/e to verify this.

Thanks for the fix.

it works now magically. don't know how and why...
so we can close it.

i have upgraded arch again and i still have this issue. i wonder what went wrong.
will try to find root cause and report here.

just type anything on terminal and then ^c. it does not do anything. ideally it should cancel out the command.

refer ad28bdfb5a464895a1a0ce3e909a

We won't fix legacy code warnings. e.g. elm_interface_scrollable.eo

In T5361#104728, @jpeg wrote:

Looks like the answer is to have a single enum for EO API, which is slightly different from the legacy:

• default
• always
• none
• on_focus
• on_drag (new in eo as it's "default" in legacy) Bbut default here should mean "whatever is in elm_config".

We can close this now.

@Martin-Furter Are you using git-master? or the latest released E?

In T5361#98753, @jpeg wrote:

No it's not completed.

type slider_func_type: __undefined_type; [[Elementary slider function type]]
type slider_freefunc_type: __undefined_type; [[Elementary slider free function type]]

And it's used in:

@property indicator_format_function {
   set {
      [[Set the format function pointer for the indicator label
 
        Set the callback function to format the indicator string.
      ]]
   }
   values {
      func: slider_func_type @nullable; [[The indicator format function.]]
      free_func: slider_freefunc_type @nullable; [[The freeing function for the format string.]]
   }
}

Are you sure the indicator APIs are specific to the slider and only the slider? They don't apply to progressbar or anything else???

In T5359#98754, @jpeg wrote:

I think it's in pretty good shape now, indeed. A few comments though:

1. Event names: ` events { press; Called when panes got pressed unpress; Called when panes are no longer pressed } `

In D5198#88113, @jpeg wrote:

In D5198#87991, @singh.amitesh wrote:

We can mark these CIDs as false alarm.

We should use efl_data_scope_get here as Coverity has already guessed that we don't do NULL checks on it. If we stay consistent, Coverity will be more useful.

I have discussed with @jpeg. efl_data_scope_get/safe_get() will always return not-null value if obj is valid.
In case of obj is invalid, the eo will do the check for us before this function gets called and warn the user.
So basically we don't need this check.
We can mark these CIDs as false alarm.

can we close this? i think its completed.

can we close this? i think its completed.

No worries, i pushed it.

Hey Jack,

new interval slider widget is added. It is a slider with two indicators. Legacy range would work on base slider as usual.

In T5900#96788, @jpeg wrote:

1. Move "range_interval" to a subclass of slider. It's confusing with other widgets (single value slider, progressbar or spinner).

It would be nice to have following feature in panes which @jpeg has already suggested before.
Please have a look at below video.

Panes theme needs to be changed.
Currently panes direction is in reference to the side bar orientation. I think it should be
according to the layout of its contents (first and second). We should fork out a task for theme.

I am summerizing the work done on panes widget.

1. ranmed to Efl.Ui.Panes 40945d985967e58d6e6c00ecfd6292a8d67b83b5
2. add first and second as content aliases 46d4df4962a723f12723a901b0d2ae2966ebb7ad
3. allow panes to respect the min size of its content 0ca98280d4b041e277d071ecfbcbc77db2eaa7cd
4. implement allow_user_size_hints on parts aaaf1a282a82f2f10b01f561ea210d5643707a1a
5. rename left/right_size_set to split_ratio & implements min_split_ratio on parts. 599bb616ec501bacd5af8cae3c9e5226509b3303
6. don't use edje restrict cal to calculate min size 521e2ab83ce0949de8101602d3596f30b7650bcb

@jpeg patch is ready! working and tested. :)

@jpeg the edje eo apis transition_duration_factor etcs would be legacy apis?

APIs content_left_size and content_right_size should be renamed to more meaningful.

well, it does not happen anymore. it may be fixed in your one of the patches related to supressing the error logs related to elm.box.

i got these warnings everywhere