elm - fix engine config mismatch with new perferred accel vs old engine

the elm config tool only lets you set accel preference. if you set
none, then the old engine config takes charge, but there is no way to
change/set this, so deprecate the old engine set and don't use it.
this accel preference has been around for a while, so use it now
instead.

elm_slider: modified each slider have its indicator visible mode.

Summary: Added indicator_visible_mode to each slider in order not to make elm_config controlls every sliders.

Reviewers: Hermet, woohyun, seoz

Subscribers: seoz

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

doc: Fix typos in documentations.

win_example: Fix win example codes.

- return value in ecore event handler callback.
- remove unnecessary variable.

win: fix an issue that profile is erased and set to NULL.

Summary:
When ecore_evas_window_profile_supported_get API return EINA_FALSE,
elm_win widget should not get profile from ecore_evas_window_profile_get API.
@fix

Reviewers: woohyun, Hermet, seoz

Reviewed By: seoz

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

fix win center trap for elm to pass in h and v params

while this is technically an abi break, it should actually have no
real impact. it will have warnings - sure, but running existing code
will simply not see the extra parammeters on the stack and not use
them so it'll continue to work. if we removed paramms, we'd be in
pain. (same applies to params passed in registers).

this fixes T2031

elm - win - dont use deprecated func - use new one.

cleanup to match efl 1.13 changes

elm_label: Fixed label disappearing after theme changed

Summary:
There is a bug that text disappears when theme is changed after label shown.
This patch fixed it.

@fix

Reviewers: woohyun, Hermet, Jaehyun, seoz

Subscribers: Jaehyun, seoz

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

[tooltip] use proper object to set tooltip layer

Summary:
tooltip could be hide by the other object which have lower layer than target object of tooltip.
the layer of content could be changed, when is set to the tooltip as a content.

@fix

Test Plan: check added test lines

Reviewers: cedric, jpeg, seoz, raster

Reviewed By: raster

Subscribers: singh.amitesh, stefan_schmidt, seoz

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

elm_label: Fixed to prevent slide's autostart before calling elm_label_slide_go()

Summary:
Label should not start sliding before elm_label_slide_go() is called.
However, label starts sliding automatically,
because resize callback emits slide start signal.
This patch prevents label from sliding before calling elm_label_slide_go().

@fix

Reviewers: woohyun, Hermet, seoz

Subscribers: CHAN, woohyun

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

datetime_module: Fix build warning.

Include the elm_widget.h

update po

datetime module: include elm_widget_datetime.h instead of elm_priv.h

The external module can't include elm_priv.h.
So Elm_Datetime_Module_Data moves in elm_widget_datetime.h

NEWS: Pedantic change of capital letter in NEWS fi.e

test_label: Cleanup label wrap test.

- remove unnecessary box.
- use ELM_SCALE_SIZE macro for a better scalability.

Gengrid : Fix memory leak in elm_gengrid_realized_items_update

Summary:
Eina_List from elm_gengrid_realized_items_get must be freed by caller,
         but elm_gengrid_realized_items_update doesn't free Eina_List.
         So memory leak is happens.
@fix

Test Plan: Call elm_gengrid_realized_items_update repeatly and check memory share increase.

Reviewers: raster, seoz

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

Updating serbian translation

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

elm_image: fix elm_image_memfile_set API fail when elm_image widget is not visible.

Summary:
It could success only when the widget was visible.
The API must return EINA_TRUE if buffer and size has no problem, not visiblity of widget.
@fix

Reviewers: Hermet, woohyun, jpeg

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

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

list/genlist/gengrid/toolbar: set focus/unfocus on item when focus is actually set on widget.

Summary: @Fix

Reviewers: raster, Hermet, seoz

Reviewed By: seoz

Subscribers: seoz

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

multibuttonentry: Delete duplicate running code.

Summary:
When layout clicked, "elm_widget_on_focus()" called with "entry_focus_in()" callback.
The "view update" and "focus set to entry" already call in those callbacks, so i deleted duplicate code and it also make focus issue when user click the entry to add item.
@fix

Test Plan:
Run elementary_test.

1. excute multibuttonentry.
2. click the entry area.
3. see the elm_widget_on_focus called twice.
   (layout callback function called view update -> elm_box_unpack called in view update ->
    entry lose focus -> on_focus_callback call again)

Reviewers: Hermet, seoz, Jaehyun_Cho, JackDanielZ

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

win: Add default return value at elm_win_type_get API according to API description.

Summary: The API must return ELM_WIN_UNKNOWN (-1) when the given object is improper.

Test Plan: None.

Reviewers: woohyun, JackDanielZ, seoz

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

test_label: Use meaningful test name for label test.

Elm_Entry: Removed empty line and added error message

Summary:
Resolved FIXME in utf8_markup_save by adding error messages
for fopen and fputs command

Signed-off-by: kabeer khan <kabeer.khan@samsung.com>
Reviewers: devilhorns, seoz

Reviewed By: seoz

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

NEWS: Clean up news file before release.

test_panes: Remove unnecessary align/weight set for panes contents.

edje_externals/elm_video: Fix bug where we override the rememberred position

This bug was there from the initial commit of elm_video. Classical
copy&andpaste bug.

CID: 1267207

panes: Fix elm_panes_fixed_set.

@fix

NEWS: Manually fix up a news entry

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

Icon: choose the fdo icon size using the evas obj geometry

Use the min between w and h of the evas object to
request the size in the icon theme. Fdo icon are always
square (1:1) so min seems correct.

Also improved the test to include a test that do
not relay on the size_hint_min of the icon.

Now in all the test I have done always the correct
fdo icon are used.

Revert "Icon: use size_hint_min instead of image_size to know the wanted size"

This reverts commit 75176a789981bd6b1ef0bfcc95793c896b5bde07.

Was not the correct solution, as min_size can be smaller than
the real wanted size. Need to find another solution.

Thanks TAsn to spot this

Icon: use size_hint_min instead of image_size to know the wanted size

This @fix fdo icons to load the correct image for the requested size.

Whitout this the small images (16x16) was never used, because once a big
(64x64 for example) image was loaded, that size was used as min.

See the Icon Standard test for a better explanation

Icon: no more reimplement sizing_eval

The code there was trying to reload the icon to have an
image that should match the requested size, this was totally
wrong, the calculation was always resulting in an image bigger
than the needed.

It was also useless as this logic is yet handled in the resize
callback.

Improve Standard Icon test

You can now select the size of the icons in the list and
also added a single icon that you can resize.

The size of the icons is important as icon themes usually
provide different images for different sizes.

This test now spot out some errors in elm_icon wrt icon
sizing, will fix soon

Add correct fdo name for the last added search icon

elm_image: Add missing semi-colon to terminate line.

No compiler warning for this because the macro was terminated with it.
It still looks wrong without it and does no harm.

elm_image: Fix memory leak in GETDAT macro.

Free data2 on error path.

CID: 1265609, 1265610, 1265611, 1265612

test_disable: Extend button array to fix memory corruption

We are accessing btn[3] in the code so we should make sure the array it big
enough for it.

CID: 1265613

transit: Fix read from pointer after free.

This fixes coverity CID 1265738.

test_win_state: Add elm_win_center() sample.

win centering: unbork!

not centering on an axis does not mean that it needs to be moved to the screen origin.

also clamp to screen x,y, not 0,0

transit: fix crash issue.

This fixes the side effect that introduced from 1e06309be42fd2b5d0900b16da0178215c111681

Since transit has supported adding a map effect on the previous map status,
transit tried to access object-state map pointer.

but theses map pointer won't be valid if the transit keeps the last state
because it doesn't back up any object-state data.

test_gengrid: Dynamically allocate item data.

Do not waste unnecessary memory.

test_colorselector: Clean up a little bit.

- Remove unnecessary data set.
- Use ELM_SCALE_SIZE on min set for a better scalability.

Yes this colorselector test needs a looooot of love.

colorselector: Remove unnecessary pointer redirection.

Elm_Colorselector_Data pointer is already there.

test_icon: Clean up icon standard sample.

- Do not set unnecessary min size for the icons. Just follow list's item
size naturally.
- Do not free unnecessary list which was already freed by
EINA_LIST_FREE.
- Do not set align for the window resize object which is not required to
be set.
- Follow conventional way of pointer declaration.

image now implements edje object size functions

      Edje.Object.size_min.get;
      Edje.Object.size_max.get;
      Edje.Object.size_min_calc;
      Edje.Object.calc_force;

fixes all image-in-edje sizing issues/errors

image animation functions now work as expected for edje images

win_example now exits fullscreen on any key press...always

fix T516

Elm_Entry: set handler style according to current widget style

Previously, entry set handlers' style as default when they are created
and changed style on theme_apply function. But in usual scenario, users create
entry and set style before handlers are created, so when they are created
the style is set as default regardless of the style users already set.
This patch fixes it.

@fix

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

Revert "genlist: added reorder feature on key events"

This reverts commit f99be3a1d6bdc63153598b592b5437c33da44bf0.

The API is not ready to be released. Please see the thread on the e-devel
mailing list under this subject. This can get another try in the 1.14
release cycle.

Conflicts:
src/lib/elm_genlist.c

genlist: fix incorrect grammar.

No allow attaching double definite articles.

Thanks vincent for reporting.

elm_transit: Fix crash issue caused by multiple transit effects.

Summary:
Fix crach issue caused by multiple transit effects on a single object.
         This patch prevents deleting obj_data while the obj_data is still used by other transit objects.
         This fix is related to commits "Support zoom keeping current map effect."(60214e6a172e42303fdfc69ef3d1996b25132242)
         and "Support rotation keeping current map effect."(1e06309be42fd2b5d0900b16da0178215c111681).
         @fix

Reviewers: Hermet

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

genlist: fix genlist multi-selected problem.

user request was ignored if they calls the genlist_item_selected_set(true)
to another item in an item selected callback. because genlist made
the selected item highlighted after user "selected" callback.

The behavior had been changed on my genlist refactoring,
so the behavior was actually just restored.

This issue resolves T2020

elm_transit: Fix to support zoom rate zero.

Summary:
Fix to support zoom rate zero by using evas_map_util_zoom() instead of evas_map_util_3d_perspective().
@fix

Reviewers: Hermet

Reviewed By: Hermet

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

widget: elm_win can be unfocused only when its ecore_evas loses focus.

configure: Fix typo in coverage option help text

tests: add coverage support.

And our current score is 19% functions coverage for Elementary.

tests: actually run the main loop properly to be able to enter in idle state.

Note that the iterate function will never trigger the idler that are registered
in Ecore. That's by definition. I changed the code to actually use the full
main loop and trigger the change detection on idle enterer. That should be enough
for Elementary as all idler should logically affect the visual aspect of something
at some point and exit idle.

Thanks marcel-hollerbach@t-online.de for helping me debug this issue.

Updating italian translation

genlist: code clean up

highlight item with one entry.

genlist: don't access item after removing it.

genlist: guard the item deletion problem.

this exceptional code prevents the deletion of the item
in the middle of the callback.

genlist: fix the access of the invalid items.

still, there were case item or genlist is cleared in the user callback.
don't access the cleared items if they are requested deletion.

genlist needs to also highlight items on keyboard select

this should probably just happen in _item_select() to avoid future incidents

ref T1986

Revert "genlist: fix a crash when deleting a item in selected callback."

This reverts commit c80b24baf815e35c160bf0dbe8ecae6ea247aef8.

this broke the selection callback completely and makes every genlist app unusable.

please, please, please, please, I'm seriously begging you hermet, do ANY amount of testing before making further genlist commits.

genlist: fix a crash when deleting a item in selected callback.

if user deletes the item in the callback,
current function(_item_select) would be corrupted.
just leave the callback call in the last of the function.

genilst: fix regression bug added in 0e7d521151ba93f3b5955caaf9b906e160ab6057

some logic of deleting item were duplicated.
this fixes fileselector crash issue.

Do not explicite include atspi headers.

Summary: fixes make check failures on atspi tests.

Reviewers: cedric

Subscribers: seoz

Maniphest Tasks: T2001

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

genlist: set missed item reference.

this missing would cause crash of the genlist when it's items are cleared.

genlist: fix a crash of genlist group tree example.

don't corrupt list pointer.
we can clear items more simply.

Thanks ya-kov for reporting.

widget: set missing elm_widget_item_magic

photocam: force reload of image on file_set.

Thanks, bug reported by thierry@substanciel.fr .

elm image - fix image rotation in non-common cases

this fixes d6f4040d0c62c220f2339d2a57126fde43e85e19 for the non-common
path that normalizes then applies new orientation

@fix

elementary: Fix dereference of null return value

Summary: This fixes Coverity CID1261370. strrchr can return NULL here,
so we need to check that 'p' is valid.

@fix

Signed-off-by: Chris Michael <cp.michael@samsung.com>

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

Elm win: Fix added internal parameter to the win constructor.

Internal hacks should not affect user facing API. This commit adds a new
API to do the internal hack. This is still bad, but it's much less than
what was there before.

This fixes the main issue introduced by commit:
63a5cdf0f8516c16a62233da0c2af0e10a13264b

Elm win: Fix elm_win_fake()'s name to follow conventions.

DnD: fix callbacks deletion for inline windows.

During deletion of a window, widgets considered as droppable targets
have to remove their DnD callbacks. To achieve this, elm_drop_target_del
is called from the DEL callback (destructor). This function has to
determine if X11 or Wayland is used. Since the parent is already unknown
at this stage, only checking the engine name can give this
information.
On a regular window, the engine name is related to the target display.

The problem happens when an inline window is used. The engine is a
buffer and no information is given regarding the target display.

The patch fixes it by checking the nature of the Ecore Evas parents. It
supports nested windows (inline inside inline... inside XWin).

@fix

elm_colorselector: Fixed to make palette_box have proper height

Summary:
palette_box had stretched height exceeding its items size.
Now it will have proper height fitting to own items.

Reviewers: Hermet, Jaehyun, raster

Reviewed By: raster

Subscribers: raster, Hermet

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

elm - fix build problem with examples - elocation api

this fixes build of location_example_01.c as per fix in D1863 but done
properly so people can't use elm  elocation unless they enable beta apis

Scroller : Fix elm_scroller_last_page_get API wrong return value

Summary:
When first page is last page, elm_scroller_last_page_get need to return 0.
         But if pagesize is bigger then current content height or width, value will be returned -1.
         So fix this cases to return 0.

Test Plan: If set elm_scroller_page_set bigger then current content size, then issue is occurred.

Reviewers: seoz, jaehwan, raster

Reviewed By: raster

Subscribers: eagleeye

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

elm - elm image orientation api fix - major bug in implementation

@fix

The elm image orient_set api looks like it is a state, and it is
documented to SET the orietnation, but the code actually DOEs that
orientation when you set it. so docs and api name didnt match
functionality at all. this was bad an an oversight. you didn't notice
until you set orientation multiple times to an image. this fixes the
implementation to be as documented - a state that is applied and
remains that way until set to something else. setting to the same
orientation has no efect as with color, size, etc. etc. in evas.

you could look on this as an api break, but then either the docs
change AND api name changes to match the code, or code changes to
match the docs. we don't keep bug compatibility in efl and in this
case i'm making the call to break the behavior of a  function as it
was broken already with respect to name and docs.

genlist: recommit 1879cc39258c63c24f0abcff56558b1f1b60167a with correction.

elm_table now supports alignment

fake win should not perform widget focus setting on canvas focus-in

Revert "genlist: use eina bool"

  This reverts commit 1879cc39258c63c24f0abcff56558b1f1b60167a.

  Hermet, please check this commit it makes Fileselector crash,
  looks like you inverted the values: 1 should be EINA_TRUE.
  I recommend you to use Exactness to run all the tests automatically.

genlist: remove unnecessary null check.

genlist: revert 6f177e5db01553276058235a81cbfcc9b2cd4e23

this broke decoration item text. reverted.

genlist: send decoration active signal to view item.

emit signal to view item to do something additional.

genlist: fix regression bug. (T1986)

newly added in 2955ebc5dd01fbfb5a3286b724c8a40915864ad8

ELM_WIN_FAKE types should allow focus callbacks on windows

entry now finds its window object more effectively for cnp init

+elm_win_fake(), ELM_WIN_FAKE type

this is a giant hack to provide widget parenting in e. do not use this function, do not use this type, do not even read the related documentation.

@feature

genlist: code refactoring.

clean up decorate_item_realize() function.

use macro and remove old debug code.
no actual logical change.

genlist: clean up genlist_smart_del() func.

removed unnecessary calls.

widget: remove redundant internal macros

we don't need these macros anymore since eo has introduced.

Do not call ecore_x_init() when not using X display.

Summary:
On some systems (e.g. OS X), both Apple's windows manager and X11 can cohabit,
therefore we need to check for the DISPLAY environment variable only when ELM_DISPLAY
is not specified.

Reviewers: cedric, devilhorns, seoz, raster

Reviewed By: raster

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