Commit Graph

168 Commits

Author SHA1 Message Date
SangHyeon Lee 782f28e9a1 genlist : fix content cache issue by content_get
Summary:
genlist content_get sometimes work weird after reusable_content_get pushed.
cached content is shown even content_get function is not exist or
return NULL.
now cache check item_class instead of item style string.

Test Plan: check genlist cache in elementary_test

Reviewers: Hermet, cedric, raster

Subscribers: Jaehyun_Cho, jpeg

Differential Revision: https://phab.enlightenment.org/D3921

Conflicts:
	src/lib/elementary/elm_genlist.c
2016-05-12 07:12:18 +09:00
Mike Blumenkrantz 51e86a4c0d elm_genlist: throw CRI when group item insertion fails
this really needs a better solution since it results in a blank genlist
item space, with the item itself typically located somewhere offscreen due
to caching
2016-04-30 14:31:29 -04:00
Mike Blumenkrantz 43d82e567a elm_genlist: when appending items to the parent, prepend to the parent
the parent/group item is located after its child items in the list of items,
so if an item is appended to the parent's list of items then it must be
prepended to the parent or else it will end up being in the wrong group

@fix
2016-04-30 14:26:51 -04:00
Mike Blumenkrantz 5c933b0176 elm_genlist: accurately handle contents for cached items
more fixes for the comically unreviewed revision from elm which continues
to cause bugs months after it was pushed

ref 4c86a66f28876b68e92a90c8f741eed1130dd034 (elm)
ref e88423e994
2016-04-30 13:53:46 -04:00
Mike Blumenkrantz 673f703bb0 elm_genlist: outdent _item_content_realize() harder
no functional changes
2016-04-30 13:53:41 -04:00
SangHyeon Lee 0181a940f1 Revert "genlist: recalc item size after item content field updated"
This reverts commit b48d3eb04e.

This patch can be occurred performance issue.
update is needed for only non-homogeneous case.
I'll revert patch and update new patch in phabricator for check.
2016-04-28 16:04:54 +09:00
SangHyeon Lee 5f7f59b3d8 Revert "genlist: scroll to item when calc is done"
This reverts commit 87b12327c3.

This patch occurred some side effect.
Need to fix the code without any side effect.
2016-04-28 16:04:54 +09:00
Andrii Kroitor 802e671cda elementary: fix genlist tree filters
Summary:
In tree mode if parent item is hidden all its subitems should also be hidden.
@fix

Reviewers: Hermet, raster, cedric

Reviewed By: cedric

Subscribers: jpeg, reutskiy.v.v

Differential Revision: https://phab.enlightenment.org/D3918

Signed-off-by: Cedric Bail <cedric@osg.samsung.com>
2016-04-26 14:54:39 -07:00
Vyacheslav Reutskiy 287f84baaf elm_genlist: fix the sorted insert after expanded tree item
Insert a new item after subitem of ELM_GENLIST_ITEM_TREE item, if
it expanded.

@fix
2016-04-26 15:16:09 +03:00
SangHyeon Lee c5427e875d genlist : arrange focus set in mouse up
Summary :
when mouse up happens, item must be focused, and after that,
select must be called.
current logic, focus is already called inside the select function
and again focus set is called in the end of mouse_up,
so if user new focused widget inside the select callback,
mouse_up forcely take the focus into the item again.
select callback should comes after the focus callback calls
to work properly.

@fix
2016-04-22 16:08:25 +09:00
SangHyeon Lee 81f45c1b90 genlist: call update_job after smart_calculation done
Summary :
if item_update happens before the smart calcuate finished,
item can be delayed updated(next frame).
to prevent this issue, if unsolved job is remaining after
smart calc done, call update_job directly and remove job.

@fix
2016-04-19 18:43:55 +09:00
SangHyeon Lee b48d3eb04e genlist: recalc item size after item content field updated
Summary :
after item content fields updated, item and block should be
recalculated about it's height because content size can be updated.
if do not recaculate, items can be overlapped.

@fix
2016-04-19 14:57:45 +09:00
Jee-Yong Um 87b12327c3 genlist: scroll to item when calc is done
Summary:
If elm_genlist_item_bring_in() is called immediately after appending
item, genlist will not scroll to the last one but stop in the middle.
This patch blocks genlist from scrolling to not calculated item.

Test Plan:
1. Uncompress attached file
2. make
3. ./test

Reviewers: cedric, SanghyeonLee

Reviewed By: SanghyeonLee

Subscribers: jpeg

Differential Revision: https://phab.enlightenment.org/D3861
2016-04-19 14:00:42 +09:00
Shuhrat Dehkanov eae53f2218 genlist: do not evaluate against max coord if the value is negative
Summary:
If x is already less than '0', there is no need to check if it is
bigger than pan_max_x. Likewise, if y is already less than '0',
there is no need to check if it is bigger than pan_max_y.

Reviewers: Hermet, cedric, SanghyeonLee, singh.amitesh

Reviewed By: singh.amitesh

Subscribers: seoz, minkyu, sju27, jpeg

Differential Revision: https://phab.enlightenment.org/D3865
2016-04-11 17:44:56 +05:30
SangHyeon Lee c0bbf53cb9 support elm_object_item_style_set/get on genlist items 2016-04-01 16:53:57 +09:00
SangHyeon Lee fc2f341e41 fix item_class refcount bug when class updated 2016-04-01 16:40:16 +09:00
SangHyeon Lee e8c3346379 fix genlist content leak in reausable case 2016-04-01 16:26:57 +09:00
Cedric BAIL c2a1c49ab2 elementary: move all legacy files to their expected new location. 2016-03-23 13:24:41 -07:00