Re: [Kicad-developers] eeschema selection appearance

2019-11-20 Thread Jonatan Liljedahl
On Wed, Nov 20, 2019 at 5:55 PM Seth Hillbrand  wrote:

> On 11/20/19 7:11 AM, Jonatan Liljedahl wrote:
>
> Ok, I'll look into making it configurable. However, I find the current
> default appearance hard to use, so maybe one could consider changing
> the defaults in that case?
>
> Everyone likes to have their preferences as the default.  This is not a
> useful line of discussion for the project.  Once the options are
> configurable, it shouldn't be an issue.
>

My point was that maybe it's not only a matter of personal preference, but
usability and good UX design. Options are good, but that's no reason not
trying to find the best possible defaults for those options. Of course, if
it's only me that thinks my proposal improves things in general, let me
know :)

> Additionally, I changed the shadow width algo constants as follows, to
> avoid the feeling of the selection shadow changing size drastically as
> you zoom:
>
> Please move this idea into a separate patch and propose it.  The selection
> highlight width works well at all zoom levels for me but if this idea helps
> some people without negatively impacting the current situation, there
> should be no issues.  Please keep this distinct from the options patch,
> however.
>

Ok, will do! BTW, this is an example where I don't think a user setting
makes sense, which might be called something like "Amount of zoom-level
impact on selection shadow thickness". Better to just find the values for
those constants that looks good and coherent.

-- 
/Jonatan
http://kymatica.com
___
Mailing list: https://launchpad.net/~kicad-developers
Post to : kicad-developers@lists.launchpad.net
Unsubscribe : https://launchpad.net/~kicad-developers
More help   : https://help.launchpad.net/ListHelp


Re: [Kicad-developers] eeschema selection appearance

2019-11-20 Thread Seth Hillbrand

On 11/20/19 7:11 AM, Jonatan Liljedahl wrote:

Ok, I'll look into making it configurable. However, I find the current
default appearance hard to use, so maybe one could consider changing
the defaults in that case?
Everyone likes to have their preferences as the default.  This is not a 
useful line of discussion for the project.  Once the options are 
configurable, it shouldn't be an issue.



Additionally, I changed the shadow width algo constants as follows, to
avoid the feeling of the selection shadow changing size drastically as
you zoom:
Please move this idea into a separate patch and propose it.  The 
selection highlight width works well at all zoom levels for me but if 
this idea helps some people without negatively impacting the current 
situation, there should be no issues.  Please keep this distinct from 
the options patch, however.




So now the question is at what granularity all this should be
configurable? Perhaps:

- selection draw child-items: bool
- selection thickness: float
- selection color (already there)

Those options make sense to me.

-Seth
--
KiCad Services Corporation KiCad Services Corporation Logo
Seth Hillbrand
*Lead Developer*
+1-530-302-5483‬ 
Davis, CA
www.kipro-pcb.com  i...@kipro-pcb.com 

https://twitter.com/KiProEDA  
https://www.linkedin.com/company/kicad 



___
Mailing list: https://launchpad.net/~kicad-developers
Post to : kicad-developers@lists.launchpad.net
Unsubscribe : https://launchpad.net/~kicad-developers
More help   : https://help.launchpad.net/ListHelp


Re: [Kicad-developers] eeschema selection appearance

2019-11-20 Thread Jonatan Liljedahl
Ok, I'll look into making it configurable. However, I find the current
default appearance hard to use, so maybe one could consider changing
the defaults in that case?

Not only does it make the text easily legible, but also the whole
symbol less cluttered when selected. You can still select individual
text fields as expected, but only the symbol and its pins are drawn as
selected when the whole component is selected.

These are the parts I've disabled for drawing of the selection shadow:
- pin labels/names
- component fields
- subsheet pins
- subsheet title/file

Additionally, I changed the shadow width algo constants as follows, to
avoid the feeling of the selection shadow changing size drastically as
you zoom:

float SCH_PAINTER::getShadowWidth()
{
const MATRIX3x3D& matrix = m_gal->GetScreenWorldMatrix();

// For best visuals the selection width must be a cross between
the zoom level and the
// default line width.
return (float) ( ( fabs( matrix.GetScale().x * 1.0 ) +
GetDefaultLineThickness() ) * 4.0 );
}

So now the question is at what granularity all this should be
configurable? Perhaps:

- selection draw child-items: bool
- selection thickness: float
- selection color (already there)

Cheers

On Wed, Nov 20, 2019 at 3:29 PM Ian McInerney  wrote:
>
> I'm on the fence about the text highlighting, on the one hand not doing it 
> does make it so the text is still easily legible when selected, but on the 
> other it can be nice to show that it is part of the selected symbol. I think 
> this would definitely be a case where making it a configurable option would 
> allow people to experiment and see what they prefer.
>
> -Ian
>
> On Wed, Nov 20, 2019 at 2:18 PM Seth Hillbrand  wrote:
>>
>> On 2019-11-20 05:48, Jonatan Liljedahl wrote:
>> > Hi,
>> >
>> > I'm tweaking the appearance of the new selection, what do you think?
>> > Except a change of color, transparency and width, it also skips
>> > drawing the fields and pin labels of components. I think it gives a
>> > much cleaner look with less clutter.
>>
>> This will always be a matter of opinion.  Right now the color is
>> configurable.  If you'd like to add additional configurable parameters
>> to the selection, you should place them in the Eeschema preferences and
>> allow the user to choose them.  Then post the patch for review.
>>
>> Otherwise, we'll end up bike shedding on this which would be nice to
>> avoid.
>>
>> Best-
>> Seth
>>
>>
>> Seth Hillbrand
>> KiCad Services Corporation
>> https://www.kipro-pcb.com
>> +1 530 302 5483 | +1 212 603 9372
>>
>> ___
>> Mailing list: https://launchpad.net/~kicad-developers
>> Post to : kicad-developers@lists.launchpad.net
>> Unsubscribe : https://launchpad.net/~kicad-developers
>> More help   : https://help.launchpad.net/ListHelp
>
> ___
> Mailing list: https://launchpad.net/~kicad-developers
> Post to : kicad-developers@lists.launchpad.net
> Unsubscribe : https://launchpad.net/~kicad-developers
> More help   : https://help.launchpad.net/ListHelp



-- 
/Jonatan
http://kymatica.com

___
Mailing list: https://launchpad.net/~kicad-developers
Post to : kicad-developers@lists.launchpad.net
Unsubscribe : https://launchpad.net/~kicad-developers
More help   : https://help.launchpad.net/ListHelp


Re: [Kicad-developers] eeschema selection appearance

2019-11-20 Thread Ian McInerney
I'm on the fence about the text highlighting, on the one hand not doing it
does make it so the text is still easily legible when selected, but on the
other it can be nice to show that it is part of the selected symbol. I
think this would definitely be a case where making it a configurable option
would allow people to experiment and see what they prefer.

-Ian

On Wed, Nov 20, 2019 at 2:18 PM Seth Hillbrand  wrote:

> On 2019-11-20 05:48, Jonatan Liljedahl wrote:
> > Hi,
> >
> > I'm tweaking the appearance of the new selection, what do you think?
> > Except a change of color, transparency and width, it also skips
> > drawing the fields and pin labels of components. I think it gives a
> > much cleaner look with less clutter.
>
> This will always be a matter of opinion.  Right now the color is
> configurable.  If you'd like to add additional configurable parameters
> to the selection, you should place them in the Eeschema preferences and
> allow the user to choose them.  Then post the patch for review.
>
> Otherwise, we'll end up bike shedding on this which would be nice to
> avoid.
>
> Best-
> Seth
>
>
> Seth Hillbrand
> KiCad Services Corporation
> https://www.kipro-pcb.com
> +1 530 302 5483 | +1 212 603 9372
>
> ___
> Mailing list: https://launchpad.net/~kicad-developers
> Post to : kicad-developers@lists.launchpad.net
> Unsubscribe : https://launchpad.net/~kicad-developers
> More help   : https://help.launchpad.net/ListHelp
>
___
Mailing list: https://launchpad.net/~kicad-developers
Post to : kicad-developers@lists.launchpad.net
Unsubscribe : https://launchpad.net/~kicad-developers
More help   : https://help.launchpad.net/ListHelp


Re: [Kicad-developers] eeschema selection appearance

2019-11-20 Thread Seth Hillbrand

On 2019-11-20 05:48, Jonatan Liljedahl wrote:

Hi,

I'm tweaking the appearance of the new selection, what do you think?
Except a change of color, transparency and width, it also skips
drawing the fields and pin labels of components. I think it gives a
much cleaner look with less clutter.


This will always be a matter of opinion.  Right now the color is 
configurable.  If you'd like to add additional configurable parameters 
to the selection, you should place them in the Eeschema preferences and 
allow the user to choose them.  Then post the patch for review.


Otherwise, we'll end up bike shedding on this which would be nice to 
avoid.


Best-
Seth


Seth Hillbrand
KiCad Services Corporation
https://www.kipro-pcb.com
+1 530 302 5483 | +1 212 603 9372

___
Mailing list: https://launchpad.net/~kicad-developers
Post to : kicad-developers@lists.launchpad.net
Unsubscribe : https://launchpad.net/~kicad-developers
More help   : https://help.launchpad.net/ListHelp


[Kicad-developers] eeschema selection appearance

2019-11-20 Thread Jonatan Liljedahl
Hi,

I'm tweaking the appearance of the new selection, what do you think?
Except a change of color, transparency and width, it also skips
drawing the fields and pin labels of components. I think it gives a
much cleaner look with less clutter.

-- 
/Jonatan
http://kymatica.com
___
Mailing list: https://launchpad.net/~kicad-developers
Post to : kicad-developers@lists.launchpad.net
Unsubscribe : https://launchpad.net/~kicad-developers
More help   : https://help.launchpad.net/ListHelp