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>

combobox: move responsibility to define filter and content to the user of a combobox

Before this patch, the filter was set inside the combobox. This is confusing for the
user of the object as it can't define a custom filter except on strings. It is also
impossible to bind as binding require to know what was put in and they can't differentiate
between a genlist filter being set from the binding to one being set internally by
the combobox.

@fix

genlist : fix tree item floating issue when homoegeneous mode is enabled

genlist item is flaoting upon the expanded item when homoegeneous mode
is enabled.
when item block is recalculated by tree expanding, new item didn't
update their properties and values becuase of size hash, but edc
actaully need to be recaculated after swallowing tree size padding.

solve the issue by adding expand depth check in size hash.

you can check the previous issue in T2708
elementary_test->genlist tree->homoegeneous check

@fix

Adjust "context_menu_disabled" configuration.

This @fix three (yes 3!) errors from commit cafda5928f9bf8116564219dc03d12c3a6e131b3

1. the initial value in the standard config is wrong, causing ctx menu to be disabled for everyone!
2. the argument of the api should be named "disabled" (like all the other disable API in this context)
3. (minor) fixed doc for elm_config_context_menu_disabled_set

NOTE: there are no configuration gui for this option, if you cleared your elm config recently you probably now have the ctx menu disabled everywhere, only solution is to wipe your config (to let elm create a new correct one)

Big SPANK to the author, the commiter and all the reviewers of D2463!!!

elm: Replace EO_EV_ by EO_BASE_EVENT_

EO_EV_ stuff is deprecated

elm_hoversel: fix calculation to make items fit to dropdown box

Summary:
While applying scroller on hoversel, several lines for item size
calculation were added, but they failed to have item get proper size.
Now redundant codes are removed and some are fixed, and hoversel will
be shown more well-ordered.

Test Plan: elementary_test hoversel

Reviewers: jpeg, raster, cedric

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

elm_widget: use internal function for clearing the focus

Summary:
This patch will remove duplicate logic

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

Subscribers: seoz

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

update author

elm_widget: cleanup _elm_widget_on_focus function

Summary:
This patch will reduce duplicate codes and code depth for readability

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

Subscribers: seoz

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

Combobox: rename the "selected" event to "item,selected" as it should be.

Also updated the test to reflect this change

elementary: Fix elm_win_window_id_get function

When using this function with wayland windows, we should be returning
the matching Ecore_Wl2_Window id, not the surface_id

ref T3058

@fix

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

Hoversel: add a test for the populate-on-click scenario

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

win32: Fix link of binaries which use functions in Evil

Reviewers: cedric, jpeg

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

gitignore: Add new example to gitignore list.

examples/combobox: Remove unused functions.

Gengrid: improve item_region_show code

There is no need to recalc row and col position of the item, they are already stored in the item struct.

The old implementation was calculating wrong values and also was storing this wrong values in it->x and it->y, resulting in wrong results for the elm_gengrid_item_pos_get() function.

so, at the end,  this is a @fix for the pos_get() function.

Also changed a bit the test to let the gengrid fill the window and thus testing behaviour on col/row changes.

I didn't find any regression in all the gengrid tests after this, let me know if it broke something for you.

config: Correct defaults in upgrade of popup conf

After bumping version we need to create a new IFCFG block.

@fix

T3008

slider: fix segfault in case of obj is deleted before wheel timeout timer expires

Issue:
1. elementary_test -> slider
2. move mousewheel on slider and popup gets shown.
3. close the slider window very fast or increase the wheel_timeout to 4 seconds
4. observe crash

SegmentControl: Fix theme to avoid text move on select

GLOW and SHADOW_BOTTOM have different paddings, which changes the
geometry of the object. This compensates that.

config: try to properly update config for popup new align value.

This idea has been suggested by Andrew Williams and I hope it does work
just fine. Please report any problem.

@fix

T3008

elm_config: close file before moving it

elm_config: print an eror message if we fail to save the derived config profile

elm_config: handle eet_data_write possible error return

CID: 1347399

examples: fix compilation from command line (file models)

box: enforce size recalc during unpack_all()

deleting subobjects here was occurring and triggering sizing eval
prior to removing the objects from the evas box, resulting in a failure
to accurately calculate and set size hints

also set delete_me to block sizing evals during subobj deletion loop
in order to avoid object thrashing during the time when nothing is happening

@fix

box: improve aspected layout for non-1:1 ratio aspects

the aspect calc function requires unaliased size params, so don't pass
aliased params

also clamp min axis size to min aspected item size as intended

[Hover] Fixed the delaying of hover dismiss on continuous mouse clicks

Summary:
When continous mouse clicks happen hover emits the hide signals continously
to edje. As the embryo program for hiding requires some time to finish the
animation, if within that time another mouse click comes the program gets
invoked again and thus the hiding will get delayed till the mouse clicks ends.
This patch fix that issue by adding a flag for emitting signals.

@fix T3006

Signed-off-by: godly.talias <godly.talias@samsung.com>
Test Plan: elementary_test

Reviewers: cedric, conr2d, prince.dubey, shilpasingh, Princekrdubey

Reviewed By: shilpasingh, Princekrdubey

Subscribers: Princekrdubey, divyesh, govi, rajeshps

Maniphest Tasks: T3006

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

els_cursor: mark internal struct static

elmn_config: mark some internal variables and functions static

atspi: mark some internal structs static

examples/combobox: include string.h for strcasestr()

Widgets: remove duplicate declaration of language,changed signal.

They are already defined in Elm.Widget, and they inherit from it.

Fixes T3208.

Widgets: remove duplicate declaration of access,changed signal.

They are already defined in Elm.Widget, and they inherit from it.

Fixes T3208.

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

modules/prefs/elm_entry: Add safety check for NULL pointer and fix memory leak

Summary: Add safety check for NULL pointer and fix memory leak

Reviewers: cedric, woohyun, Hermet, stefan_schmidt, raster

Reviewed By: stefan_schmidt, raster

Subscribers: Hermet

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

elm entry: check for null return from eina_rectangle_new

this will handle if eina_rectangle_new can't allocate a rect
(basically never will happen...)

@fix

js: fix examples functions

Summary: update functions in examples @fix

Reviewers: cedric, felipealmeida

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

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

combobox: fix recalc and hover's best_location error

Summary:
1) Recalc error occurs if combobox_hover_begin() was called by the app, it can be reproduced by keeping elm_combobox_hover_begin() in the test_combobox.c
2) Content_set in hover always considered "bottom" as it's best_location as it done during the constructor.
@fix
Signed-off-by: divyesh purohit <div.purohit@samsung.com>
Test Plan:
It can be reproduced by keeping elm_combobox_hover_begin() in the test_combobox.c.
Also the changing the position of combobox in the window would give top/bottom as it's best location.

Reviewers: cedric, shilpasingh

Reviewed By: shilpasingh

Subscribers: rajeshps, govi

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

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

box: update evas box geometry during layout after smart extents calc

smart extents may trigger a recalc during size hint setting, resulting
in the box being moved/resized during layout

hoversel: don't activate if the items were none.

it's wierd to expand hoversel without items.
the behavior made me suprised before i analyzed it.
it looked almost a bug.

if you have any other objections, please ping me.

hide indicator after mouse wheel activation.

we need to customize the duration of the indicator from api or theme data.

@fix T2348

ctxpopup: add geometry,update smart callback.

conceptually, ctxpopup won't give any geometry information of the container.
this event is a way for returning the information of the container for user scenarios.

@feature.

fix @T2042

elementary: Fix potential sizeof mismatch

Coverity CID1347398 reports that this could potentially be a
non-portable assumption resulting in a sizeof mismatch. Fix this by
using the actual structure type in the call to calloc.

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

elementary: Fix warnings of uninitialized variable usage

This patch just fixes a compiler warning about using fw/fh
uninitialized if the conditional is false

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

elementary: Fix calls to ecore_evas_wayland_window_get2

As we've renamed the function, the calls to it should be updated to
use the new function name

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

widget: check next_item existence when focusing before calling widget fns

ERR<eo>lib/eo/eo.c:753 Unable to resolve op for api func 0x7ffff7cc17ce

layout: do not unset max size hint during sizing eval

previously-applied max size hints should not be unset; these are only
able to be set externally from a parent object, so don't overwrite them
internally

@fix

box: use elm_box_recalculate() in smart recalc

ensures that the internal box will calculate and blocks further (recursive)
recalc calls

elementary_test: remove wrong usage of EINA_UNUSED.

Summary:
If argument variable is not used in function, then only
that variable should be declared as EINA_UNUSED.

Signed-off-by: Umesh Tanwar <umesh.tanwar@samsung.com>
@fix

Reviewers: Hermet, singh.amitesh

Subscribers: sachin.dev

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

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

segment_control: check item disable

Summary:
Elm segment_control: check item disable

Signed-off by: @adrien

Refer T2883

@fix

Test Plan: NA

Reviewers: cedric

Subscribers: adrien

Projects: #elementary

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

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

combobox: add documentation and example

Summary:
Added Documentation for Combobox Widget
Signed-off-by: divyesh purohit <div.purohit@samsung.com>
Test Plan: execute make doc

Reviewers: shilpasingh, cedric

Subscribers: rajeshps, govi

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

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

elm_sys_notify: add missing since tags for notification server support

elm_notify/popup: add missing since tag for new dismiss symbols

elm_combox: add missing since tags for new APIs

elm_win: add missing since tag and correct another one

toolbar: Revert a theme change for shrink mode signals.

Summary:
It was changed for testing shrink mode signals.
But, it was unnecessary change for default theme.

Test Plan: N/A

Reviewers: zmike, jaehwan

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

config: Remove stray printf.

sys_notify: fix shutdown of elm_sys_notify

Summary:
When elm_sys_notify was not needed, its singleton instance
is NULL. Eo complained about passing a NULL pointer.

Thanks to bu5hm4n (Marcel Hollerbach) for the report.

@fix

Reviewers: stefan_schmidt, cedric

Subscribers: bu5hm4n

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

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

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

elementary: Fix elementary not compiling

With the push of 1056b6ed01475817da2439012057d11865d2c5d8, elementary
compile broke due to undeclared variables. This patch fixes elm to
compile again by declaring the missing vars.

@fix

@raster, Please double check this change as I am unsure if it is what
you Really intended here.

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

elm config - monitor JUST the correct files for profile and cfg change

this fixes the config monitoring to not use flush.cfg but to use the
correct profie.cfg or base.cfg in the profile we are using. this is
necessary to do "special" profiles per screen so apps that switched to
another profile because asked by e, monitor THAT profile for changes
specifically and dont also switch profiles since they have been told
what to use now.

@fix

elm wl cnp - fix paste to not crash anymore

this used the wrong widget - it should have used requestwidget. this
fixes that.

@fix

elm config - add the ability to have derived configurations from others

this adds the ability to have one profile automatically be derived
from another with modifications applied. right now it only can do a
very limited modification - multiple scale. over time this will
expand. this is only inteded otbe sued by a DE like enlightenment, so
it's not going to be documented fully at this point.

@feature

elm config - add function to check if profile exists

this adds a method to see if a profile exists.
elm_config_profile_exists().

@feature

elm config tool - dont need to call save if flush is saving anyway

tis would also make behavior more consistent with the tool as before
sometimes it flushed + saved, sometimes just flushed which would lose
settings. now since this works right in elm config in the lib, we dont
need to save if we flush as flush saves too.

elm config - allow the ability to save out to another hidden profile

support the ability to have hidden profiles that begin with a dot (.)
char like all hidden files/dirs. to do this we need to also be able to
create profiles with a given name (a hidden name) without switching to
them, so add the ability to save out to a specific profile name
without switching to it. of course this means to list profiles we need
to list them or list ALL includiing hidden profiles.

@feature

combobox: add new widget.

Summary:
Combobox is a combinational widget of a drop-down list and single line entry.
Based on the text entered in the entry, the list items are filtered accordingly.
Signed-Off By: Cedric Bail <cedric.bail@free.fr>
Signed-Off By: Divyesh Purohit <div.purohit@samsung.com>

Test Plan: test_combobox.c is added to elementary test

Reviewers: raster, shilpasingh, cedric, jpeg, stefan_schmidt

Reviewed By: raster, shilpasingh, cedric

Subscribers: SanghyeonLee, shashank0990, singh.amitesh, tasn, raster, seoz, poornima.srinivasan, rajeshps, govi

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

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

hover: fix segfault when smart direction is used in content_set in hover

Summary:
when using elm_object_part_content_set(hover, "smart", content); it would cause segfault, this patch would fix T3004

Signed-off-by: divyesh purohit <div.purohit@samsung.com>
Test Plan:
please open the hover3 example in elementary for testing it, the hover
content shouldn't leave the window as "smart" direction is used.

Reviewers: shilpasingh, raster, cedric, Hermet

Subscribers: rajeshps, govi

Maniphest Tasks: T3004

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

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

hoversel: auto update hoversel after dismiss animation finished.

Summary:
If auto_update feature changes hoversel width,
dismiss animation seems awful because hoversel contents move to
changed position in an instant.
This patch makes hoversel update its content with clicked item
after dismiss animation finished.

Test Plan: elementary_test hoversel

Reviewers: raster, cedric, jpeg

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

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

hoversel: fix hover position in hoversel, when it goes out of the viewport

Summary:
Hoversel Widget is designed in such a way that it accepts top/bottom/left/right directions , internally it uses hover widget which does not take care if its content goes of it's parent (Elm_Win in most cases) . this patch ensures that the drop-down content remains restricted in visible hover_parent. it would fix  T2965
Signed-off-by: divyesh purohit <div.purohit@samsung.com>
Signed-off-by: Jee-Yong Um <conr2d@gmail.com>
Test Plan:
Open entry widget and right click to show a hoversel widget, check if the hoversel or hover content goes out of the window.

this is what hoversel would look like after this patch {F27551}

Screenshots of top-left / top-right/ bottom-left and bottom-right corners in entry after this patch

{F27553}

{F27554}

{F27555}

{F27557}

Reviewers: shilpasingh, conr2d, raster, cedric, Hermet

Subscribers: rajeshps, govi

Projects: #elementary

Maniphest Tasks: T2965

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

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