Commit Graph

526 Commits

Author SHA1 Message Date
Carsten Haitzler 1e4c425bea elm layout - fix possible leak
fixes CID 1357365 (new!)
2016-07-08 11:02:41 +09:00
JEONGHYUN YUN 5ada22613f elm_panes : modified to keep panes content left size after orientation changed
Reviewers: woohyun

Subscribers: cedric, jpeg

Differential Revision: https://phab.enlightenment.org/D4141
2016-07-08 09:49:51 +09:00
Stefan Schmidt 488dfa3c3e elm: efl_ui_frame: add back last efl_ui_frame_eo.h header
During the rename this header got lost and distcheck broke for me due to
problems when generating the files from eo. I fixed a similar problem after
the rename ro efl_ui_flip in commit c3c344da41

Subhransu, please keep this in mind when renaming another one so I do not have
to do another of these fixes. :)
2016-07-07 12:22:12 +02:00
Hermet Park a099b6d2b3 Revert "elm_entry: remove unnecessary text clearing in text_set"
This reverts commit 38db4b4c03.

This changes entry behavior that introduce enventor break.
Can't accept it until that break is reasonable.
2016-07-07 11:15:12 +09:00
Taehyub Kim ddeab76da0 elc_popup: move the code making buttons buffer for readability
Summary:
The code of making buttons buffer is away from related codes.
@fix

Reviewers: Hermet, Jaehyun_Cho, jpeg, raster, cedric

Subscribers: Blackmole

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

Signed-off-by: Cedric BAIL <cedric@osg.samsung.com>
2016-07-06 12:50:39 -07:00
Jiwon Kim 50b549faa2 elm_calendar: fix first week is not changed
Summary: @fix

Test Plan:
1. elm_calendar_first_day_of_week_set(cal, ELM_DAY_MONDAY);
2. First week string should be changed to "Mon"

Reviewers: tasn, cedric

Reviewed By: cedric

Subscribers: cedric, jpeg

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

Signed-off-by: Cedric BAIL <cedric@osg.samsung.com>
2016-07-06 12:49:45 -07:00
Jiwon Kim 38db4b4c03 elm_entry: remove unnecessary text clearing in text_set
Summary:
When text_set is called for entry, it sets to empty text
before calling _entry_text_append().
but, _entry_text_append() has 'set' parameter.
Therefore text is set in that function, and clearing is not needed.

It can fix "changed" smart callback is invoked multipe times
when a short text is set to entry.

In addition, if text length is larger than ELM_ENTRY_CHUNK_SIZE,
set text directly as amount of chunk size and idler will append
about remain text.

@fix

Test Plan:
1. elementary_test -to "entry5"
2. click "set 10000" button
2-1. changed message should be printed 1 time
3. click "set 10001" button
3-1. changed message should be printed 2 times

Reviewers: tasn, Hermet, id213sin, cedric

Reviewed By: cedric

Subscribers: woohyun, cedric, jpeg

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

Signed-off-by: Cedric BAIL <cedric@osg.samsung.com>
2016-07-06 12:48:59 -07:00
Cedric BAIL 323000c83b elementary: clear callbacks before destroying object to avoid crash.
@fix

T3188
2016-07-06 10:39:23 -07:00
Ji-Youn Park 7b61ff8722 efl_ui_image: fix documentation 2016-07-06 19:28:45 +08:30
Ji-Youn Park d22403b7a2 efl_ui_image: fix documentation 2016-07-06 18:59:09 +08:30
Jean-Philippe Andre 4073b172ee elm_icon: Mark lookup_order functions as deprecated
I can't mark the enum itself as deprecated as it's used in too
many legacy APIs.

Fixes T3910 (@jypark)
2016-07-06 11:43:11 +09:00
Carsten Haitzler 38ba68a7c7 elm widget - stop segv during shutdown when objects become deleted
@fix
2016-07-05 22:55:16 +09:00
Vitalii Vorobiov 497fc9ecab elc_combobox: add/remove item doesn't resize container with genlist
recalculation of item count is getting only on a first load of combobox.
whenever user add more items, size of container with items doesn't getting more
place (provoking scroller to appear).
but its event more annoying when user delete items, then there are lots of
free spaces left that looks ridiculus

@fix
2016-07-05 15:09:20 +03:00
Jinyong Park 84d48b498c notify: change return value of "theme_apply" function with inner _theme_apply
Summary:
notify's theme_apply function return value with eo_do_super(... elm_object_widget_theme_apply)
but notify's super class is widget, widget_theme_apply return always ELM_THEME_APPLY_SUCCESS.
so, notify's theme_apply function always return ELM_THEME_APPLY_SUCCESS, if it couldn't apply style correctly.

notify apply style via _notify_theme_apply function, so that function must decide theme_apply function's return value.

Reviewers: herb, singh.amitesh, Hermet, cedric, raster, jpeg

Reviewed By: jpeg

Differential Revision: https://phab.enlightenment.org/D4130
2016-07-05 21:06:21 +09:00
Ji-Youn Park e24366de25 [efl_ui_image] fix documentation 2016-07-05 15:19:02 +08:30
Vitor Sousa d4479b3f55 elementary/fileselector: Fix legacy path set/get of Fileselector.Entry 2016-07-04 17:51:09 -03:00
Vitalii Vorobiov 679d5437cd elc_combobox: fix theme and style set for combobox
@fix
2016-07-04 17:54:17 +03:00
Subhransu Mohanty 53035f23e6 elm_frame: updated the eo prefix and documentation
Summary: No idea what else can be done with this widget.

Reviewers: jpeg, cedric, woohyun

Subscribers: raster, cedric, jpeg

Differential Revision: https://phab.enlightenment.org/D3930
2016-07-04 20:21:12 +09:00
Woochan Lee 186d9938cd spinner: spinner entry reactivate when focus come again.
Summary:
When user edit spinner value on entry.
The user want to back on entry to edit spinner value even swiching view(focus out and focus in).

I considered a couple of exception case ( case that entry should not reactivate.)
1. User click spinner button to give focus.
2. User give focus to spinner's object using key action.

Test Plan:
Add sample in elementary_test.
Edit spinner value.
Gives focus to other window.
Back to spinner view.
See the action.

Reviewers: Hermet, woohyun, cedric, jpeg

Subscribers: cedric, jpeg

Differential Revision: https://phab.enlightenment.org/D4120
2016-07-04 19:55:53 +09:00
Artem Popov 71a0573187 Elementary: cnp - check on NULL data in elm_drag_start
Summary:
check on NULL add in wayland elm_drag_start API
@fix

Test Plan: Call elm_grag_start with data = NULL under wayland

Reviewers: zmike, NikaWhite, jpeg

Reviewed By: NikaWhite, jpeg

Subscribers: bu5hm4n, myoungwoon, cedric, jpeg

Differential Revision: https://phab.enlightenment.org/D4119
2016-07-04 11:01:05 +09:00
Daniel Juyung Seo 3627fa7883 elm/gengrid: Fix null pointer dereferences.
CID: 1357210 and 1357209.
2016-07-04 07:31:15 +09:00
Andy Williams 5c8ea3a6f3 elementary: Grab all key presses in elm_code for now
Working around issues where new key bindings were breaking the editor
2016-07-03 20:48:30 +01:00
Jean Guyomarc'h bc8f87c3ac elementary: document elm_table parameters 2016-07-03 21:35:07 +02:00
Daniel Hirt f13b5aebc8 Ui text: port changed_cb from entry 2016-07-03 17:10:57 +00:00
Daniel Hirt 5c776eb534 Ui text: move defer call to a function
Make it easier to call/debug this.
2016-07-03 17:10:57 +00:00
Daniel Hirt eaee241348 Ui text: fix leak in selection_changed 2016-07-03 17:10:56 +00:00
Daniel Hirt 01872dc1e6 Ui text: fix invalid call to image api 2016-07-03 17:10:56 +00:00
Daniel Hirt 391fdc5cac Ui text: send signals to cursors explicitly
The edje code used to register the cursor objects as "edje sub-objects" of the
edje entry object. This is not edje API, so will send these signals explicitly
to the objects.
2016-07-03 17:07:29 +00:00
Carsten Haitzler 6ab167afa1 elm layout - add and delete signal cbs from the same consistent obj
elm layout signal handling was all over the place. using 3 different
ways of adding or deleteing signals from the object. it uses either
obj directly, eo_super(obj) or wd->resize_obj. come on. be consistent.
so using wd->resize_obj worked before and now works properly with
sgnal cbs PROPELY deleted unlike before.

@fix
2016-07-03 15:45:46 +09:00
Jehun Lim effe9b1696 gengrid: fix focus issue when prev/next item is disabled
Summary:
After setting focus to disabled item, focus does not work properly.
It is necessary to consider disabled state of each item
when selecting next focused item.

Signed-off-by: Jehun Lim <jehun.lim@samsung.com>

Test Plan:
1. elementary_test -to 'gengrid disabled item focus'
2. check 'item select on focus disable'
3. move focus to disabled item
4. observe focus and compare before this patch

Reviewers: cedric

Subscribers: minkyu, cedric, jpeg

Differential Revision: https://phab.enlightenment.org/D4115
2016-07-01 17:20:40 +09:00
Jinyong Park 3271ec8d18 theme: return enum from elm_widget_style_set instead of bool
Summary:
if trying to apply incorrect theme, widget apply default theme and return TRUE.
so there is no way to check it really apply correct theme.
To resolve this problem, _elm_theme_set return three type enum

* related history : 4ca3ef4514
* elm_object_style_set is public api, so I didn't change it.
* typedef name [ Theme_Apply ] is temporarily, please suggest better one.

@fix

Reviewers: singh.amitesh, herb, Hermet, cedric, jpeg, raster

Subscribers: cedric, jpeg

Differential Revision: https://phab.enlightenment.org/D4073
2016-07-01 15:09:43 +09:00
Amitesh Singh 1c358205d0 elm: fix compilation
"
make[1]: *** No rule to make target 'efl_ui_text.h', needed by 'all-am'.  Stop.
"
2016-07-01 09:55:51 +05:30
Vitalii Vorobiov 744fa9bba1 elc_hoversel: just little fix of expand/noitem behavior
when hoversel has no item at all and use clicks on it, then it goes into
state called "expanded".
and so then, no matter how many items user would try to add, hoversel won't work
anymore.

@fix
2016-06-30 21:14:26 +03:00
Stefan Schmidt 86525c3dcf atspi_image: improve existing documentation
Mixin and property descriptions.
2016-06-30 18:15:44 +02:00
Stefan Schmidt 63e1a87bc7 atspi_widget_action: add mixin description 2016-06-30 18:15:44 +02:00
Stefan Schmidt b2385840d3 elm_scrollable: add mixin description 2016-06-30 18:15:44 +02:00
Stefan Schmidt fca0105b02 atspi_component: improve existing documentation
Ad more descriptions, consolidate property descriptions and fix smaller
problems.
2016-06-30 18:15:44 +02:00
Stefan Schmidt 740e438627 atspi_action: improve existing documentation
Fix typos, add parameter descriptions and consolidate property describtions.
2016-06-30 18:15:44 +02:00
Stefan Schmidt 6248d70606 atspi_accessible: improve existing documentation
Make sure we describe whole properties and not getters and setters only. Adding
some more docs while being there.
2016-06-30 18:15:43 +02:00
Chris Michael dce25efef4 elementary: Fix explicit null dereference
This patch fixes an issue reported by coverity where 'type' variable
could be null and passing null to strcmp is not good ;)

Fixes Coverity CID1357147

@fix

Signed-off-by: Chris Michael <cpmichael@osg.samsung.com>
2016-06-30 10:59:28 -04:00
Daniel Hirt ee36fdc218 Revert "elm: Fix wrong use of char instead of string and void out parameter"
This reverts commit 61f52a26fb.

This fixes the reported break on T4000.
2016-06-30 11:32:02 +00:00
Jean-Philippe Andre fd63a2c80d elm_widget: Fix mouse wheel event propagation
The wrong type was propagated. Luckily it didn't crash.

Thanks @Hermet for the report.
2016-06-30 15:33:39 +09:00
Jean-Philippe Andre d04ff8d69b evas object: Move display_mode to Efl.Gfx.Size.Hint 2016-06-30 14:34:59 +09:00
Jean-Philippe Andre 2065aedc44 win: Remove font & image cache functions
Those actually belong to elm_config, or rather Efl.Config:
  efl_config_int_set("cache_image_cache_size", 42);
  efl_config_int_set("cache_font_cache_size", 1337);
2016-06-30 14:34:59 +09:00
Jaeun Choi 92a67f1a61 layout: Add NULL check in _elm_layout_text_set function
When the parameter 'text' is NULL in elm_layout_text_set function,
the sub object data with the same part name is removed
from the layout's sub object list and the function returns immediately.
However, if the text part doesn't exist in the list,
a new sub object data is appended to the list even though the text is NULL.
This patch adds NULL check so the function can return in such cases.
2016-06-30 14:20:44 +09:00
Amitesh Singh 366b24726c photocam: add missing EOLIAN prefix 2016-06-30 10:42:18 +05:30
Chris Michael f024234090 elementary: Fix erroneous usage of EINA_UNUSED
'data' parameter is actually used in this function, so remove mis-use
of EINA_UNUSED in function

@fix

Signed-off-by: Chris Michael <cpmichael@osg.samsung.com>
2016-06-29 11:48:53 -04:00
Chris Michael 8fb9c91d8f elementary: Fix over-zealous optimization attempt
This is a fix for some over-zealous optimization attempt. Basically,
we cannot optimize out multiple calls to ecore_wl2_window_input_get as
the window can change in the for loop.

@fix

Signed-off-by: Chris Michael <cpmichael@osg.samsung.com>
2016-06-29 10:52:34 -04:00
Chris Michael 2c85f3b525 elementary: Fix bug in wl dnd where all subsequent mimetypes would be
accepted

This fixes an issue where in wl dnd, if one mimetype was accepted by a
drop, then all subsequent types would have been accepted also..

@fix

Signed-off-by: Chris Michael <cpmichael@osg.samsung.com>
2016-06-29 09:59:32 -04:00
Chris Michael 0addbf9b44 elementary: Reformat elm_cnp to be readable
NB: No functional changes, just formatting so I can read this hot mess

Signed-off-by: Chris Michael <cpmichael@osg.samsung.com>
2016-06-29 09:27:17 -04:00