genlist: code refactoring

declare/use const strings for internal signals

Gengrid: new signal: clicked,right with proper test

@feature

Genlist: do not fire clicked,right while dragging in progress

genlist: add "scroll" smart call

@feature

test_transit: Cast integer to pointer correctly to support 64 bit system.

test_glview: Refractored the shader/program log code.

Summary:
Added a function print_glerror_log to handle both shaders and programs
logs.

Test Plan: elementary_test -> glview

Reviewers: raster, jpeg, seoz

Reviewed By: seoz

Subscribers: anand.km, seoz

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

Improve -clicked- detection code by accept only if the mouse do not move too much

Too much in this case is a fixed 4px. As fingers do not have a right button I think
that taking into account finger size is not needed here.

Also someone argued about the signal name is not really correct...
should I change it to somethig like: clicked,secondary ?

Genlist: added the clicked,right event on items

@feature

Genlist: better text style for the default_style selected style

When the genlist item is selected it change the text effect from
shadow to glow, this has 2 side effects:
1. the text gets an unwanted 2px offset (because glow require more space, I think)
2. the text color the user has set usually looks ugly with the glow color

This commit change the style to always use a normal shadow, whenever the item
is selected or not. Only apply to the default_style item style.

list/genlist/gengrid: Fix memory leak.

Summary: @Fix

Reviewers: kimcinoo, SanghyeonLee, raster, seoz

Reviewed By: seoz

Subscribers: seoz

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

gengrid: Enhance realized_items_get performance.

gengrid, genlist, list, toolbar: Fix memory leak and enhance performance.

1. Fix memory leak by freeing eina_list.
2. Enhance performance by getting the item list when it is really
needed.

@fix

genlist: Return the correct pointer for object item.

No backport needed.

gengrid: Add missing "moved" callback call on item reorder by key.

This is a gengrid version of de75e6c0e852a2664c0ed022a6f79d6f431f64a0.

@fix

elm_genlist: add missing callback call

Summary:
Added missing "moved" callback call when reordering items with keyboard
"up" and "down" arrows

@fix

Reviewers: cedric, raster, seoz

Reviewed By: seoz

Subscribers: reutskiy.v.v

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

elm_toolbar: fix memory leak in _elm_toolbar_nearest_visible_item_get

@fix

Reviewers: cedric, seoz, Hermet, raster

Subscribers: reutskiy.v.v

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

transit: revise tween sample

genlist should ignore no-select items when moving with the keyboard

@fix

elm_transit: Append Elm_Transit_Tween_Mode sample to elementary_test

Summary:
A sample for elm_transit_tween mode which can be executed via
"Effects"-"Transit Tween Mode" button in elementary_test

Reviewers: Hermet

Subscribers: Hermet

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

add clipper for gadget popup shine

@fix

image: fix clipped image issue if x or y is less than zero

Summary:
After applying clipping patch about image on outside,
the width or height of the image be decreased
if x or y of an image is less than zero.
The way to calculate width/height has changed.

This fixes a side effect added in 2839881f37ea85b3469d8fd37cfaa4f9d67458fa

Reviewers: Hermet

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

elm_colorselector: Fix layout broken in ELM_COLORSELECTOR_ALL mode

Summary:
Fix layout broken in ELM_COLORSELECTOR_ALL mode
due to missing part for layout sizing evaluation

@fix

Reviewers: Hermet

Subscribers: Hermet

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

elm_transit: Support rotation keeping current map effect.

Summary:
Support rotation keeping current map effect.

Reviewers: Hermet

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

elm_transit: Support zoom keeping current map effect.

Summary:
Support zoom keeping current map effect.

Reviewers: Hermet

Reviewed By: Hermet

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

elm_colorselector: fix not intended duplication

Summary:
When colorselector widget is added, there are two align set command on palette_box part,
but that for picker part is omitted.
Apply elm_box_align_set function to picker part correctly.

@fix

Reviewers: Hermet

Reviewed By: Hermet

Subscribers: Hermet

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

image: Fix formatting for the latest commit 283988.

transit: revise sample

elm/image: Fix an unclipped issue about image bigger than object size

Summary:
When resizes an image
if "fill_inside" and "aspect_fixed" in Elm_Image_Data struct are "TRUE",
extra width or height go outside of the object area.
The width or height on outside will be clipped.
@fix

Reviewers: Hermet

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

modules: Make sure we have all linker flags setup for our modules.

We missed this here and test_mapo wanted eina_stringshare. Thanks to Andreas
Metzler for the report and initial patch.

@fix

Fix T1920

transit: clean up beizer test code...

oh please, this must be the last touch!

transit: print more detail control points message in the bezier sample

elm_widget: Pass Elm_Object_Item *item as *event_info in item->del_func.

Summary:
Pass Elm_Object_Item *item instead of Elm_Widget_Item_Data *item as
         *event_info in item->del_func to let *event_info be used.
         @fix

Reviewers: Hermet

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

transit: adjust control point position range in the bezier sample

transit: add bezier curve tween mode example

elm_label: Add an example handling "slide,end" signal

Summary:
Label widget example that handles "slide,end" signal
When each label ends its slide, it stops and makes next one slide.

Reviewers: Hermet

Subscribers: Hermet

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

elm_label: Add "slide,end" signal emission

Summary:
In label styles providing slide fuctions (slide_[long|short|bounce])
"slide,end" signal is emitted, when every slide ends.

@fix

Reviewers: raster, Hermet

Subscribers: raster, Hermet

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

transit: renamed to explicit examples.

transit: fix warning (different types comparing)

added in 8a46bf160273ecda8eac5b60c8c445b5e5a046e3

Transit: Support BEZIER CURVE

Summary:
Support BEZIER CURVE in elm transit
         @feature

Reviewers: seoz, raster, Hermet

Subscribers: raster, seoz

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

genlist: Fix dereference before null check.

This fixes coverity CID 1258636.
No backport needed.

elm_list : Resolved TODO in elm_list.c and declared corresponding macros in elm_macros.h

Summary:
As mentioned in TODO replaced expressions in if statement with macros in elm_macros.

Signed-off-by: kabeer khan <kabeer.khan@samsung.com>
Reviewers: seoz, raster

Reviewed By: raster

Subscribers: raster

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

Fix T1525 New Append/Prepend Gengrid Item cannot be seen in viewport

Reviewers: seoz, raster

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

elm - fix warnings added in f99be3a1d6bdc63153598b592b5437c33da44bf0

gengrid: Fix mirroring issue about key event

Summary:
Originally, key "Left" is mapped elm_gengrid_item_prev_get. But when gengrid is mirrored, key "Left" should remap elm_gengrid_item_next_get.
So I add more condition for checking mirroring.

1. reverse Left and Right key event
2. fix _elm_gengrid_item_edge_check logic
3. fix _item_show_region logic

Test Plan: elementary_test -> gengrid

Reviewers: SanghyeonLee

Subscribers: seoz

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

Conflicts:
src/lib/elm_gengrid.c

genlist: added reorder feature on key events

Summary: This feature allows user to reorder items using up/down keys.

Test Plan: elementary_test -to "Genlist Reorder Mode"

Reviewers: singh.amitesh, raster, seoz, SanghyeonLee

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

Conflicts:
src/bin/test_genlist.c
src/lib/elm_genlist_legacy.h

change elm_naviframe_itepm_pop_to api document

Summary:
Naviframe: modify elm_naviframe_item_pop_to api's document guide more specific to avoid confusion.

the elm_naviframe_item_pop_to api does not pop all items from the top to the target item.
It destory the items between the top and the target item, after that pop and destory the top item.
so, the descriptions of elm_naviframe_item_pop_to() is modified to avoid cunfusion like above.

Reviewers: Hermet, woohyun, seoz

Subscribers: seoz

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

test_check: deleted useless code.

Summary:
in this test sample, evas object size hint weight, align set API doesn't need to make view.
and line changed to code consistency.

Test Plan:
1. run elementary_test.
2. execution check test.
3. window resize.
   - some objects align to left side of window. in the other hands, some objects align to center of window.

Reviewers: seoz, Hermet

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

Gengrid: Focus highlight should move to nearest visible object.

Summary:
If item_loop is not enabled, the focus highlight should move to nearest visible object
with key events.

Test Plan: elementary_test -to "gengrid 2"

Reviewers: seoz, raster

Subscribers: sachin.dev, seoz

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

win: Add more description for automatic switch to alpha window by theme.

This is a documentation for e07061c5d5a9b1ec79278da82208ad486937b873.

win: Rename the internal variable to avoid confusion.

layout -> edje to explicitly say this is an edje object.
This avoids confusion between elm_layout and edje_object, thus makes
less human mistakes.

win: Support runtime alpha window switch on theme change.

Once the window alpha is on, it would not go back to non-alpha window
automatically.

- non-alpha theme -> alpha theme: switch to alpha window
- alpha theme -> non-alpha theme: stay in alpha window

@feature

win: Support elm_win style change and runtime theme change.

@feature

table: Refactor internal codes to keep consistency.

win: Fix wrong return value of window theme apply.

@fix

test_external.edc: pulse value have to be true for put in pulsing mode.

Summary: pulsing mode doen't work even though called elm_progressbar_pulse function in code , if pulse value is not true.

Test Plan:
1. run elementary_test
2. execution ExtProgressBar sample.
3. click 3rd button to start progressbar.

@fix

Reviewers: seoz, Hermet, woohyun, cedric

Subscribers: cedric

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

entry: Make documentation less confusing.

Entry can be used in different meanings. Use 'str' instead of 'entry' to
explicitly show the exact meaning of a parameter.

Multibutton entry - handle parent object null case.

Reviewers: seoz, Hermet

Subscribers: Hermet, seoz

Maniphest Tasks: T1379

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

elm - layout - allow elm.txt or elm.text.* parts to signal...

based on D1234 from Bora, but improved.

List: Focus highlight when loop is enable

Summary:
Implemented focus highlight feature in list for vertical and horizontal looping
@feature

Test Plan: elementary_test->"List Focus" & "List Focus Horizontal"

Reviewers: singh.amitesh, nirajkr, seoz, eagleeye, raster

Subscribers: sachin.dev

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

Conflicts:
src/lib/elm_list.c

DnD: fix callbacks call on many DnD operations.

A leave callback is invoked on the drop target of the previous DnD
operation, leading to inconsistency for applications.
The state of the dropable target was not resetted during drop.
This patch fixes that.

test_genlist: Remove unused value.

This fixes coverity CID 1258428.

genlist: Fix genlist crash issue when clearing genlist on item focus callback.

This fixes "genlist focus" test sample's crash issue.

@fix

test_genlist: Refactor genlist focus test.

- Add more sample items.
- Clear genlist on focus callback only for the specified item.
- Fix broken item index handling
- Remove unnecessary app data structure members.
- Free unused item class.

win: set config engine on higher priority than DISPLAY env var

DISPLAY environment variable may set regardless of Elementary/EFL.
Mostly user has no way to switch their intent engine such as "buffer"

Guarantee work application engine with their intended engine first.

@fix

README: Remove outdated version number

We have the version in configure.ac and NEWS where it gets updated for
releases. This occurrence gets forgotten to often on the other hand. Its
not needed here so we remove it.

elm_win: make it possible to specify alpha window in the theme directly.

elm_notify: added the notify's edje of popup smart member

Summary:
Summary: after popup layout refactoring, the popup object cant get evas event.

when popup using notify object block layout object will be covered with edje object.
it breaks evas smart member relationship.

Test Plan:
Test Plan:
1. added popup object.
2. key down event callback add for popup object.
3. run the elementary_test.
4. input some key on popup and check whether the registered function is executed or not.

Reviewers: eunue, Hermet, seoz

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

elm_widget_progressbar.c: added description of Progressbar data

Summary: added description.

Reviewers: Hermet, woohyun, seoz

Reviewed By: seoz

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

example/sphere_hunter: Clean up source code.

- Remove shadow variable.
- Don't call elm_init() twice. It is called automatically when you use
ELM_MAIN() macro.
- Don't set weight/align to swallowed objects.
- Set weight first before adding the object to window's resize object.
- Show window after creating all of its content.

config: improve doc.

elementary/config: improve doc.

.gitignore: ignore new example.

autotools: build Sphere Hunter with the rest of the examples.

sphere_hunter: remove warning.

autotools: fix indentation.

[Elementary/example]: Add new 3D game - Sphere Hunter.

Summary: This example shows the just using elementary with evas 3D.

Reviewers: cedric, raster, Hermet

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

elm - UNBREAK BUILD! (#undef... SOMETHING)

tooltips in windows make more attempts to not overlap with pointer

this prevents (in most cases) a race condition where the pointer will continually trigger a show-hide-show loop due to slow X message processing

tooltip now calcs position more effectively during ADJUST(FLIP)

tooltip windows now show themselves offscreen to force size calc before appearing

this improves initial tooltip placement and prevents tooltip warping due to incremental size recalcs after show

tooltip windows now set shape input rect in elm_win

this fixes a longstanding race condition where tooltip windowss could exist with a shape rectangle due to race conditions between applying window properties (eg. alpha) and reapplying the input rect

widget: Add/Fix internal documentations.

datetime: Fix edc sizing issue by putting fixed: 1 1.

Datetime edc is wrongly designed. Setting heuristic min value for the part is bad :(
This is a hotfix for the following error message when there is a button
for AMPM.

group elm/datetime/base/default has a non-fixed part 'field3'. Adding
'fixed: 1 1;' to source EDC may help. Continuing discarding faulty part.

@fix

slider: Clean up of the previous slider patch.

- This is a first clean up of 65240b5327e5ab1eb8f77835b4729444fe1f02df
- Fix wrong version for since tag.
- Fix indentation and remove trailing whitespaces.
- Make the code more readable.
- Remove a unused variable.

elm_cnp: decode escaped ASCII-encoded URI for dnd

Summary:
encoded URI is pasted in terminology on dnd with file which has blank in name
ex. Test Blank.avi -> Test%20Blank.avi

Therefore, decode it when we extract file uris from uri list by using efreet.
by the way, copy and paste code likely needs refactoring..

Reviewers: raster, cedric

Subscribers: billiob, seoz

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

Slider: Added APIs to set/get slider's indicator visibility mode.

Summary:
elm_config_slider_indicator_visible_mode_set
elm_config_slider_indicator_visible_mode_get

The patch will enable the slider's indicator to get

visible always
visible on focus
visible never
visible on slider value change

Reviewers: raster, seoz

Subscribers: sachin.dev

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

config: do not use elm_config from previous epoch.

Summary:
This commit fixes elm_config version mismatch occuring after b8606d93.
Config version in config source files has been set to proper values.
Patch deletes also previous code used to update config from previous epoch.

Current behavior when elm config is outdated:
1. if epoch is outdated - do not use user config - load system config instead.
   Leave it to user to recreate elm_config using elementary_config. Act like it
   was no config in user directory.
2. if generation is outdated and epoch is up-to-date - try to update user config.

Reviewers: raster, seoz, z.kosinski

Subscribers: seoz

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

Conflicts:
config/default/base.src.in
config/mobile/base.src.in
config/standard/base.src.in

elm config: add key bindings for elm_actionslider

Allow changing value of actionslider with keyboard. Make actionslider
focusable widget.

@feature

Conflicts:
config/default/base.src.in
config/mobile/base.src.in
config/standard/base.src.in

test_win_state: Add win lower and deiconify samples.

elm_transit: Apply evas_map_util_object_move_sync_set() to transit

Summary:
Apply evas_map_util_object_move_sync_set() to transit to keep map transformation after object move

please update efl library because evas_map_util_object_move_sync_set()'s been newly added.

Reviewers: Hermet, cedric

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

README: Update version number to latest.

Spotted by Lionel Orry. Thanks!

test_3d: Set indicator format explictly.

Indicator text is not visible if one does not set the indicator format.
This fixes T1867.

config: Refactor scrolling configuration dialog.

- Group thumb scroll configurations with frame.
- Move thumb scroll acceleration config below thumb scroll config.

test_photocam: Enhance photocam test.

- Remove wrong global variable usage. This becomes problematic when
you run multiple photocam test instances.
- Fix formatting.
- Add more comments.

test_photocam: Remove unnecessary data set.

config: Fixed null return.

Reviewers: cedric, singh.amitesh, seoz

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

test_menu: Fix wrong parent set.

Set elementary widget as an elementary widget's parent on its
creation.

widget: Add error message when the parent parameter was set wrongly.

This helps application developers debug their apps easier.

index, list, menu: Optimize eo item usage.

Remove unnecessary pointer reference.

gengrid, genlist: Optimize eo item usage.

Remove unnecessary pointer reference.