Adjust "context_menu_disabled" configuration. 60/70660/1
authorDave Andreoli <dave@gurumeditation.it>
Thu, 21 Jan 2016 21:21:30 +0000 (22:21 +0100)
committerYoungbok Shin <youngb.shin@samsung.com>
Fri, 20 May 2016 04:53:28 +0000 (13:53 +0900)
This @fix three (yes 3!) errors from commit cafda5928f9bf8116564219dc03d12c3a6e131b3

1. the initial value in the standard config is wrong, causing ctx menu to be disabled for everyone!
2. the argument of the api should be named "disabled" (like all the other disable API in this context)
3. (minor) fixed doc for elm_config_context_menu_disabled_set

NOTE: there are no configuration gui for this option, if you cleared your elm config recently you probably now have the ctx menu disabled everywhere, only solution is to wipe your config (to let elm create a new correct one)

Big SPANK to the author, the commiter and all the reviewers of D2463!!!

Change-Id: I92ddb0659d64d8dd55150aa22e7d8ab3a61bc43c

config/standard/base.src.in
src/lib/elm_config.c
src/lib/elm_config.h

index 4ddc4ec..6001829 100644 (file)
@@ -55,7 +55,7 @@ group "Elm_Config" struct {
   value "longpress_timeout" double: 1.0;
   value "effect_enable" uchar: 1;
   value "desktop_entry" uchar: 1;
-  value "context_menu_disabled" uchar: 1;
+  value "context_menu_disabled" uchar: 0;
   value "password_show_last" uchar: 0;
   value "password_show_last_timeout" double: 2.0;
   value "glayer_zoom_finger_enable" uchar: 1;
index 12ec6e6..fd5b058 100644 (file)
@@ -2910,9 +2910,9 @@ elm_config_context_menu_disabled_get(void)
 }
 
 EAPI void
-elm_config_context_menu_disabled_set(Eina_Bool enabled)
+elm_config_context_menu_disabled_set(Eina_Bool disabled)
 {
-       _elm_config->context_menu_disabled = !!enabled;
+       _elm_config->context_menu_disabled = !!disabled;
 }
 
 EAPI void
index 988d636..0354df3 100644 (file)
@@ -225,15 +225,15 @@ EAPI void         elm_config_scroll_page_scroll_friction_set(double friction);
 EAPI Eina_Bool elm_config_context_menu_disabled_get(void);
 
 /**
- * Set enable status of context menu disabled.
+ * Enable or disable the context menu in entries.
  *
- * @param enabled enable context menu if @c EINA_TRUE, disable otherwise
+ * @param disabled disable context menu if @c EINA_TRUE, enable otherwise
  *
- * @see elm_config_focus_auto_scroll_bring_in_enabled_get()
+ * @see elm_config_context_menu_disabled_get()
  * @ingroup Entry
  * @since 1.17
  */
-EAPI void elm_config_context_menu_disabled_set(Eina_Bool enabled);
+EAPI void elm_config_context_menu_disabled_set(Eina_Bool disabled);
 
 /**
  * Get the amount of inertia a scroller will impose at region bring