Page MenuHomePhabricator
Feed Advanced Search

Thu, May 28

herb updated the diff for D11876: Support WebP Animation Image Files.

remove libweb dependency for libwebpdemux

Thu, May 28, 7:32 PM · efl
herb updated the diff for D11876: Support WebP Animation Image Files.

update codes

Thu, May 28, 12:45 AM · efl
herb updated the diff for D11876: Support WebP Animation Image Files.

Add checking return value for getting next frame

Thu, May 28, 12:25 AM · efl

Wed, May 27

herb updated the diff for D11876: Support WebP Animation Image Files.

update codes

Wed, May 27, 9:53 PM · efl
herb updated the diff for D11876: Support WebP Animation Image Files.

update codes

Wed, May 27, 5:56 AM · efl

Mon, May 25

herb updated the diff for D11876: Support WebP Animation Image Files.

modified code for comments

Mon, May 25, 7:04 PM · efl
herb closed D11846: ecore_evas_wayland_common: keep the backword compatibility for wayland dnd handler.
Mon, May 25, 2:17 AM · efl
herb added a comment to D11877: elm_dnd: do not register widgets twice.

This is tested by my environments and removed the duplicated register cases.

Mon, May 25, 1:48 AM · efl
herb updated the diff for D11876: Support WebP Animation Image Files.

clean up the codes

Mon, May 25, 1:38 AM · efl
herb updated the diff for D11876: Support WebP Animation Image Files.

update codes

Mon, May 25, 1:34 AM · efl
herb updated the diff for D11876: Support WebP Animation Image Files.

update files

Mon, May 25, 1:26 AM · efl
herb updated the diff for D11876: Support WebP Animation Image Files.

update sample codes

Mon, May 25, 12:33 AM · efl
herb updated the diff for D11876: Support WebP Animation Image Files.

update codes

Mon, May 25, 12:09 AM · efl
herb requested review of D11876: Support WebP Animation Image Files.
Mon, May 25, 12:06 AM · efl

Thu, May 21

herb requested review of D11867: evas_vg_container: return NULL in evas_vg_container_add when the parent of container is NULL.
Thu, May 21, 8:12 PM · efl

Tue, May 19

herb added a comment to T8706: CnP and Dnd Compatibility.

for DnD-4: when the drag object moved to the target object and the drag is finished, the target->currently_inside flag should be false. but _enter_state_changed_cb dose not change the target->currently_inside flag to false when drag is done.
for CnP-1: I think this is the policy problem, we decide to allow the copy and paste without the input changes.
for CnP-2: The problem is that the call of loss_callback can be different depends on the server. I think it's better to give same result regardless the server.

Tue, May 19, 5:23 AM
herb added a comment to T8706: CnP and Dnd Compatibility.

hello, @bu5hm4n
for CnP-2 (new case)
I have a question about elm_cnp_loss_callback_set() function.
I think the loss_callback behavior is changed compared to before.

Tue, May 19, 12:37 AM
herb added a comment to T8706: CnP and Dnd Compatibility.

hello, @bu5hm4n
for CnP-1
The serial is updated by only input codes, so when we try to copy and paste using button(the serial is updated and copy and paste work well)
ex)
src/lib/ecore_wl2/ecore_wl2_input.c:697: input->display->serial = serial;
src/lib/ecore_wl2/ecore_wl2_input.c:721: input->display->serial = serial;
src/lib/ecore_wl2/ecore_wl2_input.c:768: input->display->serial = serial;
src/lib/ecore_wl2/ecore_wl2_input.c:942: input->display->serial = serial;
src/lib/ecore_wl2/ecore_wl2_input.c:969: input->display->serial = serial;
src/lib/ecore_wl2/ecore_wl2_input.c:1062: input->display->serial = serial;
src/lib/ecore_wl2/ecore_wl2_input.c:1241: input->display->serial = serial;

Tue, May 19, 12:18 AM

Mon, May 18

herb added a comment to T8706: CnP and Dnd Compatibility.

@bu5hm4n
for Dnd-4
before EFL_UI_DND_EVENT_DROP_ENTERED and EFL_UI_DND_EVENT_DROP_LEFT event are triggered by only motion events.
but the events in _enter_state_change_cb() will emit additional event to the application.

Mon, May 18, 10:50 PM
herb added a comment to T8706: CnP and Dnd Compatibility.

@bu5hm4n
DnD-1: drop event register twice is not reproduced with the patch in my environment.

Mon, May 18, 10:00 PM
herb added a comment to T8706: CnP and Dnd Compatibility.

I have question about _enter_state_change_cb(), I think in _motion_cb() we emit EFL_UI_DND_EVENT_DROP_LEFT, EFL_UI_DND_EVENT_DROP_ENTER already.
should we emit the event in _enter_state_change_cb()?
(before the drag enter and leave event are triggered by the drag object moves)

Mon, May 18, 5:53 AM
herb added a comment to T8706: CnP and Dnd Compatibility.

I think this seems to be bug, there is a case that drop_target is added twice.

Mon, May 18, 5:19 AM
herb added a comment to T8706: CnP and Dnd Compatibility.

Case: DnD-1
I think the drop left and drop entered event order are important.

Mon, May 18, 2:00 AM
herb added a comment to T8706: CnP and Dnd Compatibility.

@ bu5hm4n
I created the patch for resolving DnD2 problem.
please review this :)
https://phab.enlightenment.org/D11846

Mon, May 18, 12:56 AM
herb requested review of D11846: ecore_evas_wayland_common: keep the backword compatibility for wayland dnd handler.
Mon, May 18, 12:56 AM · efl

Fri, May 15

herb requested review of D11839: evas_vg_shape: return NULL in evas_vg_shape_add when the parent of shape is NULL.
Fri, May 15, 12:59 AM · efl

Mon, May 11

herb created T8706: CnP and Dnd Compatibility.
Mon, May 11, 4:43 AM
herb updated the diff for D11781: efl_canvas_vg_node: fix the implemenation function of the efl_canvas_vg_node(evas_object_above_get, evas_object_below_get).

update codes

Mon, May 11, 1:26 AM · efl
herb updated the diff for D11781: efl_canvas_vg_node: fix the implemenation function of the efl_canvas_vg_node(evas_object_above_get, evas_object_below_get).

update codes

Mon, May 11, 1:22 AM · efl
herb updated the diff for D11781: efl_canvas_vg_node: fix the implemenation function of the efl_canvas_vg_node(evas_object_above_get, evas_object_below_get).

update code

Mon, May 11, 1:09 AM · efl

Wed, May 6

herb requested review of D11781: efl_canvas_vg_node: fix the implemenation function of the efl_canvas_vg_node(evas_object_above_get, evas_object_below_get).
Wed, May 6, 5:24 AM · efl

Apr 12 2020

herb updated the diff for D11690: elm_cnp: fix memory leaks.

fix warning

Apr 12 2020, 10:55 PM · efl
herb updated the diff for D11690: elm_cnp: fix memory leaks.

update comments

Apr 12 2020, 10:40 PM · efl
herb requested review of D11690: elm_cnp: fix memory leaks.
Apr 12 2020, 10:17 PM · efl

Mar 17 2020

herb closed D11446: ecore_cocoa: fix wrong word in documentation.
Mar 17 2020, 2:32 AM · efl

Mar 16 2020

herb requested review of D11521: evas_vg: modified the join enum documentation.
Mar 16 2020, 9:03 PM · efl
herb added a child revision for D11519: evas_vg: add since tags for evas vector APIs: D11521: evas_vg: modified the join enum documentation.
Mar 16 2020, 9:02 PM · efl
herb abandoned D11511: evas_vg: modified the documentation and parameter names for the evas vector APIs.

This revision is divided into D11517, D11518, D11519

Mar 16 2020, 7:54 PM · efl
herb added a parent revision for D11518: evas_vg: refined the documentation for evas vector APIs: D11517: evas_vg: refined the evas vector APIs set.
Mar 16 2020, 7:45 PM · efl
herb added a child revision for D11517: evas_vg: refined the evas vector APIs set: D11518: evas_vg: refined the documentation for evas vector APIs.
Mar 16 2020, 7:45 PM · efl
herb requested review of D11519: evas_vg: add since tags for evas vector APIs.
Mar 16 2020, 7:44 PM · efl
herb added a child revision for D11518: evas_vg: refined the documentation for evas vector APIs: D11519: evas_vg: add since tags for evas vector APIs.
Mar 16 2020, 7:44 PM · efl
herb requested review of D11518: evas_vg: refined the documentation for evas vector APIs.
Mar 16 2020, 6:05 AM · efl
herb requested review of D11517: evas_vg: refined the evas vector APIs set.
Mar 16 2020, 6:04 AM · efl
herb abandoned D11514: evas_vg: refined the documentation for evas vector APIs.
Mar 16 2020, 5:59 AM · efl
herb abandoned D11513: evas_vg: refined the evas vector APIs set.

This patch looks very similar to D11511. Can you take into account the comments I made there?

Mar 16 2020, 5:59 AM · efl
herb updated the diff for D11514: evas_vg: refined the documentation for evas vector APIs.

Updating D11514: evas_vg: refined the documentation for evas vector APIs

Mar 16 2020, 5:55 AM · efl
herb requested review of D11514: evas_vg: refined the documentation for evas vector APIs.
Mar 16 2020, 5:51 AM · efl
herb updated the diff for D11513: evas_vg: refined the evas vector APIs set.

evas_vg: refined the evas vector APIs set

Mar 16 2020, 5:42 AM · efl
herb updated the diff for D11513: evas_vg: refined the evas vector APIs set.

update source

Mar 16 2020, 5:37 AM · efl
herb added a comment to D11511: evas_vg: modified the documentation and parameter names for the evas vector APIs.

@segfaultxvai
hello, thanks for the comment, I will divide this patch for readability(D11513 is the first path of this)
Evas_Vg_Join enum values refers to src/lib/efl/interfaces/efl_gfx_types.eot:71(Efl.Gfx.Join)
and in case of other comments, I will change to follow your comments.

Looks like you are defining new enums here that are already present in Eo files. Can you explain why?

=> because I think legacy APIs should use legacy enum values, so I defined evas enum types
ex) for hiding gfx enums
evas_vg_shape_path_set(Evas_Vg_Shape *obj, const Evas_Vg_Path_Command *op, const double *points)
evas_vg_shape_stroke_dash_get(Evas_Vg_Shape *obj, const Evas_Vg_Dash **dash, unsigned int *length)
evas_vg_gradient_stop_set(Evas_Vg_Gradient *obj, const Evas_Vg_Gradient_Stop *colors, unsigned int length)
evas_vg_shape_stroke_dash_set(Eo *obj, const Evas_Vg_Dash *dash, unsigned int length)

I think @zmike worked on the move of this stuff from Legacy to Eo some time ago, it might be worth asking his opinion.

Mar 16 2020, 4:37 AM · efl
herb updated the diff for D11513: evas_vg: refined the evas vector APIs set.

update cdes

Mar 16 2020, 4:14 AM · efl
herb requested review of D11513: evas_vg: refined the evas vector APIs set.
Mar 16 2020, 4:11 AM · efl
herb abandoned D11512: evas_vg: refined the evas vector APIs set.

abandon this revision

Mar 16 2020, 3:53 AM · efl
herb updated the diff for D11512: evas_vg: refined the evas vector APIs set.

evas_vg: refined the documentation for evas vector APIs

Mar 16 2020, 3:51 AM · efl
herb requested review of D11512: evas_vg: refined the evas vector APIs set.
Mar 16 2020, 3:49 AM · efl
herb updated the diff for D11511: evas_vg: modified the documentation and parameter names for the evas vector APIs.

update tags

Mar 16 2020, 1:16 AM · efl
herb updated the diff for D11511: evas_vg: modified the documentation and parameter names for the evas vector APIs.

updated version for new evas enums

Mar 16 2020, 1:09 AM · efl
herb abandoned D11492: evas_canvas_vg: modified the documentation for the evas vector APIs.

I will update as other patch

Mar 16 2020, 1:02 AM · efl
herb requested review of D11511: evas_vg: modified the documentation and parameter names for the evas vector APIs.
Mar 16 2020, 1:00 AM · efl

Mar 11 2020

herb requested review of D11492: evas_canvas_vg: modified the documentation for the evas vector APIs.
Mar 11 2020, 11:54 PM · efl
herb requested review of D11491: efl_canvas_vg_shape: fix the wrong parameter order in the wrapper of evas_vg_shape_append_cubic_to().
Mar 11 2020, 11:38 PM · efl

Mar 4 2020

herb updated the diff for D11446: ecore_cocoa: fix wrong word in documentation.

updated codes for adding documentation

Mar 4 2020, 6:54 PM · efl
herb requested review of D11446: ecore_cocoa: fix wrong word in documentation.
Mar 4 2020, 4:13 AM · efl

Jan 8 2020

herb added a comment to T8547: Efl.Ui.Selection: efl_ui_selection_get type clipboard not fired.

I checked this codes on my desktop and it is reproduced with x11 environment.
In case of pasting twice, it is blocked by below codes.

Jan 8 2020, 2:49 AM

Dec 5 2019

herb requested review of D10820: efl_ui_alert_popup: remove beta for efl_ui_alert_popup_title.
Dec 5 2019, 11:45 PM · efl

Dec 4 2019

herb requested review of D10794: fix the wrong method signature for evas_textgrid.
Dec 4 2019, 1:55 AM · efl

Nov 6 2019

herb abandoned D9835: prevent using the efl_gfx_hint_size_combined_min_get function from the legacy widgets.
Nov 6 2019, 8:02 PM · efl

Nov 5 2019

herb added a reviewer for D10596: efl_ui_alert_popup: add title part in parts block: zmike.
Nov 5 2019, 6:28 PM · efl
herb updated the diff for D10596: efl_ui_alert_popup: add title part in parts block.

update the codes

Nov 5 2019, 6:26 PM · efl
herb updated the diff for D10596: efl_ui_alert_popup: add title part in parts block.

add @beta tag

Nov 5 2019, 4:11 AM · efl
herb updated the diff for D10596: efl_ui_alert_popup: add title part in parts block.

modified the codes

Nov 5 2019, 4:04 AM · efl

Nov 4 2019

herb requested review of D10596: efl_ui_alert_popup: add title part in parts block.
Nov 4 2019, 11:06 PM · efl

Oct 14 2019

herb added a comment to D10386: testgen: sync the latest eolian grammar.

Hi, lauromoura,
I made a patch that synchronizes with the upstream code.
Actually the upstream eolian grammar has been changed from 2019.Oct
so I added more verbs and modifyed the event and function names.
You can apply this patch after rebasing.

Oct 14 2019, 12:09 AM · efl

Oct 13 2019

herb requested review of D10386: testgen: sync the latest eolian grammar.
Oct 13 2019, 11:51 PM · efl

Sep 4 2019

herb requested review of D9835: prevent using the efl_gfx_hint_size_combined_min_get function from the legacy widgets.
Sep 4 2019, 5:34 AM · efl

Sep 3 2019

herb requested review of D9818: elm_popup: calculate the label min size of popup before the popup group calculation.
Sep 3 2019, 3:30 AM · efl

Aug 29 2019

herb added a comment to T7946: enum Efl.Ui.Popup_Align.

After zmike's refactoring the popup, the enum will be removed

Aug 29 2019, 1:04 AM · efl: api, efl (efl-1.23), efl: language bindings

Aug 27 2019

herb added a comment to T7902: efl.ui.popup.

In fact, I don't have enough time to handle this since I have to migrate upstream code to tizen until next wednesday.
and I should make the entire EflSharp UnitTest set until end of september. I think the amount of TC set that I have to make quite large.
so I'm not sure to answer :(

Aug 27 2019, 5:30 AM · efl: api, efl (efl-1.23), efl: language bindings
herb added a comment to T7902: efl.ui.popup.

@zmike
You mean that making the parts class for the buttons, right?
I also agree with your opinion and I think the parts should implement content and text interfaces for the setting text and icon of buttons

Aug 27 2019, 5:17 AM · efl: api, efl (efl-1.23), efl: language bindings

Aug 25 2019

herb requested review of D9735: efl_ui_format: add null checking codes for eina_strftime in _do_format_string function.
Aug 25 2019, 10:17 PM · efl

Aug 20 2019

herb added a comment to T7902: efl.ui.popup.

@zmike
I talked with woohyun about this refactoring for popup,
we have to finish this task until early september since we will start the release for EflSharp after the middle of september.
please take care of this schedule.

Aug 20 2019, 9:50 PM · efl: api, efl (efl-1.23), efl: language bindings
herb added a comment to T7902: efl.ui.popup.

@zmike
The application developers in tizen need two types of popup such as context style and alert style
In case of the context style like elm_ctxpopup of efl needs priority align feature because the context popup should be shown depending on mouse position and in the screen boundary.
when the context popup is created near the screen boundary, the priority is used to find the optimal position of the context popup which the user wants.
In case of the alert style like elm_popup of efl needs align feature because some developer want the popup to be aligned whenever the screen is resized.
However If there are some ways to set the priority using algin hints or find optimal position, I think we can remove the priority set API :)

Aug 20 2019, 3:15 AM · efl: api, efl (efl-1.23), efl: language bindings

Aug 12 2019

herb added a comment to T7902: efl.ui.popup.

@zmike
Hi, I agree to your opinion, the usage of the anchor popup align is different from base popup.
so I think the align feature in anchor popup should be separated from Efl.Ui.Popup.Align
and make the anchor popup has its own priority align features.

Aug 12 2019, 9:27 PM · efl: api, efl (efl-1.23), efl: language bindings

Jul 25 2019

herb requested review of D9413: Efl.Gfx.Hint_Align: add define values for Efl.Gfx.Hint_Align.
Jul 25 2019, 11:51 PM · efl

Jul 2 2019

herb abandoned D9199: efl_ui_layout: fix the case that item list is not deleted when trying to insert the contents to the same part..

I agreed with you :)
abandon this patch

Jul 2 2019, 9:25 PM · efl

Jun 27 2019

herb requested review of D9199: efl_ui_layout: fix the case that item list is not deleted when trying to insert the contents to the same part..
Jun 27 2019, 10:33 PM · efl

Jun 26 2019

herb added a comment to T7946: enum Efl.Ui.Popup_Align.

I agree to bu5hm4n's opinion and I think segfaultxavi's idea can be good alternatives.
It looks good for me :)

Jun 26 2019, 6:37 PM · efl: api, efl (efl-1.23), efl: language bindings
herb added a comment to T7946: enum Efl.Ui.Popup_Align.

The enum values support the alignment setting for the very high frequent alignment.
I think this is the important factor from the user perspective.
Also there need to add left_top, right_top, left_bottom, right_bottom enum values.

Jun 26 2019, 2:12 AM · efl: api, efl (efl-1.23), efl: language bindings

Jun 24 2019

herb added a comment to T7902: efl.ui.popup.

@bu5hm4n
I think the max_size_hint is different with popup_size.
the max_size in efl_ui_popup means the maximum size that the popup can expand.
If the max_size_hint is used to change the popup size, the popup will not be expandable.
Therefore, I think it's not good to change popup_size to max_size_hint.
In case of the alignment API of popup, It is designed to notify the options to the users.
To use this API makes the code simple and easy.

Jun 24 2019, 12:08 AM · efl: api, efl (efl-1.23), efl: language bindings

Jun 5 2019

herb abandoned D9068: evas_out: add condition for efl_canvas_output_view_set.
Jun 5 2019, 4:21 AM · efl

Jun 4 2019

herb requested review of D9068: evas_out: add condition for efl_canvas_output_view_set.
Jun 4 2019, 4:26 AM · efl
herb requested review of D9067: efl_ui_layout_orientation: fix the bitwise condition logic.
Jun 4 2019, 4:08 AM · efl

Apr 24 2019

herb added a comment to D8696: eo_parser: fix unreachable code.
In D8696#158431, @q66 wrote:

No, it's not intended, but all you need to do is simply remove the first is_enum check and leave the second one where it is.

Apr 24 2019, 2:43 AM · efl
herb updated the diff for D8696: eo_parser: fix unreachable code.

update code

Apr 24 2019, 2:41 AM · efl
herb added a comment to D8696: eo_parser: fix unreachable code.
In D8696#158404, @q66 wrote:

The goto is unnecessary, syntax error means a long jump

@q66
Hi, you mean these codes are intended?
but actually the following codes are not reachable.
2198: if (is_enum)
2199: eo_lexer_syntax_error(ls, "enums cannot have @free");

Apr 24 2019, 1:02 AM · efl

Apr 23 2019

herb requested review of D8696: eo_parser: fix unreachable code.
Apr 23 2019, 10:14 PM · efl

Apr 22 2019

herb closed D8683: elc_hoversel: fix selected event compatibility issue.
Apr 22 2019, 11:59 PM · efl
herb updated the diff for D8683: elc_hoversel: fix selected event compatibility issue.

update code

Apr 22 2019, 11:49 PM · efl
herb requested review of D8683: elc_hoversel: fix selected event compatibility issue.
Apr 22 2019, 11:38 PM · efl