Okay, I was able to add the patches and rework the code to include some of the changes that maxy suggested.
To answer an earlier question, yes my intention was to use the snapshot instead of the full undo. Using a separate layer ignored blending and erasers so I needed to affect the active layer. I was not able to use atan2 successfully, so I left the old code in. The vector and math functions are now outside the class. I changed the "unpythonic" code to use getattr, but now the menu xml has lower case headings... is this okay? Everything is a single commit. Attribution is given to David Gowers and David Revoy. https://gitorious.org/~optigon/mypaint/optigon-mypaint/commits/rj/linemode On Fri, Feb 17, 2012 at 2:24 AM, David Gowers (kampu) <[email protected]>wrote: > On Fri, Feb 17, 2012 at 5:14 PM, Richard Jones <[email protected]> > wrote: > > Thank you very much maxy for the detailed report. > > I had a chance to fix the merge conflicts and commited the new branch as > > rj/linemode. > > The icons can be added by merging with Deevad's icon branch. > > None of the code changes have been attempted yet. > > I was not able to apply savageorange's patch after fixing the merge > > conflicts. > > I'll continue as I can. > > I suspect I already fixed the merge conflicts (I'm running a number of > patches including yours on top of my master branch), which means that > the attached patches should work for you. > > HTH, > David ('savageorange' on mypaint forums) >
_______________________________________________ Mypaint-discuss mailing list [email protected] https://mail.gna.org/listinfo/mypaint-discuss
