----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: http://patches.synfig.org/r/27/#review80 -----------------------------------------------------------
Ship it! It looks ok to me. Tested and works fine. It would be interesting that after apply this patch the icon shown in the following dialog after this recovery one were a "Exclamation" one or a "Warning" one, instead of a "Forbidden" one. Please Ship it. - Carlos On 2009-06-20 04:12:55.133328, Gerco Ballintijn wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > http://patches.synfig.org/r/27/ > ----------------------------------------------------------- > > (Updated 2009-06-20 04:12:55.133328) > > > Review request for Synfig. > > > Summary > ------- > > This patch is an extended version of the patch that Bombe added added to > SourceForge.net on 2009-02-15 (#2602803: "Patch: Improve UI on crash recovery > dialog."). I cleaned up the code a bit by making it more consistent with > surrounding code. Otherwise, it still more or less the same. The attached > images show the effect of the patch. > > > Diffs > ----- > > synfig-studio/trunk/src/gtkmm/app.cpp 2382 > synfig-studio/trunk/src/gtkmm/canvasview.cpp 2382 > synfig-studio/trunk/src/synfigapp/uimanager.h 2382 > synfig-studio/trunk/src/synfigapp/uimanager.cpp 2382 > > Diff: http://patches.synfig.org/r/27/diff > > > Testing > ------- > > Compiles, runs, and shows the improved dialog. > > > Screenshots > ----------- > > Old recovery dialog > http://patches.synfig.org/r/27/s/3/ > New recovery dialog > http://patches.synfig.org/r/27/s/4/ > > > Thanks, > > Gerco > > ------------------------------------------------------------------------------ Are you an open source citizen? Join us for the Open Source Bridge conference! Portland, OR, June 17-19. Two days of sessions, one day of unconference: $250. Need another reason to go? 24-hour hacker lounge. Register today! http://ad.doubleclick.net/clk;215844324;13503038;v?http://opensourcebridge.org _______________________________________________ Synfig-devl mailing list Synfig-devl@lists.sourceforge.net https://lists.sourceforge.net/lists/listinfo/synfig-devl
