multibuttonentry: add horizontal, vertical padding between items.

Summary:
There is no way to give padding between each items, previously.
So i added horizontal, vertical padding here.

I will add APIs for set, get padding size set, get.

@feature

Test Plan:
Change the horizontal, vertical paddidng value in multibuttonentry.edc
Run elementary_test
Execute multibuttonentry sample.
Add items.
Check the padding between items.

Reviewers: woohyun, seoz, Hermet, raster

Reviewed By: Hermet, raster

Subscribers: raster

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

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

radio: add "elm,activate,radio,on" and "elm,activate,radio,off" signal to support value change animation.

Summary:
Currently when user clicks on the radio, and when api value_set() called by
the developer we emit "elm,state,radio,*" signal. To support value change animation when
user interacts with radio. There is no way to distinguish the action in EDC.
This "elm,activate,radio,*" is a way which edc can make use to distinguish the state change signal
by the user action or due to api call.

Reviewers: Hermet, raster, cedric

Reviewed By: cedric

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

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

check: add "elm,activate,check,on" and "elm,activate,check,off" signal to support state change animation.

Summary:
Currently when user clicks on the checkbox, and when api state_check() called by
the developer we emit "elm,state,check,*" signal. To support state change animation when
user interacts with check box. There is no way to distinguish the action in EDC.
This "elm,activate,check,*" is a way which edc can make use to distinguish the stae change signal
by the user action or due to api call.

Reviewers: woohyun, raster, cedric, Hermet

Reviewed By: Hermet

Subscribers: id213sin

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

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

hoversel: add key binding for widget activation

Test Plan:
1. Install previous version of elementary
2. Remove config from homedir
3. Build new config with elemetnary_config app
4. Install patched version of elementary
5. run elementary_test Hoversel test
6. Space end Enter should open hovesel

Reviewers: seoz, jaehwan, singh.amitesh, cedric

Reviewed By: cedric

Subscribers: seoz

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

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

genlist: add SIG_CHANGED("changed") smart callback in genlist.

Summary:
SIG_CAHANGED notify user to
genlist is now changed their items and properties and all
calculation is finished, so user can get correct values in
that callback.

This SIG_CHANGED smart callback already supported in gengrid.

Test Plan:
N/A
This patch is addition of new smart callback.

Reviewers: raster, seoz, singh.amitesh

Subscribers: Hermet, cedric, eagleeye

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

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

spinner: entry text makes to registered special value instead of number.

Summary:
Entry has only number text when activated and value changed, Even user registered special value to each spinner value.
If the user registered special value, we should check then showing the special value instead of number text.
Its will more helpful to expect the result when user set a spinner value using spinner entry.

@fix

Test Plan:
There is a spinner sample in elementary_test.
Changing editable value as TRUE for the last spinner object.
Click the spinner to activated entry and check the result.

Reviewers: Hermet, Jaehyun, cedric

Reviewed By: cedric

Subscribers: cedric

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

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

configure: Switch to dev mode again. Merge window for 1.16 is open now

release: Update NEWS and bump version for 1.15.0 release

test_popup: add popup with content area only example

test_cnp: remove explicit type castings.

In C, the explicit typecasting is not required.

Revert "Adds eglfs support"

This reverts commit de52ee0192b5b6d8ad542fb2fd9e4185b39b6518.

Revert "Removes ELM_CHECK_BACKEND[EGLFS] which searches for an ecore-eglfs"

This reverts commit aff92d581ed052413aa36785a9291e47315eb518.

Removes ELM_CHECK_BACKEND[EGLFS] which searches for an ecore-eglfs

Adds eglfs support

popup: fix resize issue when action area and title area is absent.

Summary:
set scroller size in content_set.

Resolves: T2623
@fix

Test Plan:
1. run ecrire
2. Add some text in newfile
3. Close ecrire
4. a almost zero size popup.

Reviewers: tasn, herdsman, raster, SanghyeonLee, stefan_schmidt

Reviewed By: stefan_schmidt

Subscribers: stefan_schmidt, herb, seoz

Maniphest Tasks: T2623

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

prefs: add empty field to avoid build warning.

this addition is a trick to avoid g/gcc compile warning.
warning: empty struct has size 0 in C, size 1 in C++.

test_disable: rename window name & title according to test name

colorselector: fixed up the selected process

When mouse moves out of the color item area,
mouse up should not give selected smart callback.

@fix

panes: Migrate docs to new Eolian format.

grid: convert docs

menu_item: convert docs

panel, pan: convert docs

plug: convert docs

photocam: convert docs

thumb: convert docs

view_form, view_list: convert docs

diskselector: return when the new item is NULL

cnp: use ELM_SAFE_FREE

test_hoversel: add example of setting focus to item

elementary: Fix calls to ecore_wl_window_update_location

Summary: This fixes a build break from previous commit

NB: Note to self: COMPILE FIRST !!

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

elementary: Call ecore_wl_window_update_location function when opaque region gets updated

Summary: This fixes T2624 where ecore_wl_window_update_location was
being called inside the Ecore_Evas too often. It is now only called
from the Elementary window whenever we update the opaque region. This
should help to fix the resize issue that we are seeing in wl.

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

hoversel: use elm_object_focus_set to set focus on subobjects.

We should steal the focus while setting focus to subobjects from parent.
This issue is reproducable in case of setting focus on one of items of hoversel
on expanded cb of hoversel.

@fix

elm: simulate the clicked event over the object only.

Summary:
Two different behaviour are observed with clicked event.
1. When scroller is not present:
   a. Mouse down over object,
   b. mouse move outside the object
   c. mouse up, clicked happens
2. When scroller is  present:
   a. Mouse down over object,
   b. mouse move outside the object
   c. mouse up, clicked NOT happens

So clicked callback should happen over the object only i.e when mouse move out of the objcet clicked should be ignored, and behaviour will be consistent

@fix

Test Plan: NA

Reviewers: shilpasingh, cedric

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

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

elm_radio: remove implementation of elm_widget_disable as it doesn't do anything else than calling parent virtual function.

elm_radio: enable state change when radio is disabled

Summary:
Currently, radio is set EINA_FALSE when radio is disabled.
This causes unexpected behavior when the radio is selected and
get disabled. Using elm_radio_value_get() API will return
the value of disabled radio. This is not consistent with setting
EINA_FALSE when radio is disabled. Thus, disabled radio should
also have selected and unselected states.

Reviewers: woohyun, smohanty, Hermet, cedric

Reviewed By: cedric

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

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

label: convert to int first (avoid compiler warning)

interface_scrollable: convert to int before using abs first

@fix

interface_scrollable: don't use integer abs on floating point

@fix

cnp, entry: misc warning fixes

elm_atspi_bridge: fix warnings

elm_frame: Changed eo docs of elm_frame to new format

Summary:
Changed docsw of elm_frame.eo according to the new format

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

Reviewed By: q66

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

elm_flipselector: convert docs of elm_flipselector.eo to new format

Summary:
Changed docs of elm_flipselector.eo and elm_flipselector_item.eo to new format

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

Reviewed By: q66

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

elm_fileselector: Converted eo docs of fileselector_entry to new format

Summary:
Converted docs of elm_fileselector_entry.eo to the new format

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

Reviewed By: q66

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

elm_external: shutdown elm after del callback

This fixes a crash in edje_inspector as:
- delete callback is called on the object
 --> this unloaded elm
- destructor is called on the object
 --> this did some elm stuff, in particular read the config
     from _elm_config (now NULL).

Instead, shutdown elm after del, during free.

Also reset hash pointer to NULL after free.

color_class: Fix ERR due to uninitialized eldbus

This happens in edje_inspector for instance, which did not
initialize eldbus, even though it's trying to use it.

multibuttonentry: internal label object status, box pack, unpack logic clean up.

Summary:
Even user sets empty string here for text, label still packed in box with min size(finger size).
label_set internal logic is not clear before.
ex: object min set, resize check the string length then sending signal, min set again.
so i deleted the signal emit code here, we can handle label show, hide status in code and it's more clear.
Besides "sd->label" is always true all of the case. we need to check whether it's packed or not.

The label will be added to parent's sub object when it's packed into box.
label_set as empty string in adding time also meaningless.

@fix

Test Plan:
Run multibuttonentry sample.
Set a text somthing for it then set an empty string here.

Reviewers: Hermet, woohyun

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

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

colorselector: remove EINA_UNUSED that where clearly used !

Colorselector: fix wrong object for callback call function

Summary:
When spinner change it's value it should call callbacks for
colorselector but not for spinner.

@fix

Reviewers: raster, reutskiy.v.v, cedric

Reviewed By: reutskiy.v.v, cedric

Subscribers: NikaWhite

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

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

multibuttonentry: update end text(item count) even mbe shrinked.

Summary:
In some MBE use case, Application can append item using only item_append API.
in that case, the end text(item count) doesn't update properly.
So, i add a code for fix it.

@fix

Test Plan:
1. Add a button for call "elm_multibuttonentry_item_append()" in test_multibbutonentry.c
2. Make MBE shrink mode.
3. Click button to append item to shrinked MBE.
4. Check the MBE end text state.

Reviewers: Hermet, woohyun

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

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

multibuttonentry: fix bug with uneditable MBE use case.

Summary:
If there is a uneditable MBE, the MBE doesn't have to get a entry inside of the box.
It breaks MBE box layout, resize logic.

And check the box focus, entry visible is not enough to cover MBE is uneditable case.
If MBE is uneditable, box_resize callback called when item added. in this case,
we need to check what view type user want. if its not shrink view here, do not make
MBE to shrink mode.

@fix

Test Plan: Run elementary_test -> execute multibuttonentry  sample with editable set as false.

Reviewers: Hermet, seoz, woohyun, cedric

Reviewed By: cedric

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

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

entry: few times the hit_rect is not clipped properly during genlist scrolling, with entry as content.

Summary: It has been observed that while scrolling of genlist with entry, clipping is not happening and "hit_rect" comes over navi item title. e.g. During theme change such scenario comes.

@fix

Test Plan: Scenario not in elementary test. Entry as content of genlist item. Same genlist should be the content of naviframe item. In this scenario genlist height must be grater than viewport sie so scrolling possible.

Reviewers: cedric, raster, Hermet, woohyun

Subscribers: rajeshps, govi

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

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

elm_colorselector: change operation order to limit float operation imprecision.

Summary:
The numerical formula below is calculated with the double type.
-> h_pad / edje_object_base_scale_get(wd->resize_obj) * elm_widget_scale_get(obj) * elm_config_scale_get()

But if number of significant figure of the return value goes over 15, the epsilon error will happen.
Because of that, it is better to divide to do later.

Reviewers: woohyun, raster, cedric

Reviewed By: cedric

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

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

multibuttonentry: delete clicked signal preventing code when mbe shrinked.

Summary:
We don't have event block area in mbe.
Item clicked callback is not called when user clicked the item in mbe shrinked state.
However, item press effect will be shown.

@fix

Test Plan:
Delete the line always calling expanded set API in MBE clicked callback.
Execute MBE sample in elementary_test.
Check the item got a click event but not call the clicked callback.

Reviewers: Hermet, woohyun

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

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

elm_flipselector: remove redundant header file

Summary:
After applying eo to elm_flipselector_item,
APIs in elm_flipselector_common.h are generated by eolian.
elm_flipselector_common.h is no more necessary.

Reviewers: Hermet, cedric

Reviewed By: cedric

Subscribers: Hermet

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

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

elm_calendar: remove duplicated referece

Summary:
elm_calendar_weekdays_names_set() API is generated by eolian,
but its prototype is defined in both elm_calendar_common.h and
elm_calendar.eo.legacy.h.
Remove duplicated reference from elm_calendar_common.h

Reviewers: Hermet

Subscribers: Hermet

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

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

win: correct setting of wayland opaque region

Summary: Opaque region calculation was incorrect.

Reviewers: zmike, devilhorns

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

calendar: fix use of strftime.

strftime is quite a bad POSIX API. You won't know the size it needs to put the strings in
until you did give it enough byte and it could also fail without a clear way to detect it.
So this patch work around that and introduce a currently internal helper that will be moved
later to Eina.

This patch has been triggered by the issue reported and the attempt fix from D2877.

release: Update NEWS and bump version for 1.15.0-beta3 release

elm_diskselector: convert eo docs of diskselector to new format

Summary:
Converted docs of elm_diskselector.eo and elm_diskselector_item.eo to the new format

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

Reviewed By: q66

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

elm_ctxpopup: convert eo docs to new format

Summary:
Converted docs of elm_ctxpopup.eo and elm_ctxpopup_item.eo to new format

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

Reviewed By: q66

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

elm config save - work around compiler bug that loses local var value

yes. compiler bug. believe it or not.

elm_gengrid: fix missing variable initialization.

Summary:
- item's still_in variable is not initialized.

@fix

Reviewers: seoz, singh.amitesh, raster, SanghyeonLee

Reviewed By: SanghyeonLee

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

elm_focus: focus_resion_show does not need to be called for all parents

elf_fileselector: Avoid memory leak in error path

If filter_add fails here we would go out of scope and leak custom_filter. Free
it instead.

CID 1308261

elm_dayselector: converted eo docs of dayselector to new format

Summary:
Converted docs of elm_dayselector.eo to the new format

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

Reviewed By: q66

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

Elm_Prefs: use Evil API

The current test is not sufficient to check if the path is relative or
not. Use Evil API instead

@fix

elm_gengrid: Fix condition error.

Summary:
I missed "!" in if condition and missed it->unhighlighted_cb().

@fix

Reviewers: singh.amitesh, raster, cedric, SanghyeonLee

Reviewed By: SanghyeonLee

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

elementary: Fix external variable declaration to match const char

@fix

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

elementary: Fix compiler warning disgarding const qualifier

Summary: eina_environment_tmp_get returns a const char string, so we
should declare the variable to match

@fix

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

Elm_Config: fix compilation when getuid() or geteuid() are not available

@fix

Elm_Config: add getpwent support, fix compilation on Windows

@fix

Elm_Config: remove Evil functions as home is anyway set on Windows

test: elm_prefs_cc - use eina_environment_tmp_get() instead of getenv("TMPDIR")

NEWS: Remove duplicated entries and the end

Fileselector: Use eina_environment_home_get() instead of getenv("HOME")

elm_container: Updated eo docs for elm_container

Summary:
Updated docs of elm_container.eo to the new format

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

Reviewed By: q66

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

elm_clock: convert elm_clock eo docs to new format

Summary:
Converted docs of elm_clock.eo to the new format

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

Reviewed By: q66

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

elementary: Remove debug comment

Summary: No real functional changes, just removing a debug printout

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

elm_color_item: Convert eo docs to new format

Summary:
Updated docs for elm_color_item.eo and elm_colorselector.eo files

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

Reviewed By: q66

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

elementary: Fix T2573: Extra blank line in Genlist Dnd Test

Summary: This fixes ticket T2573 where an extra new line was being
added after dropping an item in the Genlist DnD tests. There's really
no need to be appending a '\n' on each item here as that ends up
adding an extra blank item to the drop.

@fix

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

elm_calendar: Convert eo docs to new format

Summary:
Changed the docs of elm_calendar.eo to the new format

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

Reviewed By: q66

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

release: Update NEWS and bump version for 1.15.0-beta2 release

win: redo wayland opaque region setting

* convert opaque region theme part to SPACER
  - add rels for shaded/maximized state
* namespace opaque region theme part
* update opaque region only on pre-render
  - fixes race condition between render size and opaque size

ref T2575

elementary: Adjust shadow spacer on maximize

Summary: This fix adjusts the shadow spacer elementary when we
maximize/unmaximize a window. When in a maximized state, we do not
have to reserve space for the window shadow. This partially fixes
T2578. I say partially because there is still a little bit of blank
space around a maximized window.

@fix

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

tooltip: no need to show content object of tooltip

content object visibility will be controlled by tooltip edje object.

genlist : fix genlist dangling pointer crash in item select

Summary:
Fix genlist crash issue of T2553 which happened by dangling pointer
access after deletion in item_select.
after item_focused callback, user call deletion and item_select din't
cover those cases.
Resolves: T2553

@fix

Test Plan: elementary_test -> genlist focus -> choose Genlist Cear on Focus
then originally crash will be happened. This patch will solve that problem.

Scroller: Improvement in _key_action_move() calculations.

Summary:
If x coordinate is equal to 0, key action is no more
effective as EINA_FALSE is returned. This is creating problem
in looping. Looping will be done when x < 0, but as soon as
x == 0, _key_action_move starts returning EINA_FALSE. Same
thing applies to y coordinate and other extremities.

@fix

Signed-off-by: Umesh Tanwar <umesh.tanwar@samsung.com>
Test Plan: elementary_test -> Scroller -> Loop in X axis -> scroll left using left key(looping does not happen)

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

Reviewed By: SanghyeonLee

Subscribers: sachin.dev, SanghyeonLee, eagleeye

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

test_popup: include icon text in title-icon example

popup: fix title icon to show up.

Title icon of popup is not getting shown.
steps to reproduce:
1. elementary_test -to "popup"
2. click "popup bottom title + text + 3 buttons"

In the test popup example, the initial parent of icon is popup.
The title_icon is reset to null while adjusting elm parent from popup
to main_layout (elm_widget_sub_object_add which calls sub_object_del of popup)
on swallowing icon to main_layout.

Resolves: T2581
@fix

elementary: Properly fix wayland opaque region setting

Summary: This patch fixes opaque region setting under wayland in a
more proper/accurate way. This now sets the opaque region to be Just
what is visible (ie: only the dialog).

NB: This does not actually fix T2575 as that problem is inside E itself.

@fix

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

theme: popup - use aliases instead for subpopup groups

Use aliases for subpopup's content,buttons{1,2,3} groups
and it reduces theme file size by 4Kb (default.edj, 5680kb to 5676Kb).

elementary: Fix program signal for enable/disable of shadows

Summary: Elementary does not support toggling of shadows on windows,
but Enlightenment does so we need to use the proper program signal for
that.

@fix

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

elementary: Fix elementary wayland border opaque region calculations

Summary: This fixes T2575 more accurately (combined with previous
theme adjustments). Previous code would use a spacer object to help
calculate window opaque region. This is no longer necessary due to the
previous theme changes, and now calculates window opaque region to
coincide with what Enlightenment is expecting.

@fix

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

elementary: Update elementary wayland border theme to match E's

Summary: This fix addresses the issue of Elementary's Wayland border
theme not match close enough to E's. Now they are very much identical
with the exception of the program and signal names.

@fix

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

elementary: Tweak elm wayland border focus to Fix T2575

Summary: This adjusts the focus frame of Elementary applications to
more closely match what is done in X11. T2575 says that the focus
rectangle was larger than expected by a few pixels, so this slight
adjustment compensates for that.

@fix

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

elm_button: Converted docs of elm_button eo to new format

Summary:
Changed the docs of elm_bubble.eo and elm_button.eo to the new format

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

Reviewed By: q66

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

elm_box: convert eo docs to new format

Summary:
Converted elm_box.eo documentation to new format

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

Reviewed By: q66

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

glview: pass render_op flag to the child surface

Without this, apps can't set a GLView to be in COPY mode.
COPY mode allows the glview to "poke holes" in the backbuffer of
the window, without the need for other objects below (eg. a
transparent rectangle in COPY mode).

elm_bg: convert eo docs to new format

Summary:
Converted docs of elm_bg.eo to new format

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

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

widget_item: check view visibility in item_onscreen_is function.