Commit Graph

130 Commits

Author SHA1 Message Date
Daniel Juyung Seo f5d144e421 elm: Use eo_isa() to check object types instead of manual string comparison. 2015-03-10 22:52:29 +09:00
woochan lee d66a2555a8 multibuttonentry: registered object change for callback add.
Summary:
API changed from "edje_object_signal_callback_add()" to "elm_layout_signal_callback_add"
to following code refactoring.

Reviewers: seoz, woohyun, Jaehyun, Hermet

Reviewed By: Hermet

Differential Revision: https://phab.enlightenment.org/D2102
2015-03-10 17:19:47 +09:00
woochan lee 03af867581 multibuttonentry: Item width size set policy change.
Summary:
Item width max size set as 130 in edc before, i thinks that value doen't have any meaning,
so i changed the item max width size to follow container width size.

For modification, item size calc, box resize logic changed.

Test Plan:
1. Run elementary_test
2. Execute multibuttonentry sample.
3. Input some text for make item.

Reviewers: Hermet, seoz, woohyun, Jaehyun

Differential Revision: https://phab.enlightenment.org/D1970
2015-03-04 20:38:04 +09:00
Daniel Juyung Seo 5e4093babe multibuttonentry: Check return value of elm_layout_theme_set.
This fixes coverity CID 1086781.
2015-02-23 22:45:33 +09:00
woochan lee 684013d25e multibuttonentry: Item selected callback called twice redundantly.
Summary:
_current_item_change() called at _item_select() internally.
it means _current_item_changed() called two times in vain.
so that's why item select callback called two times.
@fix

Test Plan:
1. Run elementary_test
2. Excute multibuttonentry sample.
3. Added some items and click the item area.
4. Select signal called two times.

Reviewers: Hermet, woohyun, JackDanielZ, seoz

Reviewed By: seoz

Differential Revision: https://phab.enlightenment.org/D1955
2015-02-14 22:56:05 +09:00
woochan lee cf07ba6b5d multibuttonentry: refactoring(makes button object Elm_Item_Object instead of edje object)
Summary:
There was no consistency with another widget which has item object, basically widgets has Elm_Object_Item object for own's item object.
but multibuttonentry used edje object for item object before.

Test Plan:
1. Run elementary test.
2. Execute multibuttonentry.
3. Doing variety action on multibuttonentry widget.(add, delete, key event, focus etc...)

Reviewers: seoz, woohyun, Jaehyun, Hermet, JackDanielZ

Subscribers: herdsman, cedric

Differential Revision: https://phab.enlightenment.org/D1864
2015-02-10 14:22:43 +09:00
woochan lee 6e8f8629aa multibuttonentry: Delete duplicate running code.
Summary:
When layout clicked, "elm_widget_on_focus()" called with "entry_focus_in()" callback.
The "view update" and "focus set to entry" already call in those callbacks, so i deleted duplicate code and it also make focus issue when user click the entry to add item.
@fix

Test Plan:
Run elementary_test.

1. excute multibuttonentry.
2. click the entry area.
3. see the elm_widget_on_focus called twice.
   (layout callback function called view update -> elm_box_unpack called in view update ->
    entry lose focus -> on_focus_callback call again)

Reviewers: Hermet, seoz, Jaehyun_Cho, JackDanielZ

Differential Revision: https://phab.enlightenment.org/D1902
2015-02-03 11:38:21 +09:00
Amitesh Singh 2b40551740 multibuttonentry: use !strcmp() instead for comparison.
Reviewers: seoz

Subscribers: seoz

Differential Revision: https://phab.enlightenment.org/D1680
2014-11-21 13:38:12 +09:00
Daniel Zaoui 7e396432ef Object Items: replace most of del_pre functions with destructor.
This change is not simple because Elementary has not been built from the
first day to work with Eo. Code had to be adapted to fit the new design.

The del_pre that have not been replaced yet can return FALSE and
prevent deletion. For these classes, code modification has to be deeper
and will be done later.
2014-11-12 12:03:58 +02:00
Daniel Zaoui 05225b162e Object Items: remove data variable from widget item.
Now, data is stored in Eo layer via eo_key_data_set/get.

Two macros have been added to facilitate access.
2014-11-12 12:03:58 +02:00
Daniel Zaoui ef03bc098e Object Items: remove legacy APIs.
They are now generated by Eolian.
2014-11-12 12:03:58 +02:00
Daniel Zaoui b01d01528a Object Items: remove casts that are no more needed.
It includes casts to Eo, Elm_Object_Item and to specific items data.
This belongs to the cleaning phase of the object items porting to Eo.
2014-11-12 12:03:57 +02:00
Daniel Zaoui aaf2ecf9e9 Elm Multibuttonentry Item: Migrate to Eo 2014-11-12 12:03:55 +02:00
Lukasz Stanislawski 2ca1f8a75a atspi: properly include at-spi headers.
Summary:
Change requested by TAsn. Previuosly AT-SPI headers were kept private
and included directly into elementary source code. From now on,
AT-SPI headers can be included from Elementary.h public header, however
will be marked as beta APIs.

Commit includes following changes:
* include all atspi headers into new elm_interfaces.h header.
* marking all at-spi interfaces methods/properties as @protected.
* wrap all common headers with EFL_BETA_API_SUPPORT.
* make some common APIs visible in lib, by adding EAPI attribute
  (if someone decides to use beta APIs).

Test Plan: out-off tree build with gcc, g++

Reviewers: tasn

Reviewed By: tasn

Subscribers: seoz, q66, kuuko

Maniphest Tasks: T1721

Differential Revision: https://phab.enlightenment.org/D1528
2014-10-17 16:57:26 +01:00
Tom Hacohen 7629c147eb Eo related: Change according to recent changes in eo_add().
For more info check out a7560dbc61 in
the EFL tree.
2014-09-25 17:39:34 +01:00
Daniel Kolesa c159cb2ebc update according to Eolian syntax changes 2014-08-21 09:27:32 +01:00
Zbigniew Kosinski 9ce23f0004 atspi:Set roles for widgets
Reviewers: stanluk, m.jagiello, raster

Differential Revision: https://phab.enlightenment.org/D930
2014-06-11 11:07:16 +09:00
Tom Hacohen efa41c7d75 Elm: Update code to use the new class names generated by eolian. 2014-06-03 11:54:44 +01:00
Daniel Zaoui 1303a6dbec Eolian: fix names for elc widgets.
Even if the file names are elc_..., the class names are supposed to be
elm_...
2014-04-22 16:39:35 +03:00
Jaeun Choi 69ddd5b7f5 multibuttonentry: change key up callback into key down callback for consistency
Summary:
Every elementary widget's action is triggered by key down event except multibuttonentry.
There's no reason why focus move and deletion should happen on key up in multibuttonentry.
This patch modifies _layout_key_up_cb into _layout_key_down_cb.

Test Plan: None

Reviewers: raster, Hermet, seoz, jpeg

Reviewed By: Hermet

Differential Revision: https://phab.enlightenment.org/D755
2014-04-18 11:33:43 +09:00
Tom Hacohen 5a1d105c5e multibuttonentry: Adjusted to Eo2. 2014-04-10 09:34:59 +01:00
Daniel Zaoui 4b163aecbc Eolian: Integration of Multi Button Entry 2014-03-31 13:48:26 +03:00
Daniel Zaoui e2206a1a22 Eolian: Integration of Elm Widget 2014-03-17 09:03:56 +02:00
ChunEon Park f8d00ae0f2 elementary - override widget translation but work also the base widget's behavior.
Some widgets override the widget translation but it didn't inherit the base widget's function.
Becuase of it,The language changed won't be properly called in the widget tree.

Now it fixed it.
2014-01-23 20:20:43 +09:00
Daniel Juyung Seo 43533a2fb9 elm: Removed unnecessary null check for eina_stringshare_del. 2014-01-22 10:54:45 +09:00
Yakov Goldberg 3379c124df all widgets: change Eo API for "elm_widget_theme()" to "theme_apply"
There are elm_widget_theme/theme_set/theme_get functions.
    In Eolian these functions will be described as "theme" method and
    "theme" property. There is clash here.
    So add suffix "_apply" to Eo API for "elm_widget_theme".
2014-01-21 11:08:29 +02:00
Daniel Juyung Seo fd5c046dfe elm: Changed __UNUSED__ to EINA_UNUSED. 2014-01-06 23:33:57 +09:00
Daniel Juyung Seo 16f687b87a multibuttonentry, genlist, layout, toolbar: Fixed dereference of null
pointers which were reported by clang builg.

https://build.enlightenment.org/job/nightly_elm_clang_x86_64/lastSuccessfulBuild/artifact/scan-build/build/2014-01-05-1/report-XZo1hW.html#EndPath
https://build.enlightenment.org/job/nightly_elm_clang_x86_64/lastSuccessfulBuild/artifact/scan-build/build/2014-01-05-1/report-isAlRt.html#EndPath
https://build.enlightenment.org/job/nightly_elm_clang_x86_64/lastSuccessfulBuild/artifact/scan-build/build/2014-01-05-1/report-8Ascv8.html#EndPath
https://build.enlightenment.org/job/nightly_elm_clang_x86_64/lastSuccessfulBuild/artifact/scan-build/build/2014-01-05-1/report-gBkVuE.html#EndPath
2014-01-06 23:26:46 +09:00
Carsten Haitzler 99b270868d multibuttonentry - emit edje signal to oedje object not item struct! 2013-12-27 11:46:35 +09:00
Gustavo Lima Chaves 89da5dfc91 Reach mbe item on signal emit hook properly.
Fixes 360bba13c7c1004f7520e5e1ebe341c2e6323d2a mistake.
2013-12-26 16:03:24 -02:00
Gustavo Lima Chaves f6ad9605d6 New Elementary API -- elm_multibuttonentry_format_function_set().
Useful when one does not want the default multi button entry counter
item's format string.
2013-12-26 15:27:39 -02:00
Gustavo Lima Chaves 31f2530bd7 Multi button entry items now support custom parts on elm_object_item_text_[gs]et() calls.
I also removed a crazy "if (!label) return", since no other item text
hook on Elementary does that and I can't find a single reason for that
to be there.
2013-12-26 13:54:57 -02:00
Gustavo Lima Chaves e9f9e9a897 Now multi button entry items will happily listen to_object_item_signal_emit() calls. 2013-12-26 13:45:22 -02:00
Daniel Juyung Seo e63dd3110a elm: Unified eina critical manro to CRI.
Being annoyed by different types of eina critical macros - CRI, CRIT,
 CRITICAL -, I concluded to unify them to one. Discussed on IRC and
 finally, CRI was chosen to meet the consistency with other macros -
 ERR, WRN, INF, DBG - in terms of the number of characters.
If there is any missing bits, please let me know.
2013-12-26 13:03:55 +09:00
Daniel Juyung Seo 6d71a23c3a elm lib: removed all the unnecessary empty lines.
being pedantic.
2013-12-15 06:18:35 +09:00
WooHyun Jung cc4a41e641 Fix build warning. 2013-11-20 20:23:49 +09:00
WooHyun Jung 10775281a7 elementary/elc_multibuttonentry.c : Multibuttonentry doesn't need to call parents' on_focus. 2013-11-20 20:17:17 +09:00
Tom Hacohen 2f1e1554e5 Changed Eo class names to be consistent.
All the class names are now of the format: Elm_Type_Subtype_Extra
2013-11-07 11:44:22 +00:00
Daniel Juyung Seo d22076681b elm: Fixed formatting about smart_event check for code consistency.
- Made it look clearer.
- Removed unnecessary local variables.
- Keep the code sequence consistent.
2013-10-28 23:30:12 +09:00
Daniel Juyung Seo 7cca8036d7 elc_multibuttonentry.c: adopted ELM_WIDGET_DATA_GET_OR_RETURN macro. 2013-10-05 17:12:50 +09:00
Daniel Juyung Seo e756f97dae elm: Longwaited trailing white space removal. Clean elm up! 2013-09-30 22:15:01 +09:00
Cedric Bail 29af5e1e43 elementary: update to new eo_parent_get/set API. 2013-09-25 13:34:27 +09:00
ChunEon Park fe0f0a445e elementary/widgets - set parents before widget constructs their body.
This avoids unnecessary setups again when obj <-> sub has connected.
2013-08-27 00:29:45 +09:00
Carsten Haitzler e663eb4411 clean up some multibutton ugliness - this breaks theme api...
multibuttonentry really didnt follow elm conventions - it didnt use
namespacing for most singals/swallows - it dug into objects reading
object part geometry where it just should have calced min size and be
done. it did all sorts of other things it shouldnt/didnt need to.. so
i fixed that now... and dark theme has the support.
2013-08-23 23:33:06 +09:00
Daniel Juyung Seo c5b0d63838 fixed documentation for consistency. Gets -> Get.
Occurance of Get is super much higher than Gets.
So change Gets to Get for consistency.
2013-08-22 05:24:46 +09:00
Carsten Haitzler 59e8bc5a35 Fix elm to use key not keyname (so xmodmap works) 2013-07-31 15:16:41 +09:00
Shinwoo Kim c47ef1b7fe [access] use proper name _elm_access_info_get(); not _elm_access_object_get(); to get access information 2013-07-31 12:57:49 +09:00
Jaehwan Kim 5e7742d4c3 Fix the wrong parameter in multibuttonentry. 2013-07-23 18:48:09 +09:00
Daniel Juyung Seo ecf6eff9a4 elm: formatting fix for consistency. 2013-07-23 06:05:16 +09:00
Daniel Juyung Seo cc9a5b3f06 elc_multibuttonentry.c: fixed dereference null return value issue which was spotted by coverity. CID 1040023.
elm_widget_item_new() can return NULL on error so it's better to check NULL.
2013-07-06 12:07:51 +09:00