← Back to team overview

kicad-developers team mailing list archive

Re: New zoom code musings.

 

Dick Hollenbeck a écrit :
>
> Vesa Solonen wrote:
> > On Thu, 5 Feb 2009, Dick Hollenbeck wrote:
> >
> >
> >> Your zoom patch for pcbnew makes the PCBNEW program too slow to use 
> when
> >> I have a delay in the drawing due to a large number of zones.
> >>
> >
> > Yes, that's very noticeable problem. Anyway the real solution is 
> somewhere
> > else than on zoom granularity, like threading for rendering...
> >
> >
> >> Please try out this array for pcbnew and let me know if your feelings
> >> would be hurt if we used it instead. The problem again is that it takes
> >> too many steps to zoom in.
> >>
> >
> > How about adding a 'turbo key' for zooming with larger step, like 
> SHIFT +
> > zoom? Your zoom series is ok too, but I'd like to keep even powers 
> of two
> > zooms along the finer steps. That would mean sqrt(2) factor. Anyway 
> this
> > should be configurable someway as otherwise we can not please every 
> one :)
> >
> >
> >> I have not tried eeschema. Drawing is faster there, but there could be
> >> some advantage in making these similar.
> >>
> >
> > Maybe less steps is a good compromise until drawing code gets worked on.
> >
> >
>
> My array seems to work. We came from powers of two. I don't know why
> that is important. sqrt(2) is a fractional power of two, not a true
> integral power of two. So this is an overly theoretical goal.
>
> The user experience is what's important here. So until we find bad
> *actual experiences* with the code I proposed, I think we should run
> with it.
>
I agree.
And shift and ctrl keys cannot be used as turbo keys because they are 
already in use (shift and ctrl + mouse wheel are H and V scrool functions)

I also believe zoom level > 1024.0 are not useful and could be removed 
(shorter list of zooms)

-- 
Jean-Pierre CHARRAS

Maître de conférences
Directeur d'études 2ieme année.
Génie Electrique et Informatique Industrielle 2
Institut Universitaire de Technologie 1 de Grenoble
BP 67, 38402 St Martin d'Heres Cedex

Recherche :
GIPSA-LIS - INPG
Rue de la Houille Blanche
38400 Saint Martin d'Heres

--------------090803070406030005030907 Content-Type: text/html; charset=ISO-8859-1
Content-Transfer-Encoding: 7bit

<!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">
Dick Hollenbeck a &eacute;crit&nbsp;:
<blockquote cite="mid:498A90A0.1000709@..." type="cite">
<meta http-equiv="Context-Type"
content="text/html; charset=ISO-8859-1">
<div>
<div>
<div>
<p>Vesa Solonen wrote:<br>
&gt; On Thu, 5 Feb 2009, Dick Hollenbeck wrote:<br>
&gt;<br>
&gt; <br>
&gt;&gt; Your zoom patch for pcbnew makes the PCBNEW program too slow
to use when<br>
&gt;&gt; I have a delay in the drawing due to a large number of zones.<br>
&gt;&gt; <br>
&gt;<br>
&gt; Yes, that's very noticeable problem. Anyway the real solution is
somewhere <br>
&gt; else than on zoom granularity, like threading for rendering...<br>
&gt;<br>
&gt; <br>
&gt;&gt; Please try out this array for pcbnew and let me know if your
feelings<br>
&gt;&gt; would be hurt if we used it instead. The problem again is that
it takes<br>
&gt;&gt; too many steps to zoom in.<br>
&gt;&gt; <br>
&gt;<br>
&gt; How about adding a 'turbo key' for zooming with larger step, like
SHIFT + <br>
&gt; zoom? Your zoom series is ok too, but I'd like to keep even powers
of two <br>
&gt; zooms along the finer steps. That would mean sqrt(2) factor.
Anyway this <br>
&gt; should be configurable someway as otherwise we can not please
every one :)<br>
&gt;<br>
&gt; <br>
&gt;&gt; I have not tried eeschema. Drawing is faster there, but there
could be<br>
&gt;&gt; some advantage in making these similar.<br>
&gt;&gt; <br>
&gt;<br>
&gt; Maybe less steps is a good compromise until drawing code gets
worked on.<br>
&gt;<br>
&gt; <br>
<br>
My array seems to work. We came from powers of two. I don't know why <br>
that is important. sqrt(2) is a fractional power of two, not a true <br>
integral power of two. So this is an overly theoretical goal. <br>
<br>
The user experience is what's important here. So until we find bad <br>
*actual experiences* with the code I proposed, I think we should run <br>
with it.<br>
</p>
</div>
</div>
</div>
</blockquote>
I agree.<br>
And shift and ctrl keys cannot be used as turbo keys because they are
already in use (shift and ctrl + mouse wheel&nbsp; are H and V scrool
functions)<br>
<br>
I also believe zoom level &gt; 1024.0 are not useful and could be
removed (shorter list of zooms)<br>
<pre class="moz-signature" cols="72">-- 
Jean-Pierre CHARRAS

Ma&icirc;tre de conf&eacute;rences
Directeur d'&eacute;tudes 2ieme ann&eacute;e.
G&eacute;nie Electrique et Informatique Industrielle 2
Institut Universitaire de Technologie 1 de Grenoble
BP 67, 38402 St Martin d'Heres Cedex

Recherche :
GIPSA-LIS - INPG
Rue de la Houille Blanche
38400 Saint Martin d'Heres
</pre>
</body>
</html>
 --------------090803070406030005030907-- 




Follow ups

References