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

Elm_Entry: Magnifier Not working properly on corner areas issue fix.

Summary:
When magnifier is moved across corners and outside entry, working of magnifier
is not proper.

Magnifier is Restricted with in the window and to entry area and its position depends
on cursor.
Signed-off-by: Thiep Ha <thiepha@gmail.com>
Signed-off-by: Shilpa Singh <shilpa.singh@samsung.com>
@fix

Test Plan:
1. Enable Magnifier export ELM_MAGNIFIER_ENABLE=1
2. Launch elementary test and launch entry3 demo
3. Move magnifier on corners and outside of second entry

The contents of magnifiers are undefined when moved outside entry
and clipper moves in a different direction as magnifier.

Reviewers: cedric, raster, thiepha

Reviewed By: thiepha

Subscribers: buds, govi, rajeshps, subodh6129

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

genlist: update all realized items in a single job

Summary:
Its really weird to call same job for each item and deletes that job in
next iteration. Since item_update() deletes the old update_job ecore job
again so basically we call update job only for the last realized item in the list.

 @fix

Reviewers: raster, jpeg, cedric, SanghyeonLee, tasn

Reviewed By: SanghyeonLee

Subscribers: seoz

Projects: #elementary

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

js: Added layout example

js: Added combobox class to JavaScript binding

calendar.edc: fix typo in text style

@fix

elementary: update mailmap addresses

Signed-off-by: Chris Michael <cpmichael@osg.samsung.com>

panel: no need for destructor anymore as the event handler will disappear in the destructor.

genlist: make automatic bounce test configurable.

naviframe: switch to use animator,tick.

genlist: switch to efl.core.animator tick.

tests: switch anim test to Efl.Core.Animator tick.

map: use Efl.Core.Animator tick.

photocam: switch to use efl.core.animator.

interface_scrollable: drop use of ecore_animator in favor of efl.core.animator tick

scroller: prevent the use of uninitialized data in some case.

panel: switch to use per window animator.

Tick provided by to efl.core.animator.

win: remove animator,tick.

multibuttonentry: Add item unfocused callback.

Summary:
Even MBE item lose focus, item state did not changed before.
So i send a signal to make item state as default when item lose own focus.

Signed-off By: Kumar Navneet <k.navneet@samsung.com>

@fix

Test Plan:
Run MBE sample.
Click any item.
Click out of MBE area.
See the item still got a highlight.

Reviewers: Hermet, cedric

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

Conflicts:
data/themes/edc/elm/multibuttonentry.edc

elm_hoversel: add "escape" key action to dismiss

Summary:
To allow to dismiss hoversel expansion without selecting item
by key, "escape" key action is added.

Test Plan: elementary_test -to hoversel

Reviewers: Jaehyun_Cho, cedric, divyesh

Reviewed By: divyesh

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

fix calendar today bug

Summary: After today mark appear once on calendar, it never disappear when user change calendar's year or month

Test Plan: elementary_test calendar2

Reviewers: Hermet

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

elm calendar: add mark_del method and move Elm_Calendar_Mark to .eo file

Summary:
Add mark_del as a method in the Elm.Calendar class.
Move the elm_calendar_mark_del() function (which do not receive an Eo object)
to elm_calendar_legacy.h.

Move the declaration of Elm_Calendar_Mark to elm_calendar.eo as an opaque
struct.
Rename it to Elm.Calendar.Mark to follow eolian standards.

Update documentation to refer to mark_del method instead of the
elm_calendar_mark_del() function.

Reviewers: felipealmeida, vinipsmaker, larryolj, cedric, tasn

Reviewed By: tasn

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

Elm entry: Fix selection using handlers.

Summary:
Fix item selection using selection handler

In case of selection using handler, there is a problem in case the entry
has the item, mouse down over handler consider the current down
coordinate to start selection and it becomes very difficult to select in the
situation shown in the test plan. So I consider the next/previous  line
geometry to fix it.

@fix

Test Plan:
1.Enable selection handler
2. Have a look at the setup in the attached image
{F27396}

3. Start selecting from last item and extent to cover the last text.
4.Observe that its impossible to select the last text until you move mouse out of the window
So, in mobile device its almost impossible to select using handlers in this situation.

Please use the attached application to test it.
{F27402}

Reviewers: thiepha, herdsman, tasn

Reviewed By: tasn

Subscribers: shilpasingh

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

Elm entry: Use edje api and simplify empty check logic

Summary: Use edje api and simplify empty check logic

Test Plan: NA

Reviewers: herdsman, tasn

Reviewed By: tasn

Subscribers: shilpasingh, cedric

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

configure: Switch to dev mode again. Merge window for 1.18 is now open.

release: Update NEWS and bump version for 1.17.0 release

theme: make sure vertical_separated_bar_disabled.png lands in the dist tarball

A ix for commit 186aeb42eb7e97c07a58a30aafbca5d05c4e2302 which introduced the
image but did not made sure it gets shipped.

layout: test code refactoring.

remove a comment.
pass immediate parents.
remove unnecessary show call.

genlist test: fix invalid free on window close

genlist/gengrid: Fix focus hilight error

The error was due to an invalid cast between real data and Eo ptr.
Thanks @DaveMDS for reporting.

Fixes T3122

@fix

test: reflow check boxes in genlist and gengrid focus tests

Without this, the items never get unrealized, except on
window close. This allows better testing of those widgets.

[toolbar] Adjust icon size along with text etc.

As discussed on the mailing list elm should scale the whole UI
based on the scale value. Make toolbar icons match this.

@fix

[toolbar] don't leak icons into the UI

This cleans up orphan icons being left in the UI if you call icon_set
after the toolbar has been displayed.

This was most noticable if the toolbar or buttons move first

@fix

elm_focus: refine focus.edc

Summary: remove whitespaces and add programs block

Reviewers: Jaehyun_Cho

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

Conflicts:
data/themes/edc/elm/focus.edc

elm_focus: fix mislocation error during continuous focus movement

Summary:
When focus is moved fast and continuously, newly created animator
conflicts with the animator for previous movement, so focus bounces
among locations.
This patch removes previous animator and move focus to the final
location before creating new animator.

Test Plan:
elementary_test -to focus
elementary_test -to genlist

Reviewers: raster, jpeg, cedric, Jaehyun_Cho

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

remove incorrect references of insert_before/after in EDC

Summary:
This patch removes unavailable references to nonexistent parts
from default theme.

Reviewers: raster, jpeg, cedric

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

combobox: avoid calling item_selected and dismissed signal on show

Summary:
After this patch https://phab.enlightenment.org/rELM0f6e1a46a402fd9cf848ffb1bd47c6b406eb7ffa
calling item,selected and dismissed signals should be avoided on combobox show.
Signed-off-by: Divyesh Purohit <purohit.div@gmail.com>
Test Plan: please run combobox example from elementary_test

Reviewers: cedric

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

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

combobox: fix continuous clicks on hover

Summary:
Continouous clicks on hover would need to ensure that variable
on_dismiss of combobox_vertical style in hover.edc should 0 when slot is being shown, otherwise events
would be disabled on hover even though swallow content is shown.

@fix

Signed-off-by: Divyesh Purohit <purohit.div@gmail.com>
Test Plan:
Click continouosly on combobox and events on hover would be
disabled.

Reviewers: cedric

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

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

combobox: fix disabled state

Summary:
Apply gray-scale to vertical_separated_bar_glow image and increase the opacity
of the button, so that the text on the entry is visible.
@fix T3075
Signed-off-by: divyesh purohit <div.purohit@samsung.com>
Test Plan: please run combobox example from elementary_test

Reviewers: shilpasingh, cedric, raster

Maniphest Tasks: T3075

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

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

Fix toogle icon sizing issue.

Strangely, toggle couldn't afford it's icon space
at elementary_test -> check toggle.

Following to default style which is normal to avoid this wierd look.

Elm Image: fix usage of animated gif on the same image object more than one time

This patch @fix this usage example:

0. image obj create
1. file_set(an animated gif)
2. animated_set(TRUE) + animated_play_set(TRUE) to make the gif play
3. file_set(a normal image)
... at this point elm_image is confused because it still think that the image is animated
4. file_set(an animated gif)
5. animated_set(TRUE) + animated_play_set(TRUE) to make the gif play
...this now do not work because elm think that the image is already animated

release: Update NEWS and bump version for 1.17.0-beta3 release

shelf: add "noshadow" data item to indicate no shadow should be used

elementary: Add missing EINA_UNUSED in combobox function

_elm_combobox_elm_genlist_filter_set function does not use the Eo *obj
parameter so mark it as EINA_UNUSED to remove compiler warning

Signed-off-by: Chris Michael <cpmichael@osg.samsung.com>

naviframe: Update documentation of item_pop_cb_set and item_pop_to.

elm_test: Show popup error when Plug can't connect

Genlist: do not select the item two times in case of a double click.

In the case of a genlist set as ELM_OBJECT_SELECT_MODE_ALWAYS the "selected" cb was called 2 times. This @fix that by skipping the selection code in case the click is not the first one (EVAS_BUTTON_NONE).

Also merge the conditions of two consecutive "if" in a single statement.

Genlist/grid: do not set again the focus on double/triple click

Focus is already setted on first mouse-up, no need to set again in double and triple click up events

@Fix the issue reported in previous commits

Reveal the same focus bug of previous commit also in genlist.

See previous commit for instructions on how to reproduce in "Genlist Focus" test.

Add a test to reveal a focus bug in Genlist/Gengrid

To reproduce:
 * elm_test -to "GenGrid Focus"
 * turn on the "Focus Highlight" checkbox
 * double-click an item to show a popup that should have the focus

As you can see the mouse-up event on the gengrid item steal the focus from the popup.
When, instead, the item is activated with the keyboard the popup correctly keep the focus.

I have this problem in both genlist and gengrid, so probably the error is not in the gengrid code, but somewhere else that I cannot find.

...any help appreciated

combobox: apply proper style

Summary:
Initially combobox used hover's default style, this patch would make combobox use hoversel_vertical style.
@Fix T3075 for 2nd point.

Signed-off-by: divyesh purohit <div.purohit@samsung.com>
Test Plan: please run combobox example from elementary_test

Reviewers: shilpasingh, cedric, raster

Subscribers: rajeshps, govi

Maniphest Tasks: T3075

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

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

combobox: "return" key handled correctly and removed unreachable code.

Summary:

_key_action_activate was triggered when "return" was pressed, _key_action_move
contained the "return" key handling, which is unreachable code.
@fix T3075 (1st point).

Signed-off-by: divyesh purohit <div.purohit@samsung.com>
Test Plan: please run combobox example in elementary_test

Reviewers: shilpasingh, cedric, raster

Subscribers: rajeshps, govi

Maniphest Tasks: T3075

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

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