hi Andrzej

On Fri, May 14, 2010 at 04:40:28PM +0200, Andrzej Giniewicz wrote:
> So, first thing I noticed was that Refresh and Open Previous Scrap
> share same hotkey, it doen't sound well that way, right?

Indeed :-)

> Subject: [PATCH 1/2] Conflict in default hotkeys.
> 
> Default hotkeys for "Reload" and "Open Previous Scrap" are same (F5). Because 
> F5
> is more obvious for "Reload", F7 is assigned for "Open Previous Scrap" instead
> of F5.

Yes but I think it is important to have the "next" button to the right of
the "previous" button.  I pushed a corresponding commit.

> Subject: [PATCH 2/2] Brush settings window doesn't respect saved position and 
> visibility.

Yes I just noticed this bug myself this morning while working on something
else.  Thanks for the fix!

For future patches, please try to keep the first line a bit shorter (see git
log --oneline for examples).

-- 
Martin Renold

_______________________________________________
Mypaint-discuss mailing list
[email protected]
https://mail.gna.org/listinfo/mypaint-discuss

Reply via email to