Automatic migration to the new eo_add syntax.

Combobox: Semi automatic migration to the new eo_add.

theme: Fix scroller colorclass typo.

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

This reverts commit 9f4c43c20dfa36e7a8be18278acf4336c13574d7.

I'm sorry but this causes a side effect(list sizing issue) at enventor.
And I couldn't find any mis-usage in enventor side.

We can't not accept this patch unless we figure the exact reason out.

elm_genlist: Added elm_genlist_filtered_items_count API

Summary: This clarifies the documentation of the new api.

Reviewers: SanghyeonLee, shilpasingh, cedric

Reviewed By: cedric

Subscribers: bu5hm4n, buds

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

fileselector: clarify the condition statement for checking ok button

Summary:
elm_fileselector_buttons_ok_cancel_set() API checks the existence of
ok button before adding it.
This patch makes condition statement more intuitive.

Reviewers: cedric, bu5hm4n

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

remove duplicated typedef

this is already defined in elm_layout.eo.h.

atspi intf: image - fix compilation warnings after eo4 migration changes

                     ======
  CC       libelementary_la-elm_interface_atspi_value.lo
  elm_interface_atspi_image.c: In function '_elm_interface_atspi_image_description_set':
  elm_interface_atspi_image.c:36:9: warning: 'old_descr' may be used uninitialized in this function [-Wmaybe-uninitialized]
           eina_stringshare_del(old_descr);
                    ^
                    ======

genlist: fix item sizing error

Summary:
Sometimes genlist expands its items' width over viewport,
so horizontal scrollbar is shown unnecessarily.
This will correct item sizing calculation.

@fix

Test Plan:
elementary_test -to "genlist tree"
elementary_test -to "fileselector"

Reviewers: SanghyeonLee, cedric

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

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

map: fix memory leak

Summary:
- in dump_cb buf is allocated by malloc() ->strdup() -> free.
  This is needless.
- And buf allocated in dump_cb is strdup again.
  This may cause memory leak.

Reviewers: Hermet, woohyun, cedric

Reviewed By: cedric

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

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

config: handle is_mirrored, and translate are not part of EET file, when config flush occurs

Summary: is_mirrored, and translate value is reset, when config flush occurs.

Reviewers: raster, cedric, jpeg, tasn

Subscribers: seoz

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

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

image: fix the wrong return value

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

Reviewers: Hermet, cedric

Reviewed By: cedric

Subscribers: seoz

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

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

Genlist: fix for an item width calculation issue on item show / scrolling

Summary:
When item_show / scrolling is happening, _item_block_position will be
called from smart_calculate where item width is set as block width.
If items are not realized _item_realize will be called and if items are
getting realized for first time we set mincalcd as false. So later
item width gets overwritten by min width of item and so issue is happening.

@fix T2603

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

Reviewers: prince.dubey, shilpasingh, SanghyeonLee, cedric

Subscribers: rajeshps, seoz, govi

Maniphest Tasks: T2603

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

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

notify: delete unnecessary smart move/resize.

Summary:
elm_notify's size is dependent on parent's size.
So _elm_notify_evas_object_smart_move/resize() are not necessary.

Reviewers: woohyun, jypark, Hermet, minkyu, cedric

Reviewed By: cedric

Subscribers: minkyu

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

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

genlist: add elm_genlist_filtered_items_count API

Summary: This API will return the number of items that have passed the filter at a given point of time. If the API is called before "filter,done" callback comes, it may or may not give the filtered count for the complete genlist.

Test Plan: Updated Genlist_filter example to print the output from filtered_count_get API in "filter,done" callback.

Reviewers: SanghyeonLee, shilpasingh, cedric

Reviewed By: cedric

Subscribers: buds

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

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

fileselector: refine code and theme

Summary:
refine code and theme, hopefully solve the issue
that the height of file list (genlist) is calculated wrong.

Test Plan: elementary_test -to fileselector

Reviewers: SanghyeonLee, jpeg, cedric

Subscribers: taxi2se

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

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

win: add check for focus object before adding new one

Summary:
Focus object is added repeatedly, when focus is enabled.
This patch checks existing focus object.

Test Plan:
elementary_test -to focus
check/uncheck "Focus Highlight Enable" with spacebar

Reviewers: Jaehyun_Cho, jpeg, cedric

Reviewed By: cedric

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

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

fileselector: add backspace key binding to move to previous path.

Summary: Add backspace key to move to previous path.

Test Plan:
remove /home/username/.elementary (to apply new elementary config file)
elementary_test -to fileselector

Reviewers: cedric, jpeg

Subscribers: bu5hm4n

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

multibuttonentry: add elm.swallow.background part

Summary: This swallow part is for custom background image.

Reviewers: kimcinoo, Hermet, cedric

Reviewed By: cedric

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

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

theme: add new time gadget themes

theme: move digit* images to O/

theme: add new wireless gadget theme

start: add alias for new gadget group

theme: add bryce edcs

combobox: store eo_constructor() return value into obj

This is what we do in every widget's base constructor.

@fix

elm_access: remove duplicated line, ELM_SAFE_FREE handles NULL assign

elm_access: mapi have to be set NULL after free(m->api)

Summary: mapi have to be set NULL after free(m->api)

Reviewers: kimcinoo

Reviewed By: kimcinoo

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

hoversel: remove EINA_UNUSED since data is getting used

Genlist test: Adjust according to the recent eo event changes.

Thanks to zmike for letting me know.

Migrate elementary to the new Eo4 syntax

Eo's syntax has changed (see efl.git for more info), this adjusts
elementary accordingly.

This is a merge commit that merges my branch will all of the fixes

Fix warnings following migration to Eo4.

Mostly unused variables.

Map: Correct broken migration.

The script failed migrating some code, so I needed to fix it manually.

Automatic migration to Eo4.

I just ran my script (email to follow) to migrate all of the EFL
automatically. This commit is *only* the automatic conversion, so it can
be easily reverted and re-run.

App client/server and entry: Mix of automatic migration and manual adjustments.

The script couldn't handle everything correctly on its own.

Manually migrate some eo_do for the migration script

The script doesn't handle a few corner cases well. Update the code
manually so it doesn't have to deal with those.

Remove redundant defines.

Scaling test: reorder instructions to set the correct scale

If the scale is set on an object before contents are set, it will not
pass to them. Because of this, in the test, scale of the first label
remains 1.0, i.e the window scale, instead of 0.5.
The patch modifies the order of the instructions by setting the scale
after setting the label as content of the frame.

radio: inherit from elm check

Summary: Radio inherits from Check now and remove the duplicate codes

Reviewers: yashu21985, cedric, tasn, raster

Reviewed By: raster

Subscribers: seoz

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

elm_cursor: don't include Ecore_Cocoa_Cursor.h (deleted in EFL)

Spinner: Swap abs with fabs because we're dealing with doubles.

build: enhance generated html output from coverage report

We do not handle branch coverage so remove that part of the output. On the
other hand at a legend to allow people to understand the color meanings.

build: fix problems with coverage check build where some files are not found

Switch to use a lcov config file which geninfo_auto_base and remove hard coding
the base dir  to src/lib. geninfo_auto_base is designed for a use case like
ours where we have several base dirs (lib, bin, tests, ...) and it detects them
automatically. This fixes failures in a coverage run like this:

genhtml: ERROR: cannot read
/srv/build/workspace/manual_elm_gcc_x86_64_exactness_coverage/src/lib/test_win_state.c

Where the file is simply looked for in the wrong directory.

Fix example with new event api.

Summary: Fix build after changes.

Reviewers: felipealmeida, cedric, tasn

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

Elm_Hoversel: import elm_icon

The icon type is needed by item_add

elm map: silence uninit var warnings that new gcc versions show

gcc thinks the vars may in theory be uniitiialized. it's right but it
won't actually happen. but having noise in warning output is bad, so
fix the warnings so we can focus on the real bugs/issues/warnings

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