From kde-kimageshop  Wed Jan 05 15:40:50 2011
From: Silvio Heinrich <plassy () web ! de>
Date: Wed, 05 Jan 2011 15:40:50 +0000
To: kde-kimageshop
Subject: Re: Patch: Brush Outline with Crosshair
Message-Id: <4D249102.6060008 () web ! de>
X-MARC-Message: https://marc.info/?l=kde-kimageshop&m=129424208319857

On 01/04/2011 10:38 PM, LukasT.dev@gmail.com wrote:
> On Tuesday 04 January 2011 01:12:39 Silvio Heinrich wrote:
>> Now i send the separated patches starting with the crosshair thing.
>> The crosshair is now painted and the invisible cursor is used.
> I think the code looks fine. I applied the patch
> and I liked the option Brush outline and crosshair.
>
> I was surprised that it is painted, but it was probably
> due some problem with showing the crosshair cursor?
>
> For the long-term view, I would prefer maybe some different
> solution. I like the idea of adding the Brush outline option as check box in
> the ui so that it is available for every cursor. You could have
>
> Brush outline + Crosshair
> Brush outline + NoCursor
> Brush outline + Tool Icon
>
> If you don't want to work on the idea I provided, feel free to commit :)
>
> Ship it from me :)
> (Wait for one more ship it probably from somebody else)
>
> Thanks for the patch!
>
> Lukas
> _______________________________________________
> kimageshop mailing list
> kimageshop@kde.org
> https://mail.kde.org/mailman/listinfo/kimageshop
The problem with the crosshair cursor is that the brush outline will lag 
behind the cursor.
If you use a big brush, select the crosshair option and paint past, you 
should see that the stroke
lags behind the cursor (because the cursor should have a higher painting 
priority at any operating system).
And Cyrille said the cursor would be slightly off center, so i would 
prefer to not use the crosshair cursor
and paint it for the crosshair only option too.
But i like the idea with the check boxes. We could have the cursor shape 
options:
Tool Icon; Arrow; No Cursor; 3D Brush Model
and then two check boxes:
Show Brush Outline; Show Crosshair

Would this be acceptable? :D
_______________________________________________
kimageshop mailing list
kimageshop@kde.org
https://mail.kde.org/mailman/listinfo/kimageshop