fix typos in APIs doc

genlist: refractor item focus update code

Add duplicate code into a new local function _elm_genlist_item_focus_update

Revert "index: fix index object got a wrong min value."

Summary:
I need to think more about item omitted case, horizontal case as well.
Now i'm working on this. It may need to refactoring internal logic.

Please revert this commit it break view when index item omitted.

@fix

This reverts commit 560338c5c31bda8912938074cb736eabdb9f1f31.

Reviewers: Hermet, cedric, woohyun

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

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

flipselector.edc: array size +1 to prevent loosing last character.

Summary:
Flipselector lose last character when 50 characters showing case.

This defined value will using in snprintf, fetch_str(_edje_embryo_fn_fetch_str).
As you already know, we have to give the location in array for '\n'.

Test Plan: Give the 50 characters for flipselector text.

Reviewers: Hermet, cedric

Reviewed By: cedric

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

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

multibuttonentry: fix MBE item clear, item delete logic error.

Summary:
elm_multibuttonentry_clear API not worked well.
It deleted only odd items. Because of the sd->items has been modified inside elm_widget_item_del().

Then we don't have to search target item inside of _item_del().

@fix

Test Plan:
Add MBE items.
Call elm_multibuttonentry_clear API.
Check the bug state.

Reviewers: Hermet, cedric

Reviewed By: cedric

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

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

elm_calendar: weekdays do not get translated when language is dynamically changed issue fix

Summary:
When calendar widget is already created and if we change language,
weekdays do not get translated because weekday string is statically
stored only during create.
Update the weekdays again when object is changed.

Test Plan:
1. Create and show calendar widget
2. Change the language
Weekdays do not get translated

Reviewers: cedric

Reviewed By: cedric

Subscribers: cedric

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

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

focus: change the focus don't stay to scroller.

The focus go to scroller only when it needs.
If the focusable object isn't in the current viewport of the scroller,
the scroller should have the focus.
If not, the focus move to the focusable object in the scroller.

@feature.

focus: add the focus_direction feature about item.

When the focus is moved, it uses focus_direction instead of
focus_origin.
It can get the focus by using the geometry of previous focused object or item

@feature

toolbar_item: use actual doc refs

spinner: Fix internal logic about special value show instead of number.

Summary:
If there is a special value for current spinner value, just jump to apply and show up the special value for entry text.
There is no need to check format.

@fix

Test Plan:
Add special value for each spinner value.
Spinner editable set as true.
Click the button layout for activated spinner entry.
Check the entry text.

Reviewers: cedric, Hermet

Reviewed By: Hermet

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

[Genlist] Added correct container_get callback for filter iterator.

Summary: Warning fixed: Container_get mechanism was wrong. Fixed it.

Test Plan: elementary_test => Genlist => Genlist filter

Reviewers: singh.amitesh, cedric, shilpasingh, SanghyeonLee

Subscribers: shilpasingh

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

calendar: weekday update when first day of week changed.

Summary:
When user call elm_calendar_first_day_of_week_set() to change first day of week.
the marked day has to update.
But, it's not updated, because of internal logic error.
I fixed internal logic for update specific day mark.

@fix

Test Plan:
Called elm_calendar_mark_add()
Then called  elm_calendar_first_day_of_week_set(
Check the mark in not updated.

Reviewers: Hermet, cedric

Reviewed By: cedric

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

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

elc_popup: ported eo_event callbacks to use eo_event arrays

Summary:
Changed eo_event_callback_add to eo_event_callback_array_add function

Signed-off-by: Vivek Ellur <vivek.ellur@samsung.com>
Reviewers: cedric

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

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

elc_player: ported eo_event callbacks to use eo_event arrays

Summary:
Changed eo_event_callback_add to use eo_event_callback_array_add

Signed-off-by: Vivek Ellur <vivek.ellur@samsung.com>
Reviewers: cedric

Reviewed By: cedric

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

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

elc_multibuttonentry: ported eo_event callbacks to use eo_event callback array

Summary:
Changed eo_event_callback_add to use eo_event_callback_array_add

Signed-off-by: Vivek Ellur <vivek.ellur@samsung.com>
Reviewers: cedric

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

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

elm_spinner: ported eo_event_callbacks to use eo_event arrays

Summary:
Ported eo_event_callback_add to use eo_event_callback_array_add

Signed-off-by: Vivek Ellur <vivek.ellur@samsung.com>
Reviewers: cedric

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

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

spinner: delete longpress timer when scroller enabled.

Summary:
If there is a spinner on scroller.
User can scrolling and spinner button longpress at the same time.

Prevent this weird action, delete longpress timer when scroller enabled.

@feature

Reviewers: Hermet, cedric

Reviewed By: cedric

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

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

test_genlist: fix compilation warnings

Elm_genlist: Add Item filtering support

Summary:
Add Filtering support in genlist based on filter function set by application.

@feature

Test Plan: Genlist filter sample added in elementary_test

Reviewers: raster, shilpasingh, cedric, SanghyeonLee

Subscribers: SanghyeonLee, divyesh, rajeshps, govi, prince.dubey, poornima.srinivasan

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

elm_toolbar: fix elm_toolbar_select_mode_set API

Summary: Item is unselected when I changed select mode "ELM_OBJECT_SELECT_MODE_DEFAULT" to "ELM_OBJECT_SELECT_MODE_ALWAYS".

Reviewers: Hermet, cedric, raster, seoz, woohyun

Reviewed By: woohyun

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

gengrid: rename function _elm_gengrid_item_focus_raise to _elm_gengrid_item_focus_update

In function _elm_gengrid_item_focus_raise, we are doing more than just focus_raise.
Rename it to something more meaningful.

gengrid: move item focus related code to internal _elm_gengrid_item_focused()

gengrid: fix duplicate edje signal (elm,state,focused) to item on focus

@fix

elm_conformant: display mode set fix

Summary:
Set the display mode to none only when both clipboard and keypad is
hidden.

Signed-Off By: Shashank Pandey <shashank.p@samsung.com>
Signed-Off By: Shilpa Singh <shilpa.singh@samsung.com>
@fix

Test Plan: Keypad opened, Clipboard opened, close clipboard

Reviewers: woohyun, CHAN, Hermet

Reviewed By: Hermet

Subscribers: CHAN, Jaehyun, shashank0990

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

[elm_win] Possible memory leak Fixed.

Summary: Memory assigned to variable "sd->wm_rot.rots" is not freed on window deletion.

Test Plan: Valgrind catch this memory leak.

Reviewers: raster, cedric, Hermet

Reviewed By: Hermet

Subscribers: govi, rajeshps

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

elm - fix focus set on focus in on window due to previous focus fix

this fix is a fix due to the fix in
933dfca5bcda4943ecdd81ab09dc34989234f7f3 whihc fixed other issues. now
focus is properly set where it should be. this affected rage

@fix

[spinner] Notify when value is edited directly

Previously an edit would trigger internal save but not notify
until the up/down button was tapped.
Now notify when value applied (unfocus). @fix

fix typos in APIs doc

Eo: move Focus Direction to elm_general.eot

test_dnd: transfer text without markup when drag label

Summary:
In Types Dnd test, if we drag from label to entry, the text is inserted
with "<br/>". The reason is that we drag with type ELM_SEL_FORMAT_TEXT
but the transferred data is markup.
This patch fixes it by changing the transferred data to text.

@fix

Reviewers: JackDanielZ

Subscribers: seoz

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

dnd/x11: correct coordinates in pos callback

Summary:
The x, y coordinates in pos callback should be relative to
the top-left of the object.
This patch corrects the x,y coordinates and adds poscb to elementary test
to clearly show the bug case.

Test:
      Run Genlist Dnd Dflt Anim with the change in this patch.
      Drag an item to other genlist and see the printed out item, x, y.

@fix

Reviewers: JackDanielZ

Subscribers: seoz

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

focus: remove checking the focus origin in highlight job.

In some case, it can take misoperation.

@fix

gengrid: remove EINA_UNUSED

elm_hoversel: add support elm_object_item_part_text_set()

Summary:
There was override for elm_object_item_part_text_get() only.
This patch implements elm_object_item_part_text_set().

Reviewers: Hermet, cedric

Subscribers: Hermet

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

elc_multibuttonentry: Input panel should be controlled only if
multibuttonentry is editable

Summary:
Input panel should be controlled only if Multibuttonentry is editable, else causes issues in applications controlling keypad by themselves

@fix

Reviewers: CHAN, Hermet

Reviewed By: Hermet

Subscribers: navnbeet

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

slider: slider should be focused, when slider is handled with mouse or touch event.

Summary: slider is not focused after slider is dragged.

Test Plan: elementary_test -> slider

Reviewers: woohyun, seoz, smohanty, cedric, raster

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

build: fix header ordering so the library builds

please compile test before making commits.

ref 286ab4d7cf3ca794d88bd06ec027bb2df6401f50
ref 431492ec497eeaa130f5c2da27ce7a4c7eefe7f9

example/performance: Remove unused variable.

focus: Add elm_focus_item.h to make header include order correctly.

This fixes the cpp example compile error which was triggered by the commit 431492e.

focus: Fix typo in commit 431492e.

focus: add the APIs to set the focus next about item.

Add elm_object_focus_next_item_set/get,
elm_object_item_focus_next_object_set/get and
elm_object_item_focus_next_item_set/get.

If the item and object is set at the same time,
the item is preference to object when the focus moves.

This feature is implemented about gengrid.
It will be implemented about ohter widgets.

@feature

elm_image : call the evas_object_image_load_error_get() before calling the image_preload api

Summary:
If file path is wrong, don't need to call the image preload api.

And the result of evas_object_image_load_error_get api is not correct
after calling the image_preload api.
I will check more about it.

Reviewers: cedric, Hermet, jpeg

Reviewed By: jpeg

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

fix elm performance example to init elm properly and not do engine stuff

cleanup of examples

multibuttonentry: Fix box item append rtl when mirror enabled.

Summary:
MBE item append logic changed when mirrored set as true.

The item appended from left side of internal box to right side, even mirrored set as true.

@fix

Test Plan:
Precondition : elm_widget_mirrored_set as true.

Run mbe sample in elementary_test.
Add item.

Reviewers: cedric, Hermet

Reviewed By: Hermet

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

elm scroller - improve thumbscroll smoothness even more

this does even more smoothing of scrolling follow on from
939d8ef4c3c48120046560814db63858175a2d12

elm toolbar 0 size items fix if icon not found

this fixes a bug with horizontal style toolbar items and toolbar code
itself simply emitting the wrong signals that causes 0 height items.
this fixes T2710

@fix

elm config - fix labels in config ui for smooth time window

elm - thumbscroll finger scrolling - make it far smoother

smooth out scrolling and simplifyconfig as well as expose config apis
to configure it all and elm config ui sliders and checkboxes for
related config values etc. etc. - with this it is much smoother if you
use the default values, though latency is a bit worse. it looks nicer
though.

@feature

elm - when focus goes back in to a window dont have scrollers jump

there is an annoyance in elm that when focus goes back toa window it
restores focus to the last focused widget. if that widget happens to
be invisible in a scroller then this is really annoying as the
scrollers reset. this makes restoring optional. only if there is a
hilight object AND it's visible - then restore, otherwise don't and
revert focus to the window itself again.

@fix

elm autofocus show - fix cases where focus is inside containers

i found that the focus doesnt auto-show when focusing things inside
scrollers and so on. i had to add more points to trigger auto show.
this fixes that

@fix

calendar: fix signal process timing issue when view update.

Summary:
When the calendar view updated(Press next button to activated next month),
a lot of signals process(signal emitted) to calendar edc.

The signals which related with view update, should process in same loop.
If not, the view looks weird.

To ensure signals process same time, i called "edje_object_message_signal_process" after layout sizing eval called.

@fix

Reviewers: Hermet, cedric, woohyun

Subscribers: id213sin

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

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

elm theme - make sure key sounds are on input channel for muting config

@fix

index: rename elm_index_priority_set to elm_index_standard_priority_set

this change is also for consistency with elm_toolbar_standard_priority_set
conceptually the priority meaning here is same.

win: fix FAKE window geometry updating + resize object adding

users of this will want the win to automatically match the geometry
of the ee in all cases

fix T2692

hoversel: rename api scroll_enabled_set to scrollable_set

modify api for consistency.
scrollable_set is more closed to other similar apis.

update mailmap

build: ensure we do not fail clean if check-results.xml does not exist

If we did not run make check this file would not exist and a clean would
fail over it. Using -f ignores this case.

elm_test: Add test case for page scroller

Signed-off-by: Jean-Philippe Andre <jp.andre@samsung.com>

elm_image: fix image preload issue

Summary:
There are two issues about preload
1. elm_image_preload_disabled_set api does not work.
    Always returned before call the evas_object_image_preload()

2. image preload does not work at file_set time.
    If image's show property is not TRUE, do not call evas_object_image_preload() at file_set time.
    But there is no action when image's show property will be TRUE after call the file_set api.

@fix

Reviewers: Hermet, jpeg

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

Signed-off-by: Jean-Philippe Andre <jp.andre@samsung.com>

elm_web2: fix build break & warnings after migration to eo callbacks.

Label test: split Slide and Ellipsis in two separate tests

this reveal the ellipsis + markup bug as explained in T2709

Label: add support for all the fancy tags already provided for entry

Comes with a simple test

@feature

NOTE FOR THEMERS: you must update your theme to support this!

FileSelector Entry: fix behaviour for events

Fix behaviour for changed, activated, press events.
Forwarders must be set on entry's original events,
so they will call corresponding fileselector entry's callbacks.

@fix

elm map - fix crash on badly handled zoom timeout callback

if sd->obj is null then the a zoom timeout has a null data param and
that means we cant get sd data... so dont set up a zoom timeout unless
sd->obj is valid.

@fix

elm map - fix module load craziness loading all elm modules

so elm map loads every module it can find in a recursive dir walk of
all elm modules. this si nuts. this can accidentallly load OLD modules
and thats a recipe for disaster. so check module arch dir aagainst
module arch string to load the right version and be quiet if module
doesn thave the right symbols - it's the wrong module type.

@fix

Genlist: add homogeneos mode to the tree test

show tree+homogeneous brackage, as explained in T2708

Genlist test: cosmetics

* add separator comments between different tests
* keep function definitions on 2 lines

FileSelector: fix behaviour when the file is chosen

The wrong event was sent on the Ok file selector button when the file is
chosen, leading to not update the upper layer.

@fix

genlist: fix indentation & remove whitespaces

gengrid: fix crash issue.

Sometimes the focused_item can be null.

@fix

naviframe: fix a focus issue when deleting the first item

Summary:
When the first item is deleted, focus should not be reverted back
to an object in the content of the item.

@fix

scroller: fix page calculation logic when looping is enabled.

Test Plan:
1. elementary_test -> scroller
           2. Lopping in x axis
           3. click prev page or next page

Reviewers: SanghyeonLee, singh.amitesh, Hermet, seoz, cedric, raster, jaehwan

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

Scroller: Scroll indicatin arrows should be enable if loop is set.

Summary:
When loop is set, the arrows on horizontal/vertical bar
should not be disabled. These arrows show the scrollable direction,
if enabled. Disabled arrows show no more scroll in that direction.

Signed-off-by: Umesh Tanwar <umesh.tanwar@samsung.com>
Merge branch 'master' of http://git.enlightenment.org/core/elementary

Test Plan: elementary_test -> scroller -> loop in x axis.

Reviewers: raster, Hermet, singh.amitesh

Subscribers: sachin.dev, SanghyeonLee, eagleeye

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

elm_genlist: fix sub-items prepending

Summary:
fix sub-item position after prepending if parent item has no other
sub-items

Test Plan:
1. add genlist
2. add parent item
3. compare results of following actions:
 case 1: append sub-item "child1" and then prepend sub-item "child2"
 case 2: prepend sub-item "child2" and then append sub-item "child1"

ExR results are the same
CuR in case 2 sub-items are placed in wrong position

Reviewers: cedric, raster, reutskiy.v.v, SanghyeonLee, Hermet

Reviewed By: Hermet

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

Genlist: Dont select first item if its DISPLAY_ONLY

Summary:
Updated genlist to select first item which is not in
DISPLAY_ONLY mode

@fix

Signed-off-by: Vaibhav Gupta <g.vaibhav1@samsung.com>
Test Plan:
Following is the test to verify:
1. Create a genlist
2. Make 1st item as ELM_OBJECT_SELECT_MODE_DISPLAY_ONLY
3. Now bring the focus to genlist by key down
4. It should bring focus on 2nd item and not 1st.

Reviewers: raster, Hermet, SanghyeonLee, singh.amitesh

Subscribers: sachin.dev

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

Elc_Multibuttonentry: Item resize issue on text set

Summary:
When text is set on a multibuttonentry item, item does not resize
again based on text.

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

@fix

Test Plan: Set long text to multibuttonentry item, after item is created and shown.

Reviewers: woohyun, CHAN, Hermet

Reviewed By: CHAN, Hermet

Subscribers: navnbeet

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

multibuttonentry: Fix mbe  clicked signal is not work.

Summary:
MBE clicked callback doesn't called when user click label, padding area of MBE.
Because of the MBE edc only has the SWALLOW part for internal box. never emit the signal for clicked.
So i added RECT part for bg of mbe.

@fix

Test Plan: Run elementary_test -> execute multibuttonentry sample.

Reviewers: seoz, woohyun, cedric, Hermet

Reviewed By: Hermet

Subscribers: cedric

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

Elc_Naviframe: When push is in progress, block pop until push is complete.

Summary:
When push is in progress, block popping until
push is complete to allow animation to complete.

@fix

Reviewers: Hermet

Subscribers: subodh6129, shashank0990

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

Revert "Elm_Interface_Scrollable: Improvement in looping behavior"

Summary:
This reverts commit bba4c217f2ce4f032fca6298cc8b2fc6ff3d7c19.

"x and y position variable assumes values ranging from min to max " is right.
but in looping behavior, x and y are able to have value bigger than max or smaller than min.

@fix

Test Plan:
1. elementary_test -> scroller
           2. on Loop in X axis
           3. scrolling page to left.
           4. page is moved last page directly.

Reviewers: SanghyeonLee, tanwar.umesh07, raster, cedric, jaehwan, Hermet

Reviewed By: Hermet

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

test_gengrid: add item_select_on_focus_disabled option in gengrid2 test

elementary: Port evas object smart callbacks to use eo event callbacks

Summary:
Changed evas_object_smart_callback_add APIs to use eo_event_callback_add API

Signed-off-by: Vivek Ellur <vivek.ellur@samsung.com>
Reviewers: cedric, Hermet

Reviewed By: Hermet

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

elm_colorclass: Changed evas object smart callbacks to use eo event callbacks

Summary:
Replaced evas_object_smart_callback_add with eo_event_callback_add api

Signed-off-by: Vivek Ellur <vivek.ellur@samsung.com>
Reviewers: cedric, Hermet

Reviewed By: Hermet

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

elm_conform: Port evas smart callbacks to eo

Summary:
Changed evas_object_smart_callback_add functions to eo_event_callback_add functions

Signed-off-by: Vivek Ellur <vivek.ellur@samsung.com>
Reviewers: cedric, Hermet

Reviewed By: Hermet

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

elementary: Port evas smart callbacks to Eo callbacks

Summary:
Changed evas_object_smart_callback_add to eo_event_callback_add functions

Signed-off-by: Vivek Ellur <vivek.ellur@samsung.com>
Reviewers: cedric, Hermet

Reviewed By: Hermet

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

elm_hover: Set min size hint for size and offset spacer before moving or resizing hover

Summary:
Size or offset spacer make hover be able to move its swallows to right positions.
In elm_layout_sizing_eval() for hover, moving and resizing spacers are done
after moving and resizing hover.
It causes in hover's move or resize callback, geometry can not be got correctly.

Reviewers: SanghyeonLee, Hermet

Reviewed By: Hermet

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

multibuttonentry: Use elm focus for items instead of evas focus.

Summary:
Item layout will be added in elm focus tree.
evas_object_focus_set is no meaning here for MBE operation.
So i changed focus API from "evas_object_focus_set()" to "elm_object_focus_set()"

elm_object_focus_allow_set() already called in item adding function.

@fix

Reviewers: Hermet, cedric

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

Gengrid: Fix wrong focus movement in _item_single_select_down

Summary:
The focus movement is wrong when down key press
is done of first item of last row, intsead of
taking the focus out, it moves the focus to next
item (right).

@fix

Signed-off-by: Vaibhav Gupta <g.vaibhav1@samsung.com>
Test Plan:
1. Launch elementary_test -> Gengrid 2 sample
2. Add a few items such that there are 2 items in last row
3. Do a down key press on 1st item of last row
4. Observe focus movement

Reviewers: raster, Hermet, SanghyeonLee, singh.amitesh

Subscribers: sachin.dev

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

gengrid: fix item looping in case of horizontal mode

This mistake is done in a8afaebf3109f1cb8e397e29

elm win - fix to ensure smart calc is done before show

@fix

elm test EXPLODE! feature! explore widgets and their content!

hit ctrl+middle mouse on something...

Genlist: reset item tree expanded state when putting item in cache

Item expandend state need to be resetted (contracted) when the
item goes in the items cache, otherwise when the item will be reused
it will have unwanted state (expanded).

Quite visible in the "Genlist Tree and Decorate All Mode" test

@fix

config: revert first_item_focus_on_first_focus_in to 0

This fixes a behavioural change introduced in D1135.
Steps to reproduce:
1. elementary_test -> genlist/list/gengrid
2. scroll dragging the scroll-bar
3. an item nearest to viewport gets selected when you release the mouse button

Most of the time, we don't want item to be selected or focused on dragging
scroll-bar via mouse. Revert config flag to 0 to have old behaviour before D1135.

fixes: T2662
@fix

toolbar: fix issue of focus not coming to item on mouse click

resize object was not getting focused in case of mouse click.

Steps to reproduce:
1. elementary_test -to "toolbar focus".
2. set focus to toolbar item via mouse.
3. focus does not come to toolbar.

fixes T2697
@fix

gengrid: no need to loop in case of reorder mode is enabled

There is no need to loop when reorder mode is enabled. This looks
ugly and nobody wants it. This fault was introduced in 7aaa5c8d0a4a5714b4f1bf79e

This patch also fixes following crash
   1. elementary_test -to "Gengrid 2".
   2. Enable all the options available.
   3. Do reorder of item via keys.
   4. Observe segv on reordering edge items.

@fix

Add missing documentation of DirectDraw rendering

gengrid: fix item focus movement in case of horizontal mode

Steps to reproduce:
1. elementary -> gengrid focus
2. change mode to horizontal
3. set focus to one of item in 2nd column.
4. Press Left direction key
issue: focus moves out of gengrid.

@fix

Elm win: Fix according to the new behaviour of parent_set.

This new behaviour was introduced in
9c78ee0bf4125c095e85f7fcf9921586cda64a52.

gengrid: add curly braces to avoid ambiguous 'if'

elm_win: Initilize the variable 'preferred_rot' as a -1 before invoking _elm_win_xwin_update() in _elm_win_finalize_internal().

Summary:
since the variable 'preferred_rot' is checked its validation in _elm_win_xwin_update(),
so, should be initialize it before invoking.

@fix

Reviewers: jypark, woohyun, Hermet

Reviewed By: Hermet

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

progressbar: fix memory leak

Summary: when elm_progressbar_value_set is called, progress_status is always created.

Test Plan: elementary_test -> progressbar

Reviewers: woohyun, Hermet, cedric, raster

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