elm_gen: update highlight geometry after clearing items

Summary:
After genlist/genrid items are cleared, item highlight still
remains in blank space. This patch fixes this by updating highlight.

Test Plan:
elementary_test -to fileselector
enter blank directory and see highlight still remains

Reviewers: jpeg, SanghyeonLee, cedric

Reviewed By: cedric

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

Signed-off-by: Cedric BAIL <cedric@osg.samsung.com>

image: remove EINA_UNUSED

Summary:
parameter is used

Signed-off-by: Minkyu Kang <mk7.kang@samsung.com>
Reviewers: Hermet, seoz, cedric

Reviewed By: cedric

Subscribers: seoz

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

Signed-off-by: Cedric BAIL <cedric@osg.samsung.com>

spinner: add label format validator

Summary:
Check for all error conditions case in elm_spinner_label_format_set
and set label format only if its valid.

Signed-off-by: Shilpa Singh <shilpa.singh@samsung.com>
@feature

Test Plan:
1. Launch elementary test spinner
2. Set invalid formats e.g. formats with out %, formats with more than
one format specifier "%d %s" etc:-

Reviewers: jpeg, cedric

Reviewed By: cedric

Subscribers: govi, buds, subodh6129

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

Signed-off-by: Cedric BAIL <cedric@osg.samsung.com>

Eo events: Migrate all of elm to the new event callback signatures

list: set update hints on item edje objects

without size hints on the edje object, a list item will never be sized
accurately, meaning that the size hints set on subobjects (icon, end) are
ignored in nearly all cases, and any previous sizing which was accurate was,
in fact, merely coincidentally accurate

@fix

list: set internal box as homogeneous if no separator items are present

this is a significant performance optimization since all non-separator
items should always be sized identically

list test: use 1.0 instead of EVAS_HINT_EXPAND

these are functionally identical but semantically different

list: do not change scr_min flags when setting an unenforced list mode

only LIMIT and EXPAND modes prohibit manually setting these flags. changing
them in any other scenario can result in overriding user-set flags

@fix

entry: send focus/unfocus signals to scroller interface

In case of scrollable entry, send focus/unfocus signals to scroller
interface edje.

view list: free style in base destructor.

typecast const char * to void * and pass ptr to free() since it doesn't modify the value pointed to,
either conceptually or in practice, it merely looks up the memory block using the pointer and deallocates it.

@fix

genlist: fix a typo.

elm_web: define elm_web_zoom_mode in eolian file elm_web.eo

elm_toolbar define elm_toolbar_item_state in elm toolbar eolian files

elm_thumb_animation: moved Elm_Thumb_Animation_Settings to elm_thumb.eo

moved eolian undefined enums to elm_systray.eo and elm_slider.eo

elm_prefs: move Elm Prefs_Reset_Mode to elm_general.eo

list: always apply new size hints for subobjects

if a content's size hints change, reapplying the list's size hints
to override existing hints is required to prevent some list items from
having a bad layout

@fix

list: do not constantly reset scr_min flags

these get set during mode_set and when directly changing the flags.
at no other time should they be changed

@fix

panes: add the style to fold by clicking

Some area that is split by this style panes can be folded by clicking.
The styles are "left-fold", "right-fold", "up-fold", "down-fold".

@feature

hoversel: set hover layer to layer of hover_parent

ref T3203

@fix

examples: fix evas3d example's "compile with" comment correctly

Summary: In "compile with" comment, filenames are different from the actual ones.

Reviewers: cedric, Oleksander, Hermet

Reviewed By: Oleksander, Hermet

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

genlist: focus out by pressing "up" key when no item is selected

Summary:
move focus out of genlist by pressing "up" key when no item
is selected.

Test Plan: elementary_test -to fileselector

Reviewers: SanghyeonLee, cedric, Jaehyun_Cho, Hermet

Reviewed By: Hermet

Subscribers: Hermet

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

fix the wrong operation of mirrored set

Summary:
fix the wrong operation of mirrored set.
when mirrored set is applied, notify change the theme using mirrored flag.
however, it is unnecessary operation.

@fix

Reviewers: Hermet, kimcinoo

Reviewed By: kimcinoo

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

Elm entry: No use of top object if not editable.

Summary: No use of top object if not editable.

Reviewers: cedric, Hermet

Reviewed By: Hermet

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

elm_image: fix wrong API call for scaling

Summary: Signed-off-by: Minkyu Kang <mk7.kang@samsung.com>

Reviewers: cedric, Hermet

Reviewed By: Hermet

Subscribers: seoz

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

elm widget: move some types used in elm_widget.eo to Eolian files

Move declarations of types used in elm_widget.eo to elm_general.eot and
elm_widget.eo.
Give them a Eolian name and use it instead of the old ones.

Correct the type name of some other types used as method parameters, by
using the Eolian names instead of legacy/generated names.

Reorganize the positions of elm_widget.eo.h inclusion and
of callback type declarations in elm_widget.h.
This make a clearer distinction of the types in elm_widget.h that are
still required in elm_widget.eo.h.

elm widget: add documentation from elm_focus.h to elm_widget.eo

Elm_photocam: moved enum Elm_photocam_Zoom_mode to elm_photocam.eo

elm_layout: moved Elm_Layout_Part_Alias_Description to elm_layout.eo

elm.Image: use eolian eva.images enum values in elm_image.eo and remove unused elm_image_common.h

elm win: use actual window size not requested to resize child objects

using requested size may not be correct, so use real size. this should
solve issues in some wm's and actually also solved some kind of object
size lag in wayland too.

@fix - onward from 79b6edd1a6ad763258337b7aac5bd7799411bb44

genlist: remove unused variable

In 4c86a66f this var was added without any user. Remove it.

edje_external: check icon param after other param is set in entry

Summary:
- icon param add an elm_icon and content set to icon part.
- entry's style is changing as other params are set and
  icon part may not exist in certain condition(default has no icon part)
- checking icon param later than other params will set icon part
  after the style is changed.

Reviewers: cedric, woohyun, jpeg, herdsman, tasn

Reviewed By: tasn

Subscribers: conr2d

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

Signed-off-by: Cedric BAIL <cedric@osg.samsung.com>

examples: fix evas3d_object_on_button_example

Summary:
Remove mirror map tranformation (needn't, we had already fixed inverted texture
in evas.canvas3d).
Use hardware pick instead of cpu pick

Reviewers: cedric, Hermet, raster

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

Signed-off-by: Cedric BAIL <cedric@osg.samsung.com>

scrollable: multiply default step values by elm scale

bigger scale = bigger objects = bigger scrollers = more wheeling needed

config: set larger vertical step size for Scrolling tab scroller

scrollers with lots of large items need larger or configurable step sizes to
make them usable

config: add option to modify scroll acceleration factor

when using a touchpad or frictionless mouse wheel, it becomes impossible
to accurately use a scroller which accelerates as scrollers on mobile devices
do. by setting this new option to 0, acceleration can be disabled and regular
behavior can be restored

ref 2ac2628612060114cf6e5205e5331044221178c6

@feature

scrolling: add option to disable scroll animations

when I scroll on a computer, I want my app to respond immediately. I don't
want to wait for it to finish scrolling and I don't want to have to guess
where the scroll will end.

ref 2ac2628612060114cf6e5205e5331044221178c6

@feature

config: add more frames for scrolling options

this looked weird having options outside a frame

@fix

elm win: fix window redraw if norender is enabled on withdrawn windows

we dont set this config by default so we dont see this - but a window
starts as withdrawn then becomes normal. this means the window starts
not rendering and may take 0.5 seconds to START because the eval is
deferred by 0.5 sec to avoid rapid switch to/from norender and other
states.

@fix

elm win: fix excess resize callbacks on windows when not really resizing

this fixes excess callbacks on window (and content) objhects resizing
down then back up again when windows are forcibly resized like
fullscreen, maximized and remembered windows.

@fix

elm_widget: do not set NULL as parent for object

When widget has no parent object, it breakes usage of a custom
Elm_Theme. This commit fixes it.

@fix

map: add scale_cb for map module

Summary:
- in elm_map's _source_mod_cb(), scale_cb is needed to
  load MAP tile module.
- currently, test_map mod.c lacks this scale_cb
  and fails to load Map Tile source.

Reviewers: woohyun, Hermet, cedric

Reviewed By: cedric

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

Signed-off-by: Cedric BAIL <cedric@osg.samsung.com>

elm_gen: highlight itself when there are no items

Summary:
When there are no items in genlist/gengrid, highlight stays at one dot of
top-left corner. This patch makes genlist/gengrid itself highlighted
when there are no items.

Reviewers: jpeg, SanghyeonLee

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

Signed-off-by: Cedric BAIL <cedric@osg.samsung.com>

edje_external: fix wrong logic in index

Summary:
- Eina_Bool active_exists is to check whether Eina_Bool active
  is valid or not.
- Setting active_exists to autohide_disabled_set is apparently wrong.

Reviewers: Hermet, cedric, jpeg, woohyun

Subscribers: conr2d

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

Signed-off-by: Cedric BAIL <cedric@osg.samsung.com>

entry: add elm.swallow.background part to all of groups

Summary:
elm.swallow.background part will be a default part for
widget's background image.
When entry widget's scrollable status is changed,
the background object will be moved to scr_edje or entry_edje.

Test Plan: N/A

Reviewers: kimcinoo, woohyun, cedric

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

Signed-off-by: Cedric BAIL <cedric@osg.samsung.com>

calendar: add elm.swallow.background part to all of groups

Summary:
elm.swallow.background part will be a default part for
widget's background image.

Test Plan: N/A

Reviewers: woohyun, kimcinoo, cedric, Hermet

Subscribers: Hermet

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

Signed-off-by: Cedric BAIL <cedric@osg.samsung.com>

spinner: add invalid input validity filter

Summary:
Do not allow invalid input in to spinner.
Invalid Input can be:
1. '-' should not be inserted anywhere but the starting position, and no more than one '-' can be inputted.
2.  No more than one '.' can be inserted.

Signed-off-by: Shilpa Singh <shilpa.singh@samsung.com>
@feature

Test Plan:
1. Launch elementary test
2. Run spinner demo
3. Enter invalid inputs like inserting more than one '.', '-' or insert '-' anywhere else part from 1st position.

Reviewers: raster, CHAN, subodh6129, cedric

Subscribers: govi, buds

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

Signed-off-by: Cedric BAIL <cedric@osg.samsung.com>

cursor: do not unconditionally unset cursor before setting an engine cursor

this is unnecessary: every single engine unsets the cursor when the object
has been deleted

this deletes enlightenment's cursor object

ref 678437003f219f783ba14859a805ea27997470fd

@fix

ctxpopup: completely disable focus-setting on show

this was previously disabled for internal list objects, and now
it is disabled for all subobjects of ctxpopup. applying focus here
invokes every possible focus bug in elementary and prevents the application
from being useful in any way

ref 974c8b05a4c97367b636593f5420ff01e8364aac
ref T3088

@fix

elm_spinner: do not SIGSEV on NULL format

Usually NULL was correct value, so lets return it back again

gengrid: add all contents unset API for content unset and reusing

Add elm_gengrid_item_all_contents_unset API for support content reuse
and unset. User can get back content objects in unrealized smart callback in item.
see elm_genlist_item_all_contents_unset also.

@feature

transit: rename from elm_transit_revert_go() to elm_transit_revert()

elm_transit_revert_go() sounds like it triggers animation on reverse way.

but this api acutally just change a current state.
It doesn't trigger the transtion.

genlist: bump item class version to 3 to fix abi breakage

when adding new methods to public interface structs (such as item class),
the version must be bumped and checked in the new interface method calls
in order to avoid breaking abi for previously-compiled apps

ref 4c86a66f28876b68e92a90c8f741eed1130dd034

tooltip: set pass events on the tooltip, not the content

tooltip content is swallowed, meaning that edje will be setting pass events
based on the theme

@fix

spinner: crash on illegal format set issue fix.

Summary:
Set any illegal format, spinner crashes, the format set to spinner
has to be valid hence a check is added initially itself to check for validity of
label formats.

Signed-off-by: Shilpa Singh <shilpa.singh@samsung.com>
Test Plan:
1. Set illegal format to spinner for e.g: elm_spinner_label_format_set(sp, "d");
2. Run spinner demo
Crash is observed

Reviewers: cedric

Subscribers: govi, buds

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

Signed-off-by: Cedric BAIL <cedric@osg.samsung.com>

spinner: fix vertical drag style and logic

Summary:
Fix vertical drag style and logic.

For vertical spinner graggable should move
in y direction and since drag value is inverse
of spinner increment and decrement so delta should be inversed.

@fix

Test Plan:
1. Elementary test
2. Vertical spinner

Reviewers: cedric

Projects: #elementary

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

Signed-off-by: Cedric BAIL <cedric@osg.samsung.com>

layout: send signal when content is set to bg

Summary:
 - elm.swallow.background is a part reserved for customized bg.
 - When default bg image is already set, default bg has to be hidden when customized bg is set.
 - Emitting signal when a content is set to this part makes this hiding process a lot easier.

Reviewers: kimcinoo, cedric

Subscribers: cedric, kimcinoo

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

Signed-off-by: Cedric BAIL <cedric@osg.samsung.com>

box: add support size hint padding

Summary: This patch adds suppoting size hint padding for elm_box.

Reviewers: raster, cedric, Hermet

Subscribers: cedric, kimcinoo

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

Signed-off-by: Cedric BAIL <cedric@osg.samsung.com>

spinner: add min, max filter

Summary:
In spinner's entry allow numbers only with in min/max range
Signed-off-by: Shilpa Singh <shilpa.singh@samsung.com>
@feature

Test Plan:
1. Set min/max value to spinner
2. Enter numbers beyond min/max value

Reviewers: CHAN, cedric

Reviewed By: cedric

Subscribers: subodh6129, buds, govi, CHAN, cedric, raster, id213sin

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

Signed-off-by: Cedric BAIL <cedric@osg.samsung.com>

genlist: add reusable_content_get item class function for support automatical content reusing in genlist.

Summary:
   Add new Item Class function reusable_content_get for improve genlist
   performance.
   If user want to reuse specific part contents, return old content
   which was passed by parameter of rreusable_content_get function.
   genlist will automatically caching those contents when item unrealized,
   and reusing it on item realizing.

   reusable_content_get will be performed content_get with old object caching.
   But if user return NULL in reusable_content_get, content_get is called to
   get new object.

   This feature is refered in Task T2813

   @feature

Test Plan: add test in genlist 1 case.

Reviewers: raster, singh.amitesh, cedric

Reviewed By: cedric

Subscribers: smohanty, seoz, woohyun, Hermet, eagleeye

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

Signed-off-by: Cedric BAIL <cedric@osg.samsung.com>

widget: don't access a widget focus if the object is not a widget.

entry: add elm_entry_select_region_get() API

Summary:
Already, there is a way to set a selection region:
elm_entry_select_region_set()
The get() API also useful and there is needs for this
inside of elm_entry.c. Add the API and replace codes
in atspi_text_selection_get with the API.
@feature

Test Plan:
1. Run "elementary_test -to entry3"
2. Make a selection on text.
3. Press "Sel" button.

Reviewers: tasn, herdsman, cedric, woohyun, Jaehyun, Hermet

Subscribers: Hermet

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

els_box: fix item sizing error

Summary:
Although items' size hint align are not set to fill,
Elm.Box expands its items.
This patch fixes item sizing error.

Test Plan: attached sample

Reviewers: cedric, jpeg, Hermet, Jaehyun, Jaehyun_Cho

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

[gengrid] possible crash issue fixed.

Summary: reset show_it, bring_in_it and corresponding flags on item deletion

Test Plan: delete item with animation and rotate the handset display.

Reviewers: raster, cedric

Subscribers: shilpasingh, godlytalias, govi, rajeshps

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

label: add elm.swallow.background part to all of groups

Summary:
elm.swallow.background part will be a default part for
widget's background image.

Test Plan: N/A

Reviewers: woohyun, cedric, kimcinoo

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

eolian helper: add --silent-types to mirror EFL

elementary: Use new object name "Elm.Widget_Item" instead of old name

Replace the old name "Elm.Object.Item" by the new name of the class
"Elm.Widget_Item" in all .eo files.

Elm_Image: moved undefined types to eolian file

elm win: replace Evas_Modifier_Mask by Evas.Modifier.Mask in elm_win.eo

elm genlist: use Eolian conventions for Elm.List.Mode in elm_genlist.eo

Use Eolian names for types and values of the enumeration Elm.List.Mode,
instead of legacy names, in elm_genlist.eo.

Elm.Map: move emun Zoom Mode, Source Type and Route Type to Eolian

move enum e structs defines of elm_atspi_text to eolian

elementary: move types to elm_interface_atspi_accessible.eo

Move structs, enums and typedefs definitions from
elm_interface_atspi_accessible.h to elm_interface_atspi_accessible.eo.
Give them namespace names to conform with Eolian policies.

Replace all occurrences in elm_interface_atspi_accessible.eo with the
Eolian names.

Adjust include order in elm_interface_atspi_accessible.h to avoid
errors.

elm_glview: move enums to eolian files

elm_gesture_layer: move enums to .eo

elm gengrid: replace Ecore_Pos_Map with Ecore.Pos_Map in elm_gengrid.eo

Replace Ecore_Pos_Map with Ecore.Pos_Map in elm_gengrid.eo to correctly
use the Eolian name of the type.

elementary: move definition of Elm_Glob_Match_Flags to elm_general.eot

Move definition of Elm_Glob_Match_Flags from elm_gen.h to
elm_general.eot.
Use the Eolian name Elm.Glob.Match_Flags and replace the older name
by it in all .eo files.

Keep the legacy prefix "ELM_GLOB_MATCH".

elementary: use the Eolian name for Elm.Genlist.Item.Type in .eo files

Replace Elm_Genlist_Item_Type with Elm.Genlist.Item.Type in .eo files
to use the proper Eolian name of this enumeration.

elementary: move definition of Elm_Gen_Item_Class to elm_general.eot

Move the definition of Elm_Gen_Item_Class from elm_gen.h to
elm_general.eot.
Also move the definition of its aliases Elm_Gengrid_Item_Class and
Elm_Genlist_Item_Class from elm_gengrid_common.h and
elm_genlist_common.h to elm_general.eot.

Rename these types to Elm.Gen.Item.Class, Elm.Gengrid.Item.Class and
Elm.Genlist.Item.Class to follow Eolian standards. And rename all
occurrences in .eo files accordingly.

Put elm_gen.h include above elm_general.h to allow the callback fields
to be defined in elm_general.eot.

elm_calendar,elm_datetime: move struct tm typedefs to .eo files

Move elm_calendar and elm_datetime struct tm typedefs to theirs
respective .eo files.
Use Ecore.Time, an alias to struct tm, to refer to this type in the
.eo files.

Use namespaced names for these typedefs, in order to follow Eolian
standards.

elementary: move definition of Elm_App_View_State to elm_general.eot

Move the definition of the enumeration Elm_App_View_State from
elm_app_common.h to elm_general.eot, and give it the the Eolian name:
Elm.App.View_State.

Replace Elm_App_View_State with Elm.App.View_State in
elm_app_client_view.eo and elm_app_server_view.eo.

elm gengrid: fix Elm.Gengrid.Item.Scrollto_Type usage in .eo file

Use the proper Eolian name "Elm.Gengrid.Item.Scrollto_Type" instead of
Elm_Gengrid_Item_Scrollto_Type in functions in elm_gengrid_item.eo.

elm entry: Move the declaration of Elm_Cnp_Mode to elm_entry.eo

Move the enumeration Elm_Cnp_Mode to elm_entry.eo to make it more
accessible for binding generation.
Also rename it to Elm.Cnp_Mode.

elementary: Rename Elm_Icon_Type to Elm.Icon.Type in .eo files

Correct the usage of the type Elm_Icon_Type by replacing it with its
Eolian name (Elm.Icon.Type) in .eo files.

elementary: rename classes in Eo files to use Elm as a namespace, not a prefix

Rename classes in Eo files that still uses "Elm" as a prefix to use it as a
namespace.

theme: fix typo for adding background in ctxpopup

Introduced in 6cf09175874cca9e5859d580a5bad706012b8880. Thanks a lot to Davide
Andreoli for spotting this!

button: add elm.swallow.background part

Summary:
- This swallow part is for custom bg image.
- previous bg has to be hid.

Reviewers: kimcinoo

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

Signed-off-by: Cedric BAIL <cedric@osg.samsung.com>

button: remove unnecessary action in program

Summary: @fix

Reviewers: kimcinoo, Hermet, cedric

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

Signed-off-by: Cedric BAIL <cedric@osg.samsung.com>

popup: add elm.swallow.background for background customize

Summary: add elm.swallow.background part in elm/popup/base/default

Reviewers: kimcinoo

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

Signed-off-by: Cedric BAIL <cedric@osg.samsung.com>

ctxpopup: add elm.swallow.backgournd for backgournd customize

Summary: add elm.swallow.background part in elm/ctxpopup/base/default

Reviewers: kimcinoo

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

Signed-off-by: Cedric BAIL <cedric@osg.samsung.com>

genlist: remove dead code.

win: fix a possible crash in elm_win_add

Summary:
The enginelist[0] can be NULL in some very rare cases.
Then enginelist[0] will be used for strcmp and it make a crash.
@fix

Test Plan: N/A

Reviewers: raster, cedric, woohyun, jypark, jaehwan

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

Signed-off-by: Cedric BAIL <cedric@osg.samsung.com>

elm: add Elm.Layout as edje externals' contents instead of Edje.Object

Summary:
When edje external parts add its content, edje object is added,
but it breaks elementary widget hierarchy.
In Elementary, edje external can use Elm.Layout to add its content.

Reviewers: cedric

Reviewed By: cedric

Subscribers: woohyun

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

Signed-off-by: Cedric BAIL <cedric@osg.samsung.com>

genlist: fix for a bug in tree expansion, T3132

Summary:
@fix T3132

If nobody sees issue with this change, this should be backported.

Signed-off-by: Godly T Alias <godly.talias@samsung.com>
Test Plan: elementary_test Genlist Tree, enable Tree effect

Reviewers: prince.dubey, SanghyeonLee, shilpasingh

Subscribers: govi, rajeshps

Maniphest Tasks: T3132

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

Signed-off-by: Cedric BAIL <cedric@osg.samsung.com>

transit: add possibility to revert an ongoing transition during play

Summary:
Currently the feature available in transit to reverse play a transition is
auto-reverse which will play a transition in reverse mode once a transition
is complete. This feature helps the user to revert a transition at any point
of time when transition is going on.

New API added.

@feature T3019

Use Case:
While doing pinch zoom, we will be doing zoom transit effect from one step to other, during that transit if user do the pinch in reverse direction this API can be called so that transition will be reverted easily.

Signed-off-by: godly.talias <godly.talias@samsung.com>
Test Plan: elementary_test Transit Resizing / Transit Zoom / Transit Bezier / Transit Custom / Transit Fade / Transit Flip

Reviewers: raster, prince.dubey, shilpasingh, Hermet, seoz, cedric

Subscribers: rajeshps, govi

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

Signed-off-by: Cedric BAIL <cedric@osg.samsung.com>

hoversel: improve key_action_move to handle disabled items

Summary:
Existing key_action_move logic used elm_widget_focus_cycle on hover,
but only handled the first and last item to move focus circularly.
So if the first or last item is disabled, focus movement is blocked.
This patch makes hoversel handle focus movement by itself to handle
disabled item better.

Test Plan: elementary_test -to hoversel

Reviewers: Hermet, cedric, Jaehyun

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

Signed-off-by: Cedric BAIL <cedric@osg.samsung.com>

combobox: prevent empty swallow to be shown on hover_dismiss animation

Summary:
Hover dismiss animation takes 0.5 sec to complete. if the genlist item_count
becomes zero, empty swallow is shown in dismiss animation.
@fix T3124
Signed-off-by: divyesh purohit <div.purohit@samsung.com>
Test Plan:
please run combobox example from elementary_test.

Open hover by pressing on left side button of combobox or pressing "enter/space" key in entry
contained in combobox.
Type a char which is not in any genlist item , for example "a' is not presented in any genlist
item.
empty swallow shouldn't seen in dismiss animation.

Reviewers: shilpasingh, cedric, raster

Subscribers: rajeshps, govi

Maniphest Tasks: T3124

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

Signed-off-by: Cedric BAIL <cedric@osg.samsung.com>

multibuttonentry: fix Segfault on pressing Delete Key

Summary:
Pressing "Delete" Key in MBE layout causes Segfault as MBE item is not deleted properly and also focus is not transferred back to MBE.

@fix

Signed-off-by: divyesh purohit <div.purohit@samsung.com>
Test Plan: Run MultibuttonEntry example from elementart_test . Add some MBE items, now press the delete key, it should delete items without causing segfaults and transfer focus to previous MBE item.

Reviewers: raster, cedric, CHAN, shilpasingh

Reviewed By: shilpasingh

Subscribers: rajeshps, govi

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

Signed-off-by: Cedric BAIL <cedric@osg.samsung.com>

entry: remove unnecessary variables

remove unnecessary dw, dh variables