From: thiep ha <thiep.ha@samsung.com>

Subject: [E-devel] [Patch] [Elementary] ctxpopup - correct position
and size of ctxpopup

With ctxpopup, if it has many items and user uses -1 as parameter in
set direction priority function,
the ctxpopup position and size are calculated incorrectly.
Ex:
elm_ctxpopup_direction_priority_set(ctxpopup,
ELM_CTXPOPUP_DIRECTION_DOWN, ELM_CTXPOPUP_DIRECTION_UP, -1, -1);

SVN revision: 76505

diff --git a/ChangeLog b/ChangeLog
index ab6a8e3..7c97034 100644 (file)
--- a/ChangeLog
+++ b/ChangeLog
@@ -467,3 +467,8 @@
 
         * Add access features to calendar.
 
+
+2012-09-12  Thiep Ha
+
+        * Fix ctxpopup direction if -1 is uses as priority.
+

diff --git a/NEWS b/NEWS
index 09ff1e7..ae10460 100644 (file)
--- a/NEWS
+++ b/NEWS
@@ -21,6 +21,7 @@ Fixes:
    * Now elm_datetime_field_limit_set() can set year limits wihtout problems.
    * Fix re-order animation when it doesn't end correctly.
    * Fix popup to apply the same style to the notify sub-widget.
+   * Fix Ctxpopup direction if -1 priority used
    
 Removals:
 

diff --git a/src/lib/elc_ctxpopup.c b/src/lib/elc_ctxpopup.c
index 8125aa0..b307da5 100644 (file)
--- a/src/lib/elc_ctxpopup.c
+++ b/src/lib/elc_ctxpopup.c
@@ -200,7 +200,7 @@ _base_geometry_calc(Evas_Object *obj,
              break;
 
            default:
-             break;
+             continue;
           }
         break;
      }