jpeg pushed a commit to branch master.

http://git.enlightenment.org/core/efl.git/commit/?id=9df368588465b37e8c6d775aeecded2173b1884f

commit 9df368588465b37e8c6d775aeecded2173b1884f
Author: JinYong Park <[email protected]>
Date:   Fri May 12 14:57:01 2017 +0900

    popup: enable to set last_button_number to 0
    
    Summary:
    If last added button is removed, any statements to set last_button_number 
in for loop doesn't be executed.
    So last button number has previous value, not 0.
    Because of that, action area is not removed, but that area has zero height, 
so it doesn't be shown.
    
    @fix
    
    Reviewers: herb, jpeg, cedric, singh.amitesh, raster
    
    Reviewed By: jpeg
    
    Subscribers: woohyun
    
    Differential Revision: https://phab.enlightenment.org/D4870
---
 src/lib/elementary/elc_popup.c | 2 ++
 1 file changed, 2 insertions(+)

diff --git a/src/lib/elementary/elc_popup.c b/src/lib/elementary/elc_popup.c
index 7fa35ad..a24c1c0 100644
--- a/src/lib/elementary/elc_popup.c
+++ b/src/lib/elementary/elc_popup.c
@@ -613,6 +613,8 @@ _button_remove(Evas_Object *obj,
 
    ELM_SAFE_FREE(sd->buttons[pos], free);
 
+   sd->last_button_number = 0;
+
    for (i = ELM_POPUP_ACTION_BUTTON_MAX - 1; i >= 0; i--)
      {
         if (sd->buttons[i])

-- 


Reply via email to