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

client frame client clip should be relative to client during clip, not border

ref T1930

genlist: if the item was selected, then skip item_select() procedure.

genlist: simple code clean up

no logic change.

genlist: use eina bool

genilst: descriptive comment++

genlist: remove internal callback usage.

there were multiple ports for item procedures.
one port is enough.

genilst: get rid of lazy deletion.

delete genlist item directly.
instead, remove the bunch of code that was related to lazy deletion.

we can success, if we clean up the resources and callback calls properly.
here i only found item focus callback that (may) required the lazy deletion.

theme: Add newly added icons to EXTRA_DIST to make sure they end up in the tarball

We need to list the icons here to let autotools know we want them int the release
tarballs.

Revert "win INLINE_IMAGE test now crashes as expected when trying to use it in a non-useless manner"

This reverts commit fd71874024d14e5374aef93eae7d4cfbc0ea4521.

As the commit message says, the test crashes now. The ticket 1973 is
related to that issue.
Now, I understand there could be a bug inside inline stuff but it doesn't
mean that you have to push it and screw up all the test. You could just
attach a patch to the ticket and that's all folks!

genlist: send signal to deco_all_view additionally.

genlist: +warning

genlist: make up lack of behaviors in item_disable

genlist: replace to internal call

elm_win: Fixed the issue of updating ecore_wl_window location

Summary:
@Fix

Signed-off-by: vivek <vivek.ellur@samsung.com>
Reviewers: devilhorns

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

Gengrid: fix gengrid reorder animation bug

Summary: When gengrid reordering by thumb scroll, some items disapear on screan.

Test Plan:
1. Enabled thumb scroll in elm config
           2. Launch gengrid test in elementary_test
           3. Check reorder mode enabled
           4. Try to move reordering item to upon or lower area by thumb scrolling,
              items disapear.

Reviewers: seoz, raster

Reviewed By: raster

Subscribers: bluezery

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

[spinner] use strncmp, not strcmp to support styles starting with vertical

Summary: Use strncpm, not strcmp to support styles starting with vertical

Test Plan: Define spinner style which has 'vertcial' as a prefix

Reviewers: raster, cedric, seoz, woohyun, jaehwan, Hermet, jpeg

Subscribers: seoz

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

genlist: +parent validation check

genlist: code refactoring.

there are 2 variables that has same purpose.
cut off one dependency.

elm_widget: elm_object_disabled_set API fix

Summary:
When elm_object_disabled_set API is called on any widget, All the
subobjects of the widget also has to be disabled issue fix.
@fix

Test Plan:
Call the API elm_object_disabled_set on any container widget like box,
layout etc:-, fix can be tested with test_box.c and test_layout.c test
codes.

Reviewers: Hermet, seoz, raster

Subscribers: govi, rajeshps

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

elm_image: Fixed to have exact size using elm_image_fill_outside_set()

Summary:
There was a problem that image is stretched over its size
when elm_image_fill_outside_set() is applied.
This patch fixed that.

@fix

Reviewers: Hermet

Subscribers: Hermet

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

genlist: fix crash issue.

"selected" list has the eo list, not elm_gen_item.

this is side effect caused while on refactoring genlist.

elm_font: Fixed resource leak in function elm_font_available_hash_add

Summary: @Fix

Reviewers: raster, singh.amitesh, seoz, kimcinoo, Hermet

Reviewed By: Hermet

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

theme: fix misprint in popup style alias name

@fix

Reviewers: cedric, raster, seoz

Subscribers: reutskiy.v.v

Projects: #elementary

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

elm filesel - fix eio access of widget data after de in eio threads

@fix

genlist: add more arguments validation checks

elm progressbar - fix invalid access to freed units string

@fix

fileselector: added a search icon

Summary:
fileselector: redirect keystrokes to the searchbar

Keystrokes like a-z A-Z 0-9 are redirected to the searchbar, so you dont
need to click arround to search files in the fileselector.

The search icon is from ApB (Apostolos B.), thx for that! :)

@feature Improved the searchbar

Reviewers: raster, seoz

Subscribers: ryuan.choi, cedric, seoz

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

Conflicts:
src/lib/elc_fileselector.c

genlist: + parent validation check

genlist: code refactoring.

type check than boolean(to be removed)

genlist: use simple macro

genlist: null check++

genlist: clean up missing resources in genlist_clear()

genlist: cancel multi selected items.

cancel multi selected items if the multi-select mode is canceled.

@fix

colorselector: memory leak fix  in _color_bars_add

Summary:
this function is called at least twice: in _smart_add and every time _theme_apply is called.
so we need either correctly free sd->cb_data or reuse it

@fix

Reviewers: seoz, cedric, raster, Hermet

Subscribers: reutskiy.v.v

Projects: #elementary

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

elm_segment_control: Fix elm_object_item_text_set bug.

Summary:
Cause : When part name is NULL, edje_object_part_text_escaped_set is not called.
Measure : Add code for each conditions.

@fix

Reviewers: Hermet

Subscribers: Hermet

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

elm_diskselector.c: idle enterer add condition changed.

Summary:
The idle enterer added even after deleted sd for diskselector.
it makes crash.

@fix

Test Plan:
1. Run elementary_test
2. Execution diskselector.
3. Close diskselector sample window.
4. Crash occured.

Reviewers: Hermet, seoz, Jaehyun

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

win INLINE_IMAGE test now crashes as expected when trying to use it in a non-useless manner

ref T1973

elm_image now inherits edje signal emit

@feature

thumb: Make sure to stop pulse animation on thumbs that failed to generate

genlist: refactor code.

select/unselect items by using common select function.

theme - more efm icons from appb

fixes T1582

genlist: use macro as possible.

gelnist: unswallow makes sense. not smart member del.

elementary: Fix issue with event data causing an invalid free

Summary: Since we end up calling free on the event->data parameter
after a drop is completed, we should actually be allocating a string
for this data (to be free'd later). This also fixes an issue where the
event length was not being set.

@fix

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

elementary: Fix return values to use Eina_Bool

Summary: This function is expecting Eina_Bool as the return value, not
ints so let's use EINA_FALSE here.

NB: this just fixes a copy/paste issue

@fix

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

elementary: Fix DnD to work again in wayland

Summary: In order for ecore_wl to start a drag, we need to send the
parent window of the object being dragged. Previous code was not
getting the parent window properly.

NB: This patch also fixes an issue where the data being sent to the
drop callback was being incorrectly sent.

@fix

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

genlist: keep the item class as given argument.

genlist: Fix genlist index get bug

Because not all item block have "max_items_per_block" number of items in it's block.
So the calculation is not correct when items are added & removed in each block.

Anyhow, we need more efficient way for this API because genlist uses inlinst.

genlist: add some comment in code.

genlist: use macro as possible.

genlist: use ELM_NEW

genilst: unselect items if the genlist select mode is display only or none.

make it behavior same with elm_genlist_item_select_mode_set()

@fix

genlist: unselect item if the select mode is display only or none.

@fix

ggenlist: code refactoring.

don't do anything if same mode is applied.

Improve win dialog test to also include a test for standard_win

Elm_Entry: Fix dropped text data insertion

Fixes bad drag&drop into entry widgets, since the drop callback
assumed that the dropped text ends with a NULL character.

How to reproduce:
1. Open 'elementary_test -to Entry'
2. Open some application (e.g. Firefox)
3. Select some (preferably short) text, and drag&drop it to the entry
widget. This sometimes pastes additional corrupted text (better try
with a 1-3 characters text).

@fix

Added the elm_win_util_dialog_add() API function

This new @feature is just a convenient way to create dialogs, It
mimic the elm_win_util_standard_add() function but crete a
standard dialog, with background and the parent set.

WinDialog test changed to use the new function.

Icon: added documentation for the new fdo icon names

genlist: remove useless deco texts.

genlist: + null check

genlist: code refactoring

refactor _item_content_realize() prototype to keep the consistency with state,text_realize()

genlist: code refactoring.

apply facade at _view_inflate() for simple item realization.

genlist: code refactoring.

remove contents_obj but use contents instead.