1 Apr 2008 16:27
[sugar] [PATCH] add a secondary label and icon to palettes WAS Merging sugar-toolkit changes from tomeu repository
Eben Eliason <eben.eliason <at> gmail.com>
2008-04-01 14:27:53 GMT
2008-04-01 14:27:53 GMT
> Hmm, I guess our minds differ in how we read this logically. I follow > the same steps as you propose, but I mentally split them between "pre" > and "post" commands (the first two lines, and the second two lines), > and then construct all of a containers children in the middle. That > is, I "push" after child.props.foo = bar, set up all subchildren, and > "pop" with container.add(child) and child.show() where needed. Heh, upon further inspection I realize that I don't actually follow this very consistently. It's still the method that makes more sense to me, though, since it can help to visualize the hierarchy. I guess we could use an in-order traversal or something instead, while adhering to your code structure. - Eben
RSS Feed