elementary: Remove unused static variable

Summary: clang reports _list_focus_objects as being an unused static
variable, so remove

@fix

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

elementary: Remove unused static variable

Summary: clang reports SIG_THEME_CHANGED as being an unused static
variable, so remove

@fix

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

elementary: Remove unused variables

Summary: clang reports SIG_BLOCK_CLICKED and SIG_TIMEOUT as being
unused variables, so remove

@fix

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

elementary: Remove unused function

Summary: clang reports _icon_size_min_get as being an unused function,
so remove

@fix

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

elementary: Remove unused static varible

Summary: clang reports _smart_callbacks as being an unused static
variable, so remove

@fix

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

elementary: Remove unused variable

Summary: clang reports SIG_LANG_CHANGED as being unused here, so remove

@fix

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

elementary: Fix passing wrong parameter

Summary: clang reports passing EINA_FALSE here is being treated as an
Eo object, so pass NULL instead

@fix

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

elementary: Fix passing wrong parameter

Summary: clang reports passing EINA_FALSE here is being treated as an
Eo object, so pass NULL instead

@fix

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

elementary: Remove unused static variable

Summary: clang reports ELM_APP_SERVER_VIEW_PROPS_ENTRY as being an
unused variable, so remove.

@fix

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

elementary: Remove unused static variable

Summary: clang reports SIG_CLICKED as being unused so remove the
variable

@fix

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

elm_test: Fix bug in GLView test

Scenario:
  const char * accel = accel_get();
  accel_set("gl:depth");

Now accel may not be valid anymore since the original
Eina_Stringshare might have been destroyed. So there was
no point in saving the const char * pointer. It was not valid
after the call to set(). This is a bit confusing here. Returning
char* would be clear: the caller would own the data.

toolbar: do not change align for "noicon" toolbar items

too many rels and complexity to track this to a more coherent
message

probably broken since 1.0 since there are zero "noicon" toolbar
tests

@fix

fix T2782

image: remove unused variables

elm_image: let's not trigger confusing update add.

Elm_Image rely on Evas_Image ability to rotate content. There is no
pixels manipulation involved that require an update_add here. In fact
this cause a bug where the content would disapear due to evas trying
to get some pixels data that are not where it think they are.

elm_hoversel: fix to show dismiss animation

Summary:
In hover.edc, hoversel dismiss animations are defined,
but it doesn't work at all. (Hover is removed before animation)
This patch makes hoversel remove its hover after dismiss animation.

@fix

Test Plan: elementary_test hoversel

Reviewers: Hermet, cedric

Reviewed By: cedric

Subscribers: woohyun

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

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

elementary: Fix calling ecore_evas_wayland_move with improper values

Summary: As ecore_evas_wayland_move ends up updating the location of
the ecore_wl_window, we should be calling move with the proper
geometry as this ends up updating the internal ecore_wl_window
geometry (which in turn sends to xdg_surface_set_window_geometry).
This is part of a fix where xdg_surface_set_window_geometry was
getting sent inproper values.

@fix

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

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

Revert "gengrid : add all_contents_unset API in gengrid for reusing contents."

This reverts commit 1094fb77a0ee23cbe1e8b15783e406a8d93b7fd4.

Feature is freezed for new released version.
This patch must updated after release finished.

toolbar: Fix crash on more_item when shrink mode is set with no item.

Toolbar creates a more_item when shrink mode is
ELM_TOOLBAR_SHRINK_EXPAND.
If select mode is ELM_OBJECT_SELECT_MODE_ALWAYS and there is no item
appended, then toolbar sets a selection on a newly created item.
In this case, sd->more_item is referenced before the new more_item is
assigned to sd->more_item.

@fix

gengrid : add all_contents_unset API in gengrid for reusing contents.

Summary:
Add elm_gengrid_item_all_contents_unset API for reusing contents in application side.
User can unset all contents or re-use their contents for increase scrolling performance.
genlist already support this API.

@feature

Test Plan: Need to add test in elementary_test.

Reviewers: raster, singh.amitesh, cedric

Subscribers: woohyun, Hermet, seoz

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

elm cursor - fix hotspot tracking to actually work right

hotspot tracking was broken. this fixes it.

@fix

elm cusro config - move elm to by default look for cursors in theme

previous config ignored elm theme if ithas cursors. this was just
wrong, so switch default back to using theme and update configs
accordingly etc. this is houw it should have worked.

elm cursors - if min size is too small then force it to be sane

cursor if less than 8x8 then make it 8x8. avoidx 0 sized pointers.

@fix

tests: add new tests for AT-SPI beta API.

list: set atspi parent despite atspi mode.

Path fixes failing test case when atspi mode was off.

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

po: update po files

tests: disable new atspi parent test for me as it breaks

Breaking distcheck for me and delaying the alpha1 tarballs. The API is still
marked as unstable but we might want to look into it during the beta's.

elementary: Comment out (#if 0) unused functions

Summary: Unsure whether these are left-over, or are going to be
implemented in the near future...but for now, #if 0 out these 2
functions as they are currently unused.

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

elm run - be explicit about elm run not nesting in docs

@feature

atspi: add new relationship append/remove/clear API

Allow to add some extra contextul information about accessibility objects
which can be used by Assistive Technology to provide better user experience.

API is still marked as beta.

@feature

atspi: support org.a11y.atspi.Collection interface

org.a11y.atspi.Collection interface allows to perform fast search
queries on accessibility objects tree.
Due to lack of specification of collection interface this implementation
base on gnome project implementation from https://git.gnome.org/browse/at-spi2-atk/.

@feature

Revert "Genlist: only get item data if needed."

This actually doesn't work for coverity, I guess we just need to mark it
as false-positive, as it will never be null.

This reverts commit 178e7e6ac9419742d1bbe583fd252fc6536e5649.

doc: make sure we actually run the command to get the example screenshots

These screenshots are used in the documentation but we never made sure that we
have them actually. This resulted in our docs missing image links.

Thanks to Jean Rene Dawin for reporting it at the Enlightenment Developer Day.

Genlist: only get item data if needed.

Moved the data fetching to be after the tests. This doesn't really
matter and is mostly there to silence coverity. Coverity was complaining
about "it" being dereferenced before the null check (implicit in
EO_OBJ), so reordering it will fix it.

CID 1040000

config: fix NULL access on invalid config loading

CID 1308381

elm_hover: delay hover dismiss while handling edje signal

Summary:
elm_hover_dismiss() doesn't hide hover directly, but sends signal only.
It can be translated that "dismiss" do not hide hover only, but handles
some works. (EDC program, callbacks etc.)
However, "dismiss" sends signals after hide hover itself, so following
edje signals are ignored. (Hover cannot be seen already!)
This patch makes hover dismiss delayed while handling edje signal.

@fix

Test Plan: elementary_test hover2

Reviewers: Hermet, cedric

Reviewed By: cedric

Subscribers: woohyun

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

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

elc_naviframe: fix crash in strcmp, if text_set is NULL issue

Summary:
Issue: If text set is NULL to naviframe, crash happens in strcmp
Soln: Check for text if NULL,  before passing to strcmp

@fix

Test Plan:
//Pass the text as NULL
elm_object_part_text_set(nf, "title", NULL);

Reviewers: Hermet, cedric

Reviewed By: cedric

Subscribers: cedric

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

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

elm_hoversel: add elm_hoversel_label_auto_changed_set/get() API

Summary:
Hoversel can be used like dropdown or pulldown menu.
In that case, changing hoversel label into selected item lable can be
considered as common usage of hoversel.
This API automatizes changing label.

Reviewers: SanghyeonLee, Hermet, cedric

Reviewed By: cedric

Subscribers: cedric

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

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

elm_hoversel: refine hoversel scrollability function

Summary:
Some modifications for hoversel scrollability are applied.
For example,
The logic for generating scroller is changed. (newly added during activation)
If hoversel's contents are larger than hover,they will be resized to fit hover.
If hoversel is moved (by window rotation etc.), scroller will be resized.
Reduandant code are removed from popup for scrollability.

Test Plan:
You can see the sample for scrollable hoversel in second one
of "elementary_test hoversel".

Reviewers: Hermet, SanghyeonLee, cedric

Reviewed By: cedric

Subscribers: godlytalias, Hermet

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

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

elm_video: port eo_event callbacks to use eo_event arrays

Summary:
Changed eo_event_callback_add to use eo_event_callback_array_add
in elm_video.c

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

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

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

atspi: add translation domain for name and description.

@feature

atspi: properly set parent.

Set proper atspi parents in cases when AT-SPI object tree structure
should be different then elementary tree (mostly in cases of elm_widget_items)

Add regression tests for those cases.

widget_item: emit atspi children-changed only for non-destroyed parent.

atspi: fix state macros after extending Elm_Atspi_State_Type.

atspi: encapsulate common behaviour in atspi mixin class.

This patch moves some duplicated implementation from elm_widget
and elm_widget_item classes into atspi base mixin. It allows to
better share code and reduce maintainability.

Change-Id: I8d264661e7ffade7da72299f29075a536d71eaa3

Revert "elm_calendar: weekdays do not get translated when language is ..."

As agreed on the ML, eina_tmpstr_strftime() should be removed.

This reverts commit 10b94fb1409487e256a01b3c45ff5a9cd9f678da.

test_genlist: remove check for array comparision against NULL

CID: 1324955

Merge branch 'devs/stanluk/perf'

Serie of patches reducing overall memory overhead of atspi mode.

The main changes include:
* use recently fixed eldbus_service_register_fallback function instead
  of regular ones.
* use global accessibility events emitter and handler to avoid registering array
  of callbacks on every accessible object.
* do not keep strings with object paths

atspi: add global event emitter

Introduce global event emitter for accessibility events. With such emitter
there is no need to register array of callbacks on every accessibility object.

atspi: use eldbus fallback interface

Fallback eldbus interface allows to handle dbus method call on
multiple object paths only with single callback. This greatly
reduces a ammount of memory since there is no need
to register eldbus interface for every accessible object.

atspi: reduce memory footprint

Use pointer hash instead of string hash to account information
about accessible objects.
Use statically allocated string to return object paths.

build: remove colorclass generated files

elc_multibuttonentry: add item disable support

Summary:
Add item disable support in multibuttonentry widget

@feature

Test Plan: elm_object_item_disabled_set on MBE item

Reviewers: CHAN, Hermet

Reviewed By: Hermet

Subscribers: CHAN, Hermet, navnbeet

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

multibuttonentry: Fix MBE item box min size calc error.

Summary:
Before item packed again in mbe box. box calc own min height.
In this case, there is no need to increase line number when  entry is box item.
Actually the min height of box calculated wrong here.

@fix

Test Plan:
Give a color to internal box.
Then items add.
Check the internal box area got a wrong height value.

Reviewers: Hermet, cedric

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

multibuttonentry: code refactoring.

remove duplicated computations.

multibuttonentry: Fix padding to be affected by base_scale in edc.

Summary:
MBE internal box get its padding size from edc.
Previously, it does not consider when calculating the padding size.
In this commit, the padding size from edc is divided by its base_scale
so that desirable size can be calculated.

@fix

Reviewers: Hermet, cedric

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

entry: remove evas_object_propagate_events_set() calls on buttons in test.

Seems unnecessary in test code.
It just looks  copied & pasted from wrong code.

po: update po files

test/entry: add editable set

Summary:
There is no good entry test for changing editable mode.
This patch adds a test for changing editable in entry.

Reviewers: Hermet, tasn, herdsman

Subscribers: seoz

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

elc_multibuttonentry: Memory leak fix.

Summary:
If item filters are appended and multibuttonentry is deleted, the
filter list is not freed causing leak
Signed-Off By: Kumar Navneet <k.navneet@samsung.com>
Signed-Off By: Shilpa Singh <shilpa.singh@samsung.com>

Test Plan:
append item filters and when multibuttonentry is deleted leak is
observed, run valgrind in this scenario.

Reviewers: Hermet, cedric, CHAN

Subscribers: navnbeet

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

colorclass: fix leaks on editor creation

colorclass: remove remote editor, make base api public

the remote editor needs to be rewritten to use x11 client messages
and wayland shell extensions at some point. need to make this stable
now to use in e20 release

po: Add Vietnamese translation

Summary: Add translation for Vietnamese.

Reviewers: seoz, stefan_schmidt, Hermet

Subscribers: seoz

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

test_gengrid: Add test case for elm_gengrid_at_xy_item_get

spinner: fix extra callback calls

After a52a53802aebf03d297aa857f0cb6a9779949a49 callback was called after every
spinner value change even if it was made from code.
This commit fixes unfocus issues without breaking callbacks logic

@fix

Reviewers: CHAN, ajwillia.ms

Subscribers: CHAN, ajwillia.ms

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

Signed-off-by: Andy Williams

fix typo in elm_map_overlay_group_members_get API doc

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