Marco,<br><br><div><span class="gmail_quote">On 11/8/07, <b class="gmail_sendername">Marco Hugentobler</b> &lt;<a href="mailto:marco.hugentobler@karto.baug.ethz.ch">marco.hugentobler@karto.baug.ethz.ch</a>&gt; wrote:</span>
<blockquote class="gmail_quote" style="border-left: 1px solid rgb(204, 204, 204); margin: 0pt 0pt 0pt 0.8ex; padding-left: 1ex;">Hi Matthew,<br><br>I had a look at the color ramps patch and in my opinion, this is a really cool
<br>and useful enhancement.<br><br>Some suggestions:<br>- As you already wrote, it would be good to parse the directory and have a<br>dropdown list</blockquote><div><br>Yes. It&#39;s an easy fix and I have that working on my dev version (see screenshot at 
<a href="http://perrygeo.net/img/Screenshot.jpg">http://perrygeo.net/img/Screenshot.jpg</a>). I&#39;ll get another patch out this weekend.<br><br>IMO the next UI enhancement would be to, instead of list the names of the color ramp, show a small jpg so you can inspect the ramp visually.
<br><br></div><blockquote class="gmail_quote" style="border-left: 1px solid rgb(204, 204, 204); margin: 0pt 0pt 0pt 0.8ex; padding-left: 1ex;">- It would be more Qt style (and probably more platform independent) to use<br>
the QTextStream class instead of C i/o methods</blockquote><div><br>Good to know. The C i/o and parsing style always felt a bit clunky.<br></div><br><blockquote class="gmail_quote" style="border-left: 1px solid rgb(204, 204, 204); margin: 0pt 0pt 0pt 0.8ex; padding-left: 1ex;">
- It seems to me that these .cpt files already define the number of steps and<br>the class boundaries. Maybe it would be an option to have a button &#39;from<br>color ramp...&#39; in the graduated symbol dialog that opens a dialog where the
<br>available ranges are visualised. Then, after selecting and clicking ok, the<br>classes, number of classes and their boundaries are directly inserted into<br>the list. The same dialog could also be used for the continuous renderer
<br>(where the interpretation of the ramps would be continuous) so the ramps<br>could be stored in one directore. What do you think?</blockquote></div><br>I like having the color ramp as another standard option in the main dialog window. I&#39;m not sure if Qt can do this but it would be nice to see a dropdown of visual representations of the ramps (a bunch of  100x20 jpgs) to select from. But in terms of a dialog box that could be used in other contexts, your idea of having a &quot;From Color Ramp&quot; button might be better.
<br><br>In terms of getting this implemented, I just hacked this patch together for graduated symbols but it would be worthwhile to abstract the color ramp handling into a separate class so that all the various renderers can it.
<br><br>-- <br>Matthew T. Perry<br><a href="http://www.perrygeo.net">http://www.perrygeo.net</a>