Hi Michael,<br><br>Thanks for reviewing this.<br><br>I discussed this with tomeu and the problem actually lies in the fact that icons take up as much horizontal space as they want (which is primarily governed by the text-label width below the icon). <br>
<br>Longer text labels should be broken into two or more lines and the overall icon width (including the label) should be kept confined.<br><br>I&#39;ll rework this patch and send it to the m.l. again.<br><br>While we&#39;re discussing this, I also noticed that currently, there is no minimum row spacing between the icons. So if I were to display 4 or more rows, the layout would seem a little cramped vertically. You can test this in your environment with 1024x768 resolution by commenting the following line<br>
<br>&gt;&gt; +        self._table.set_row_spacings(style.GRID_CELL_SIZE)<br><br>Should this be kept as is or should the above line be included (probably introducing a vertical scrollbar).<br><br>--<br clear="all">Anish<br>

<br><br><div class="gmail_quote">On Fri, Jun 25, 2010 at 5:26 AM, Michael Stone <span dir="ltr">&lt;<a href="mailto:michael@laptop.org">michael@laptop.org</a>&gt;</span> wrote:<br><blockquote class="gmail_quote" style="border-left: 1px solid rgb(204, 204, 204); margin: 0pt 0pt 0pt 0.8ex; padding-left: 1ex;">
<div class="im">On Sun, Jun 20, 2010 at 10:27:46PM -0000, Anish Mangal wrote:<br>
<blockquote class="gmail_quote" style="border-left: 1px solid rgb(204, 204, 204); margin: 0pt 0pt 0pt 0.8ex; padding-left: 1ex;">
From: anishmangal2002 &lt;<a href="mailto:anishmangal2002@gmail.com" target="_blank">anishmangal2002@gmail.com</a>&gt;<br>
<br>
This patch sets the number of icon-columns in the control<br>
panel based on the screen resolution. This patch also sets<br>
the table row spacing to GRID_CELL_SIZE.<br>
</blockquote>
<br></div>
Anish,<br>
<br>
I like the basic idea underlying this patch but I&#39;m not really satisfied<br>
with the current results.<br>
<br>
With 8 icons at 1024x768, I get two vertical columns of icons, each with<br>
four rows. Three rows scan be displayed at a time.<br>
<br>
With 8 icons at 800x600, I get one horizontal row of icons with four<br>
icons initially displayed.<br>
<br>
Neither of these layouts seems ideal.<br>
<br>
Thoughts?<br>
<br>
Michael<br>
<br>
P.S. - This problem actually reminds me rather strongly of the &quot;how do<br>
we place activity icons?&quot; problem that we had when activity icons were<br>
stuck on the frame. Do we have enough control panel entries now that we<br>
should think about re-using the home-view layout to place control panel<br>
icons too?<br>
<br>
P.P.S. - In any case, thanks for the patch!<br>
</blockquote></div><br>