elm_icon_external: cleanup ERR msg in error case

Summary:
Setting the group without checking it exists causes a errormessage, this
is pretty annoying for people which want to set a standart icon. And
give a better errormessage if the standart setting failed.

Reviewers: raster, cedric

Reviewed By: cedric

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

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

atspi_bridge: fix memory leak issue

Summary:
Call eldbus_message_unref to free Eldbus_Message structure.
And delete unused Eldbus_Message variable.
CID-386474
CID-391869
CID-394549

Reviewers: raster, woohyun, CHAN, Hermet, seoz, lukasz.stanislawski, cedric

Reviewed By: cedric

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

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

test_gengrid: remove EINA_UNUSED for a used argument

Summary: Signed-off-by: Vaibhav Gupta <g.vaibhav1@samsung.com>

Reviewers: raster, Hermet, cedric

Reviewed By: cedric

Subscribers: sachin.dev, singh.amitesh

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

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

elm_gen_common: fix typo in Struct variable documentation

Reviewers: Hermet, cedric

Reviewed By: cedric

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

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

edje_externals: add safety check for NULL pointer

Summary: Dereferencing null pointers in external_signal()

Reviewers: bdilly, raster, woohyun, cedric

Subscribers: cedric

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

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

app_server: add null check for dbus_state_set/elm_app_server_views_eet_new

Reviewers: Hermet, woohyun, raster, cedric

Reviewed By: cedric

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

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

elm_image: Solve clash on the meaning of "animated"

Summary:
Both Elm_Image and its base interface Efl.Image have a property named
"animated".
The meaning for Efl.Image is whether or not the image has animation.
On the other hand, Elm_Image use it to set whether an image, which supports
animation, is to animate itself or not.
Thus Elm_Image have another property (animated_available) to tell whether the
image have animation or not, making it even more confusing.

Renamed the "animated" property of Elm_Image to "animated_enable" to make
the intended meaning clearer.
Also replaced the "animated_available" property with the "animated" property
from the Efl.Image interface, since both refers to the same thing, in order
to reduce the API.

Added implementations of the legacy methods for backward compatibility.

Address the issue:
- Elm image and the image interface clash because of the meaning of "animated".
Specified in the page:
https://phab.enlightenment.org/w/efl_interfaces/

Reviewers: felipealmeida, tasn

Subscribers: raster

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

config: fix comment

config: make a config directory, if it is not.

create the config directory to register flush file to eio_monitor.

fix more elm config file pointer access

again - follow up from changes to config code to fix seg

elm config files - don't write to file if file handle is null

fixes seg with new file only config for elm

win: fix bug about engine value.

When application does not have any value of engine,
this app should be launched with elm_config's value.

Signed-Off-By: Wonsik Jung <sidein@samsung.com>

win: do not use deferred ecore evas deletion for FAKE wins

these ees will be deleted externally anyway

@fix

win: only trigger del trap if win type is not FAKE

@fix

theme: migrate all e users of darken_rounded_square to new darken_square image

nearly everything in e is square except for this, which was a holdover from the
b&w theme. the new image looks more coherent with the rest of the square look

image created by me, licensing to match rest of theme images

@feature

notification: set fixed on app icon to eliminate edje calc error

entry: emit changed,user signal when text is pasted or dropped

Summary:
When text is pasted or dropped into entry, the "changed,user" signal
should be emitted, not "changed" signal.
Currently, the "changed" signal is emitted.
This patch fixes this issue.

@fix

Reviewers: herdsman, tasn

Subscribers: tasn

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

elm_config: remove X dependency.

Summary:
In wayland, elm_config_all_flush doesn't work.
It has X dependency totally.
It uses ecore_file_monitor instead of X property.

Reviewers: Hermet, devilhorns, raster, cedric

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

comp effects: add tooltip visibility effect

comp effects: increase size of vertical visibility effect to include shadows

elm_flip: small error handle fix

if we cannot create slice, we had better stop to change slice

index: Add NULL check for malloc memory allocation.

layout: Add NULL check for calloc memory allocation.

layout: Fix to pass correct argument for edje_object_part_unswallow.

layout: Add NULL check for ELM_NEW memory allocation.

clock: Add safety check for NULL pointer from localtime() func.

Summary:
localtime() can return NULL when there is an error.
@fix

Reviewers: raster, woohyun, Hermet, JackDanielZ

Subscribers: JackDanielZ

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

cnp: Add safety check for NULL pointer from malloc().

Summary:
It could make crash when malloc is failed.
@fix

Reviewers: raster, woohyun, Hermet, JackDanielZ, thiepha

Subscribers: JackDanielZ

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

cnp: Fix memory leak issue.

Summary: Call efreet_uri_free() API to free Efreet_Uri structure.

Test Plan: None.

Reviewers: raster, cedric, woohyun, Hermet, thiepha, JackDanielZ

Subscribers: JackDanielZ

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

calendar: + null check.

clendar: + null check

gmtime can return NULL vaule.
if so strftime will cause crash.

this patch just prevent that potential situation.

elm_interface_scrollable : Fix logic about setting EVAS_EVENT_FLAG_ON_HOLD

Summary: Even if scroller is not scrolled, EVAS_EVENT_FLAG_ON_HOLD is set. So clicked event is canceled in Edje.

Test Plan:
On the button in the vertical scroller, click button and move mouse horizontal.
Mouse up on the button, so now button is not clicked

Reviewers: Hermet, CHAN, seoz, jaehwan, woohyun, SanghyeonLee

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

DnD: fix memory leak

DnD/X11: Fix invalid access to memory

It seems X doesn't send the data with the termination.

DnD/X11: rename notifiers

As the notify functions don't notify anymore, they have to be renamed to
reflect their real job, i.e data preparers before calling user
functions.

DnD/X11: refactor the notifiers

Each data notifier was preparing data and then invoking the user callbacks
for DnD or selection protocols.
Most of the notifiers code was handling the user callbacks invocation
and was the same for all of them.

This patch simplifies the notifiers by moving the user callbacks
invocation into a common place for all the types, i.e into the notify
dispatcher.
The notifiers prototype has been changed to suit the code modifications.

DnD/X11: remove unused function

fileselector: add NULL check for memory allocation

entry: add dnd image handling

When image is dropped into entry, we should insert image to it.
This patch adds image inserting in dropping.

@fix

Cnp/X11: correct object parameter in datacb

The object parameter in datacb must be the selection request object,
not the selection owner object.
This patch corrects that.

Test case: open two windows with entry, copy text from one entry in
first window, paste to entry in other window.

@fix

Cnp/X11: move entry-only code in elm_cnp to entry

There is code in elm_cnp.c which is used to paste data to entry widget only.
This can cause error if widget is not entry.
This patch removes that code in elm_cnp.c and adds datacb to entry
to insert content to entry.

@fix

Dnd/X11: add dnd support for image, uri types

There is no dnd callbacks for image, uri types.
This patch adds dnd callbacks for those types.

@fix

cnp: Add safety check for NULL pointer from strdup().

Summary:
It could make crash when strdup() is failed.
CID - 329115
@fix

Reviewers: raster, thiepha, woohyun, Hermet

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

cnp: Add safety check for NULL pointer from calloc() for anim_icon.

Summary:
It could be failed to allocate memory for Anim_Icon.
CID - 337351
@fix

Reviewers: raster, thiepha, woohyun, Hermet

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

cnp: Add safety check for NULL pointer from _elm_util_text_to_mkup().

Summary:
_elm_util_text_to_mkup() func could return NULL when malloc is failed internally.
CID - 398441
@fix

Reviewers: raster, thiepha, woohyun, Hermet

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

elm_win: fix keygrab bug to use wrong keygrab mode

The code set worng ecore_x_keygrab mode according keygrab mode.
and changed some codes for readablity.

eo: migrate all event docs to new syntax

comp effects: add vertical visibility effect

eo: migrate enum docs to new format

widget: Add null check

Summary: Adding NULL checking when allocate memory.

Reviewers: Hermet, seoz, cedric

Reviewed By: cedric

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

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

transit: add NULL check for memory allocation.

Summary: Adding Null checking when allocate memory.

Reviewers: Hermet, seoz

Subscribers: cedric

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

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

Updating italian translation

list: add atspi SELECTABLE state to elm_list items.

[layout] support mirrored set for layout which is using elm_layout_file_set()

Summary: mirroed mode does not work, if layout uses elm_layout_file_set().

Test Plan:
the following is test code.
[test.edc]
collections {

   group {
      name: "layout/test";

      parts {

         part {
            name: "bg";
            type: RECT;
               description {
               state: "default" 0.0;
               color: 255 255 0 100;
            }
         }

         part {
            name: "test.rect";
            type: RECT;
               description {
               state: "default" 0.0;
               color: 255 0 0 255;
               rel1.to: bg;
               rel1.relative: 0.2 0.1;
               rel2.to: bg;
               rel2.relative: 0.5 0.2;
            }
         }

      } /* parts */
   } /* group */
} /* collections */

[test.c]
//Compile with:
//gcc -g test.c -o test `pkg-config --cflags --libs elementary`

#include <Elementary.h>
#include <Ecore_X.h>

static void
_bt_click(void *data, Evas_Object *obj, void *event_info)
{
   Eina_Bool mirrored;
   Evas_Object *layout;

   layout = data;

   mirrored = elm_config_mirrored_get();
   mirrored = !mirrored;
   printf("mirred: %d\n", mirrored);
   elm_config_mirrored_set(mirrored);
}

EAPI_MAIN int
elm_main(int argc, char **argv)
{
   Evas_Object *win, *box, *layout, *bt, *check;
   char buf[PATH_MAX];

   elm_app_info_set(elm_main, "elementary", "./test.edj");
   elm_policy_set(ELM_POLICY_QUIT, ELM_POLICY_QUIT_LAST_WINDOW_CLOSED);

   win = elm_win_add(NULL, "Layout", ELM_WIN_BASIC);
   elm_win_autodel_set(win, EINA_TRUE);

   box = elm_box_add(win);
   evas_object_size_hint_weight_set(box, EVAS_HINT_EXPAND, EVAS_HINT_EXPAND);
   elm_win_resize_object_add(win, box);
   evas_object_show(box);

   // Adding layout and filling it with widgets
   layout = elm_layout_add(win);
   evas_object_size_hint_weight_set(layout, EVAS_HINT_EXPAND, EVAS_HINT_EXPAND);
   evas_object_size_hint_align_set(layout, EVAS_HINT_FILL, EVAS_HINT_FILL);
   snprintf(buf, sizeof(buf), "./test.edj");
   elm_layout_file_set(layout, buf, "layout/test");
   elm_box_pack_end(box, layout);
   evas_object_show(layout);

   bt = elm_button_add(win);
   elm_object_text_set(bt, "mirrored");
   evas_object_size_hint_weight_set(bt, EVAS_HINT_EXPAND, EVAS_HINT_EXPAND);
   evas_object_size_hint_align_set(bt, EVAS_HINT_FILL, EVAS_HINT_FILL);
   evas_object_smart_callback_add(bt, "clicked", _bt_click, layout);
   elm_box_pack_end(box, bt);
   evas_object_show(bt);

   check = elm_check_add(win);
   elm_object_text_set(check, "test");
   evas_object_size_hint_weight_set(check, EVAS_HINT_EXPAND, EVAS_HINT_EXPAND);
   evas_object_size_hint_align_set(check, EVAS_HINT_FILL, EVAS_HINT_FILL);
   elm_box_pack_end(box, check);
   evas_object_show(check);

   evas_object_resize(win, 500, 500);
   evas_object_show(win);

   elm_run();
   elm_shutdown();

   return 0;
}
ELM_MAIN()

Reviewers: seoz, raster, tasn, Hermet

Subscribers: seoz, cedric

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

fix list item mode change on elm list

@fix

elm list - theme - make compress moder work again

someone.. somewhere along the way.. likely a long time ago.. broke
compress mode. likely the addition of the ellipsis value as an
ellipsis position or something related to that broke compatibility,
as this used to work. be that as it may... finding out just where is
virtually impossible, so fix it the simpler way - add elipsis values

pager16: add signals for toggling content visibility

shelf: use spacer for content sizing, toggle content visibility

shelf: use constant rects for eventing

test_win_keygrab: Better use strncpy when copying into fixed size buffer.

We should never copy more than PATH_MAX.

CID: 1298016

elm theme - toolbar - adjust padding for the toolbar to not look so bad

therw was always a small 2 pixel padding inside the toolbar. it just
looks bad for toolbar so expand out to boundary of toolbar

atspi: fix popup role and signal emission

tooltip: fix tooltip multi line text issue

Summary:
This is fixed by removing wrap=word from tooltip style.

Resolves: T2034

@fix

Test Plan:
1. elementary_test -> tooltip
2. click "unlock tooltip visibility"
3. check tooltip text. full text is not visible.

Reviewers: seoz, raster, Hermet, cedric, tasn

Reviewed By: tasn

Subscribers: seoz

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

entry: improve at-spi2 accessibility support

* return guide text instead of content
* mask passwords
* set proper at-spi roles when entry enables password mode.

atspi: fix signal name

interface scrollerable: modify the loop_v condition in _elm_scroll_momentum_animator() function

Summary:
change the 'sid->loop_v' condition to '!sid->loop_v' to delete momentum animator
when the vertical scroller reached the end of the content

@fix

Reviewers: raster, Hermet, woohyun

Subscribers: CHAN

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

Gengrid: Fix typo in API Documentation.

Summary: Signed-off-by: Vaibhav Gupta <g.vaibhav1@samsung.com>

Reviewers: raster, Hermet

Reviewed By: Hermet

Subscribers: sachin.dev, singh.amitesh

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

Genlist: Fix typo in API Documentation

Summary: Signed-off-by: Vaibhav Gupta <g.vaibhav1@samsung.com>

Reviewers: raster, Hermet

Reviewed By: Hermet

Subscribers: sachin.dev, singh.amitesh

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

Elm: Add context,open signal in elm_entry.

Summary:
When context,open signal is sent from elm_entry before showing context
menu, dynamic addition of context menu items to entry becomes easier.

@feature

Test Plan:
Run elementary_test
Launch Entry Scrolled test
Right Click on Multiline Markup entry

Reviewers: tasn

Reviewed By: tasn

Subscribers: subodh6129, poornima.srinivasan

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

examples: Replace setenv() with elm_config_accel_preference_set().

Replace environment value set with elm_config_accel_preference_set.

examples: Replace preferred engine "opengl_x11" with accel preference "3d".

Replace preferred engine "opengl_x11" with accel preference "3d".

Focus: add and fix comments

missed from prvious commit

elm_focus: added new focus move policy and elm_object_focus_move_policy_set/get

New focus move policy, ELM_FOCUS_MOVE_POLICY_KEY_ONLY, is added.
If you set this policy as base focus move policy, objects cannot steal
focus by using mouse click or mouse in. Only keyboard input (such as Left,
Right, Up, Down, ...) can make focus be moved.
Additaionally, an object can have its own focus move policy by using
elm_object_focus_move_policy_set API.

@feature

Elm widget: Fix some Eolian warnings.

Revert "Spinner: Add changed callback call when spinner value set."

This reverts commit 8278dff6a742c391f7c04c3de1033574d818dbb6.

This is not good for scenarios and will break the compatibility.

Thanks to David for reporting.

tooltip: fix positioning when tooltip is affixed to a window object

@fix

glview: preserved changed state when altering render mode

fixes the case where glview is marked changed and then the render policy
is modified prior to the render callback executing

@fix

Adjust according to Eo changes.

gengrid/genlist: decrement item counter before triggering item del callback

@fix

Gengrid: fix mirroring bug in gengrid widget.

Summary:
Below bugs are exist in gengrid widget when use horizontal mode and mirrored set.

1. Gengrid item is placed wrong position when mirrored set.
   Current calulation for mirroring in _item_place only consider pan object positions.
   if widget is mirrored, item should be placed opposited position so object x position must be
   consider in mirroring calculation also.

2. Gengrid scroll(pan) minimum size is returned wrong value when mirroed set.
   As the result of 1's calculation present oposite position of items, so align also reversed when mirrored.
   but current gengrid didn't change align x so minimum size is return wrong value
   and scroller expanded wrong direction.

@fix

Test Plan:
1. run elementary_test and set mirroring On
2. run Gengrid2 in elementary_test
3. see how items are placed by push append button repeatly.
4. see scroller shows correct position and items.
5. change usr/bin/test_gengrid.c to set another align value on gengrid and test again.

Reviewers: raster, seoz, Hermet, jaehwan

Subscribers: Jaehyun, anand.km, eagleeye, singh.amitesh

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

Spinner: Add changed callback call when spinner value set.

Summary:
There is a code to compare value is changed.
1256: if (sd->val == val) return;

Even value changed the value changed callback didn't called before.

Reviewers: woohyun, shilpasingh, Hermet

Reviewed By: shilpasingh, Hermet

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

genlist: do a proper clean up in clear().

Summary: Signed-off-by: Amitesh Singh <amitesh.sh@samsung.com>

Reviewers: raster, Hermet

Subscribers: seoz, sachin.dev

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

clipper: removed.

we've discussed and concluded it's unncessary.

elm - fix profile handling if ELM_PROFILE is set - dont listen to x msg

@fix

don't listen to x profile set msg if ELM_PROFILE env var is used -
this overrides.

elm = fix window norender handling

this delays norender and throttle handling to allow for quick
show/hide etc. changes an app may make so to not jump state so often.
this also fixes an issue wqhere config may change leaving all windows
in norender mode.

po/ko: Updated Korean translation.

examples/evas3d: Use correct variable type for a return value.

elm theme - remove commented out efm icon lines so as not to confus people

gengrid: implement Atspi_Selection interface

Allow to select and deselect gengrid items by Assistive Technology
Clients through org.a11y.atspi.Selection dbus interface.

test_glview: remove warning while compiling the elementary package

Summary:
test_glview.c: In function ‘_print_gl_log’:
test_glview.c:347:10: warning: declaration of ‘log’ shadows a global declaration [-Wshadow]

Reviewers: JackDanielZ

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

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

toolbar: implement Atspi_Selection interface

Allow to select and deselect toolbar items by Assistive Technology
Clients through org.a11y.atspi.Selection dbus interface.

genlist: implement Atspi_Selection interface

Allow to select and deselect genlist items by Assistive Technology
Clients through org.a11y.atspi.Selection dbus interface.

theme - efm - redo folder to stand out more amongst all the file icons

the paper with earmarked corner and dir in it just doesn't stand out.
too plain and so you can't tell dirs from other files easily at all.
replace with a more normal foler icon that now stands out.

update po's

theme - add more efm basic file type icons

Icons by ApB - see T1582 - fixes T1582

not really a bug ... just more icons. :)

Genlist: support of ELM_OBJECT_SELECT_MODE_DISPLAY_ONLY mode of genlist item for ELM_ITEM_SELECT_ON_FOCUS_DISABLE config variable.

Summary:
When environment variable ELM_ITEM_SELECT_ON_FOCUS_DISABLE is set
_item_focused_next() function doesn't support for ELM_OBJECT_SELECT_MODE_DISPLAY_ONLY mode.

Signed-off-by: Umesh Tanwar <umesh.tanwar@samsung.com>
@fix

Reviewers: raster, Hermet

Subscribers: singh.amitesh, sachin.dev

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

elm config - init wl if env vars indicate we should

this ensures event id's are initted and more if we are in wl mode like
in x11 mode

@fix

test_genlist: Fix warnings while running "Genlist Textblock".

Summary:
'Genlist Textblock' uses message style which does not have elm.swallow.icon
part.

Signed-off-by: Amitesh Singh <amitesh.sh@samsung.com>
Test Plan: elementary_test -to "Genlist Textblock"

Reviewers: raster, Hermet, JackDanielZ

Subscribers: seoz, sachin.dev

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

fileselector button: correct window parent finding

Summary:
Fileselector button fails to run in inwin mode.
It is caused by incorrect window parent finding.
This patch corrects the window parent finding.

@fix

Test Plan:
run FileSelector Button, toggle Inwin mode option to on,
eo gives error and nothing is shown.

Reviewers: Hermet, seoz

Reviewed By: seoz

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

Gengrid: Updated gengrid to select the last item.

Summary:
Updated gengrid to move item selection in 'vertical' mode to the last
item, when currently selected item is not in the last row. As per the
original behaviour, on moving down from such an item, it takes the
focus out of the gengrid widget. Now it is updated to go to the last
item (in last row), and then on further pressing down, focus comes
out of gengrid.

This is something similar to what happens in other UI systems like in
Ubuntu and Windows, when pressing down on (n-1)th row, the last item
in the nth row gets focused.

Signed-off-by: Vaibhav Gupta <g.vaibhav1@samsung.com>
Reviewers: Hermet, SanghyeonLee, raster

Reviewed By: raster

Subscribers: sachin.dev, singh.amitesh

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