From: Gary B. <bur...@ya...> - 2012-04-17 07:17:38
|
----- Forwarded Message ----- > From: Gary Burton <bur...@ya...> > To: Mathieu MD <mat...@gm...> > Cc: > Sent: Tuesday, 17 April 2012, 8:15 > Subject: Re: [Gramps-devel] [GraphView] Please provide feedback > > Hello Matthieu > > Nice to see someone else taking an interest in improving this plugin. > > I looked at your patch. It seems to contain changes for a number of different > things. I see some changes for adding a menu, some changes affecting the way the > graphs as drawn. I suggest breaking the patch up so that you have a separate > patch file for each new feature/change that you wish to add. This will make it > easier to review the changes and will be needed when committing changes to the > SVN repository. > > Thanks > > Gary > > ----- Original Message ----- >> From: Mathieu MD <mat...@gm...> >> To: Gramps Devel List <gra...@li...> >> Cc: >> Sent: Monday, 16 April 2012, 23:28 >> Subject: [Gramps-devel] [GraphView] Please provide feedback >> >> Hello, >> >> I work little by little on the Graphview addon, with the objective to >> get the bold line from the current person down/up to the home person. >> However its still too far from my skills, so I gave a smaller try by >> adding a configuration menu (for date and images). >> >> Could you give it a quick look for any suggestion? >> >> Bests, >> Mathieu >> > |