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.

Eolian helper: sync eolian helper from the efl.

This should fix issues with missing directories and remove redundancy.

Elm: fix eolian_gen call on MSYS2

toolbar: item content unset function called when user content unset to toolbar item.

Summary:
It's related with https://phab.enlightenment.org/D2777

I missed doing on content_unset case.

Reviewers: Hermet

Reviewed By: Hermet

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

gengrid: remove memset to optimize items swapping via keys.

No need to call memset(0) as struct values are getting fill eventually.

elm_macro: add cast.

It is used by pixel calculation.
It should be changed to int.

elm_app_server: Convert eo docs to new format

Summary:
Converted docs of elm_app_server.eo and elm_app_server_view.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/D2816

elm_app_client: convert eo docs to new format

Summary:
Converted docs of elm_app_client.eo and elm_app_client_view.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/D2814

Entry: Fix behaviour of markup filters when setting entry text.

The issue was introduced in 28e6878f7cce36efae1438b5e032eb9c0878016a.
While this is a fix in the sense that it reverts to the previously
defined behaviour (defined both in docs and code), the previous
behaviour was stupid and inconsistent. So now we are back to stupid and
inconsistent.

I can't fix this further, because unfortunately, the fix requires
intrusive changes that are just too dangerous to make this late in the
release cycle. Luckily for us, the issues only surface in a very
specific case that is quite uncommon. In order to trigger the bug you
need to set a very long piece of text on an entry that has text filters
set.

Fixes T2561

@fix

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

test_win_state: remove commented code and newline

slider: remove useless code

toolbar: Check return value of elm_layout_theme_set.

colorselector: fix compilation warnings.

color_class: Check return value of elm_layout_theme_set

Elm.Colorselector: Fix crash in infinite callback loop

In most cases the code would not crash (I can't understand why),
but in some cases Eo would complain about call stack overflow.
Yep, that's a lot of recursive calls.

The mouse_down callback was trying to cancel its triggering
event and then send again the same mouse_down event. Why? I guess
because it was easier that also having a callback for move (drag
the colorselector arrows). Ugly...

This widget is in dire need for some love.

Window: Set urgent flag only when not already set

cat <binary_file_with_lots_of_BEL> in terminology would slow down
everything because of the urgent flag (and talking to X).

theme - make square more opaque

popup: fix timeout cb.

popup: fix block,clicked cb

NEWS: Remove reverted commits and redundant logs.

Thanks to Amitesh for spotting this.

Elm Entry: Fix context menu dismissal

The wrong event was used during the port from smart callbacks to eo
callbacks. Now the correct ELM_HOVERSEL_EVENT_DISMISSED event is used.

theme adjust - replace blurry grey haloed black box with nice rect+shad

popup: add missing AT-SPI2 MODAL state

slider: compare slider values elaborately.

because of the double type nature,
the comparison is not correct.

we decide the value changes with range of double epsilon to avoid this.

@fix

image: fix a crash, 0 divide problem.

if the gif is invalid animated image,
the frame count could be 0.

@fix

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

elm: Fix distcheck.

elm: update new-style docs across elm so they validate right

po: Updated po.

elm_general: move definition of enum Elm.Wrap.Type to elm_general.eot

move definition of enum Elm.Wrap.Type from elm_entry.eo to elm_general.eot
And import elm_general.eot into elm_entry.eo, elm_label.eo, elm_popup.eo

elm_hover: Move enums to elm_hover.eo

Move enums from elm_hover_common.h to elm_hover.eo in order to make them
more accessible for bindings. Delete elm_hover_common.h.

elm_label: Move enums to elm_label.eo

Move enums from elm_label_common.h to elm_label.eo in order to make them
more accessible for bindings. Delete elm_label_common.h.

elm_list: Move enums to elm_list.eo

Move enums from elm_list_common.h to elm_list.eo in order to make them
more accessible for bindings. Delete elm_list_common.h.

elm_genlist: move enums to elm_genlist_item.eo

Move enums from elm_genlist_common.h to elm_genlist_item.eo in order to make them
more accessible for bindings.

elm_gengrid: move enums to elm_gengrid_item.eo

Move enums from elm_gengrid_common.h to elm_gengrid_item.eo in order to make them
more accessible for bindings.

elm_entry: moved part of enums to elm_entry.eo

Move part of enums from elm_entry_common.h to elm_entry.eo in order to make them
more accessible for bindings.

elm_dayselector: Move enums to elm_dayselector.eo

Move enums from elm_dayselector_common.h to elm_dayselector.eo in order to make them
more accessible for bindings. Delete elm_dayselector_common.h.

elm_datetime: Move enums to elm_datetime.eo

Move enums from elm_datetime_common.h to elm_datetime.eo in order to make them
more accessible for bindings.

elm_colorselector: Move enums to elm_colorselector.eo

Move enums from elm_colorselector_common.h to elm_colorselector.eo in order to make them
more accessible for bindings.

elm_clock: Move enums to elm_clock.eo

Move enums from elm_clock_common.h to elm_clock.eo in order to make them
more accessible for bindings.

elm_calendar: Move enums to elm_calendar.eo

Move enums from elm_calendar_common.h to elm_calendar.eo in order to make them
more accessible for bindings.

elm_bubble: Move enums to elm_bubble.eo

Move enums from elm_bubble_common.h to elm_bubble.eo in order to make them
more accessible for bindings. Delete elm_bubble_common.h.

elm_bg: Move enums to elm_bg.eo

Move enums from elm_bg_common.h to elm_bg.eo in order to make them
more accessible for bindings. Delete elm_bg_common.h.