<!DOCTYPE HTML PUBLIC "-//W3C//DTD HTML 4.01 Transitional//EN">
<html>
<head>
<meta content="text/html; charset=ISO-8859-1"
http-equiv="Content-Type">
<title></title>
</head>
<body bgcolor="#ffffff" text="#000000">
On 07/13/2011 07:59 PM, Sven Langkamp wrote:
<blockquote
cite="mid:CAAmsBfnHVnM7-ktMaPSh8q2VbBEKamhn=1E9Cmft=m5M=BvKMw@mail.gmail.com"
type="cite">
<div class="gmail_quote">On Wed, Jul 13, 2011 at 4:30 PM,
Boudewijn Rempt <span dir="ltr"><<a moz-do-not-send="true"
href="mailto:boud@valdyas.org">boud@valdyas.org</a>></span>
wrote:<br>
<blockquote class="gmail_quote" style="margin: 0pt 0pt 0pt
0.8ex; border-left: 1px solid rgb(204, 204, 204);
padding-left: 1ex;">
I tried to paint today, and it was still quite painful, and
not just because my digital painting skills suck, it's because
we've got quite a few bugs left. Here's a few that I found,
and that Ramon Miranda noticed on irc today...<br>
<br>
brush editor/paintop popup<br>
<br>
* when changing paintop in the popup the name of the preset
should be cleared<br>
</blockquote>
<div>Fixed that locally. Will commit soon. <br>
</div>
</div>
</blockquote>
<br>
Uff... sorry I reverted your commit. It makes everything worse
instead of better.<br>
I think you haven't tested it with a tablet because every time I
switch to the mouse or the eraser of the pen the preset name and the
scratchpad is cleared.<br>
I see no point in clearing the name and the scratchpad
automatically, the user can clear it by himself if he want.<br>
And the current behavior has two really bad drawbacks, thats why I
reverted it.<br>
<br>
1. The original intention to have the name of the last loaded preset
in the edit field was to get a fast work flow with the presets. It's
maybe not obvious anymore because the most people here already have
the presets they normally work with but any newcomer who wants to
create his own presets will likely work like this:<br>
load a preset -> paint a bit -> adjust the preset settings a
bit -> save preset -> paint a bit -> adjust the preset
settings a bit -> save preset -> paint a bit -> and so on
-> and so on.<br>
When the name is cleared every time while painting it is likely to
slow down the workflow and annoy the user because he has to retype
the name again all the time (especially annoying with long names).<br>
<br>
2. Kind of the same problem for the scratch pad plus one addition.
It is a lot harder to make icons for the presets now. For example if
you want to make a preset for an other paintop as the pixel brush
and want to write something something in the box it was possible to
switch to the pixel brush write in the box and go back to the
paintop you wanted to save and save it. This don't work anymore
because the scratchpad is cleared.<br>
<br>
This is why I think that the scratchpad and the preset name should
only be cleared explicitly by the user with the only exception when
you load a saved preset.<br>
Than it should overwrite the name in the edit field and the icon box
in the scratch pad.<br>
<br>
</body>
</html>