On Sat, 7 Jan 2012 21:27:13 -0500 Youness Alaoui <[email protected]> said:
well i lean towards leaving it as is. i do see your point though. and well the previous behaviour was a helpful indicator visually. we don't have any way to detect orphaned objects since ownership falls back into the flat base for objects which is where they are all born and they don't require ownership at all. it'd be nice to be able to detect and warn here. :) maybe literally add an orphaned flag to an object - edje can set it once it has orphaned item and if at evas_render it finds orphaned objects it can complain? any further evas calls (show/hide/move/resize etc.) always clear this flag? > Understood, but I don't consider this behavior as a "tool" :p > I don't know about others who use unswallow, but in my case, the unswallow > was happening after an animation put the part in a 0x0 geometry state, so > it wasn't visible anyways. > Either way, what do you think the behavior should be? should I understand > from your comment that unswallow should not hide it ? > > On Sat, Jan 7, 2012 at 9:12 PM, Carsten Haitzler <[email protected]>wrote: > > > On Sat, 7 Jan 2012 20:50:26 -0500 Youness Alaoui > > <[email protected]> said: > > > > the real issue here is, leak-wise, that there is no tool to help u find > > such > > leaks as the object will never appear in any leak checker as it is tracked > > by > > evas anyway. :( (nb - unswallow == edje releases control of the object.) > > > > > @Gustavo: I understand your point about leaks but I'd expect a developer > > to > > > not need a visual aid for him to write proper code. Not leaking is > > standard > > > programing knowledge, so it's not about being "novice" in using the EFL. > > > My issue is that I don't want to destroy the objects, just hide them > > > (scrolling a list, I unswallow non visible objects and swallow the new > > > ones). I've had this bug for a while and I didn't understand that I had > > to > > > hide the objects, for me, the unswallow means it does not appear > > anymore. I > > > use an edje object, I swallow/unswallow objects to it, that's it, I don't > > > need to know that after I unswallow it will suddenly pop and look like an > > > artifact on screen or whatever. > > > Also, I never did a evas_object_show() on it, so there's no reason for me > > > to do the evas_object_hide(). One could argue that during the swallow, > > edje > > > should check what was the previous state (shown/hidden) and restore to > > that > > > state when you unswallow. > > > also, in my case, it would only be visible if I "cancel" the > > > animation/state change and that leaves the object in a weird state > > > (wherever it was left in the animation), but if I don't scroll too fast > > or > > > whatever, the part goes to a state of visible:0 (with 0x0 geometry) > > before > > > the unswallow happens, so it really wasn't an aid unless some weird race > > > condition happens then I get a weird artifact on screen. Talking as a > > > novice, this was clearly not an indication to hide the object but rather > > a > > > "wtf" moment getting me to hunt down the "bug" in edje. > > > > > > @Ivan, @Michael. I discussed this with Cedric before doing the commit, I > > > wanted to make sure whether or not I should do the hide in my code or in > > > edje directly. We discussed it and the conclusion was that it was not > > > documented, so it's "unexpected behavior". It shouldn't affect anyone > > > because I doubt someone unswallows an object then expects it to stay > > shown > > > on evas. There is a change in behavior, but it goes from "unexpected" to > > > "expected" so it's not a "major change". > > > You are right though, I will document it and put it in the > > changelog/news. > > > > > > @all: I don't mind reverting the change either way. I already hide it in > > my > > > code (since I'd like to stay compatible with the 1.1 release), so let's > > > discuss it, should the behavior be expected to hide the object, to leave > > it > > > in whatever state it was before the unswallow (which could be weird if it > > > happens during an animation), to restore the state to what it was before > > > the swallow was called ? any other suggestions? > > > > > > Thanks, > > > KaKaRoTo > > > > > > On Sat, Jan 7, 2012 at 4:35 PM, Cedric BAIL <[email protected]> wrote: > > > > > > > On Sat, Jan 7, 2012 at 7:21 PM, Michael Blumenkrantz > > > > <[email protected]> wrote: > > > > > On Sat, 7 Jan 2012 18:16:04 +0000 > > > > > Iván Briano (Sachiel) <[email protected]> wrote: > > > > > > > > > >> 2012/1/7 Gustavo Sverzut Barbieri <[email protected]>: > > > > >> > On Sat, Jan 7, 2012 at 9:39 AM, Enlightenment SVN > > > > >> > <[email protected]> wrote: > > > > >> >> Log: > > > > >> >> Edje: hide an object after unswallow > > > > >> >> Simply doing an unswallow would leave the object where it was > > in the > > > > >> >> evas, visible, but edje would not be handling it anymore. > > > > >> > > > > > >> > nah, you're supposed to do this in the application or edje user. > > Very > > > > >> > likely you'll delete the object, sometimes hide it. > > > > >> > > > > > >> > If you hide by default, novice will not see the object and will > > > > >> > leak... it's like a warning. > > > > >> > > > > > >> > > > > >> And if for some reason the change stays in, it's one of those > > > > >> very special things that deserve big bold letters in Changelog > > > > >> and NEWS files. > > > > >> > > > > > I'm pretty sure it's wrong to implement things that completely change > > > > expected > > > > > behavior like this in a non-major version... > > > > > > > > It's not changing any expected behaviour. When edje unswallow an > > > > object, you are not supposed to expect it in any particular state. Now > > > > you can expect it to be hidden. That's just what it does. It defines > > > > the output state, something that wasn't before. > > > > > > > > The point that make sense is the one that Gustavo raise. With previous > > > > behaviour, in some case you will notice that an object was not handle > > > > by edje anymore, because it was visibly lying around. I may be wrong, > > > > but if the part was not visible, unswallow would have issued an hidden > > > > object, like this patch does. So I don't know now, if the best is to > > > > force its visibility or to hide. > > > > -- > > > > Cedric BAIL > > > > > > > > > > > > > > ------------------------------------------------------------------------------ > > > > Ridiculously easy VDI. With Citrix VDI-in-a-Box, you don't need a > > complex > > > > infrastructure or vast IT resources to deliver seamless, secure access > > to > > > > virtual desktops. With this all-in-one solution, easily deploy virtual > > > > desktops for less than the cost of PCs and save 60% on VDI > > infrastructure > > > > costs. Try it free! http://p.sf.net/sfu/Citrix-VDIinabox > > > > _______________________________________________ > > > > enlightenment-devel mailing list > > > > [email protected] > > > > https://lists.sourceforge.net/lists/listinfo/enlightenment-devel > > > > > > > > > ------------------------------------------------------------------------------ > > > Ridiculously easy VDI. With Citrix VDI-in-a-Box, you don't need a complex > > > infrastructure or vast IT resources to deliver seamless, secure access to > > > virtual desktops. With this all-in-one solution, easily deploy virtual > > > desktops for less than the cost of PCs and save 60% on VDI infrastructure > > > costs. Try it free! http://p.sf.net/sfu/Citrix-VDIinabox > > > _______________________________________________ > > > enlightenment-devel mailing list > > > [email protected] > > > https://lists.sourceforge.net/lists/listinfo/enlightenment-devel > > > > > > > > > -- > > ------------- Codito, ergo sum - "I code, therefore I am" -------------- > > The Rasterman (Carsten Haitzler) [email protected] > > > > > ------------------------------------------------------------------------------ > Ridiculously easy VDI. With Citrix VDI-in-a-Box, you don't need a complex > infrastructure or vast IT resources to deliver seamless, secure access to > virtual desktops. With this all-in-one solution, easily deploy virtual > desktops for less than the cost of PCs and save 60% on VDI infrastructure > costs. Try it free! http://p.sf.net/sfu/Citrix-VDIinabox > _______________________________________________ > enlightenment-devel mailing list > [email protected] > https://lists.sourceforge.net/lists/listinfo/enlightenment-devel > -- ------------- Codito, ergo sum - "I code, therefore I am" -------------- The Rasterman (Carsten Haitzler) [email protected] ------------------------------------------------------------------------------ Ridiculously easy VDI. With Citrix VDI-in-a-Box, you don't need a complex infrastructure or vast IT resources to deliver seamless, secure access to virtual desktops. With this all-in-one solution, easily deploy virtual desktops for less than the cost of PCs and save 60% on VDI infrastructure costs. Try it free! http://p.sf.net/sfu/Citrix-VDIinabox _______________________________________________ enlightenment-devel mailing list [email protected] https://lists.sourceforge.net/lists/listinfo/enlightenment-devel
