Dear cinoo,
Thank for your patch.
But I found something wrong there.
1. //if user does not use dragable part.
........
I guess you can remove these lines which are vestige for theme compatibility.
2. 558: edje_object_signal_emit(wd->arrow, "elm,state,bottom", "elm");
Spank! it is called twice!
3. How about renaming arrow swallow part with lowercase letter for consistency?
"elm.swallow.arrowRight" -> "elm.swallow.arrow_right" or ...
4. please check the arrow position calculation again. it works incorrectly.
Could you check the above cases?
Thanks.
------------------------------------
Let's run together for the best moment!
-Regards, Hermet-
-----Original Message-----
From: "cnook"<kimci...@gmail.com>
To: enlightenment-devel@lists.sourceforge.net
Cc:
Sent: 11-05-27(금) 19:08:44
Subject: [E-devel] [Patch][CtxPopup] Add Arrow on the drag-able area of
BaseDear All,
Hi~
If CtxPopup has elm_slider for content, the elm_slider indicator is covered
by CtxPopup arrow.
So, I have made a patch for resolving this issue. As the title the arrow
will be swallowed in the base.
Even though user uses his/her own style, the attached patch will work
properly.
Sincerely,
Shinwoo Kim.
------------------------------------------------------------------------------
Simplify data backup and recovery for your virtual environment with vRanger.
Installation's a snap, and flexible recovery options mean your data is safe,
secure and there when you need it. Data protection magic?
Nope - It's vRanger. Get your free trial download today.
http://p.sf.net/sfu/quest-sfdev2dev
_______________________________________________
enlightenment-devel mailing list
enlightenment-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/enlightenment-devel
