<br><br><div class="gmail_quote">On Jan 2, 2008 10:29 PM, Christopher Blauvelt &lt;<a href="mailto:cblauvelt@gmail.com">cblauvelt@gmail.com</a>&gt; wrote:<br><blockquote class="gmail_quote" style="border-left: 1px solid rgb(204, 204, 204); margin: 0pt 0pt 0pt 0.8ex; padding-left: 1ex;">
<br><br><div class="gmail_quote"><div><div></div><div class="Wj3C7c">On Jan 2, 2008 10:03 AM, Aaron J. Seigo &lt;<a href="mailto:aseigo@kde.org" target="_blank">aseigo@kde.org</a>&gt; wrote:<br></div></div><blockquote class="gmail_quote" style="border-left: 1px solid rgb(204, 204, 204); margin: 0pt 0pt 0pt 0.8ex; padding-left: 1ex;">
<div><div></div><div class="Wj3C7c">
<div>On Wednesday 02 January 2008, Christopher Blauvelt wrote:<br>&gt; On Jan 1, 2008 11:38 PM, Aaron J. Seigo &lt;<a href="mailto:aseigo@kde.org" target="_blank">aseigo@kde.org</a>&gt; wrote:<br>&gt; &gt; On Tuesday 01 January 2008, Christopher Blauvelt wrote:
<br>&gt; &gt; &gt; Attached is a patch that allows you to align icons to a grid. &nbsp;Further,<br>&gt; &gt;<br>&gt; &gt; it<br>&gt; &gt;<br>&gt; &gt; * there is a write- and readEntry in KConfigGroup that takes a QPointF.<br>

&gt; &gt; you<br>&gt; &gt; don&#39;t need to do the parsing yourself (indeed, shouldn&#39;t)<br>&gt;<br>&gt; I&#39;m not following which code segment you&#39;re talking about.<br><br></div>the inline readPos and writePos methods.
<br><br>it&#39;s unfortunate you even have to store/restore those and can&#39;t just let the<br>usual applet loading handle it for you ... but oh well =)<br><div><br>&gt; &gt; immutability and locking of the containment; Show Icons should go into
<br>&gt; &gt; the config dialog, not the context menu;<br>&gt;<br>&gt; I would have liked to make a submenu to address icons but I couldn&#39;t see<br>&gt; how that was possible given that only QAction*&#39;s can be returned from
<br>&gt; contextMenu().<br><br></div>this is something i&#39;d like to find an elegant solution to for 4.1. one that<br>doesn&#39;t involve returning QMenus, either ;)<br><div><br>&gt; I remove Icon related items when they&#39;re not shown but if
<br>&gt; it needs to be added to the config dialog I can add the extra tab that had<br>&gt; been previously discussed. &nbsp;Unless you had a different proposal.<br><br></div>well, i don&#39;t think we need a tab for just 2 checkboxes. they can simply be
<br>added to the bottom of the dialog imho.<br><div><br>&gt; &gt; + &nbsp; &nbsp;if (iconsVisible) {<br>&gt; &gt; + &nbsp; &nbsp; &nbsp; &nbsp;m_desktopDir.openUrl(KGlobalSettings::desktopPath());<br>&gt; &gt; + &nbsp; &nbsp;}<br>&gt; &gt; + &nbsp; &nbsp;else {
<br>&gt; &gt;<br>&gt; &gt; ;)<br>&gt;<br>&gt; Man, you&#39;re worse than my sixth-grade english teacher :) &nbsp;Fixed.<br><br></div>yeah, i know. just imagine being me though and cringing every time i spot one<br>of these issues thinking, &quot;oh man, they are going to think i&#39;m the most
<br>horribly OCD person in the WORLD!&quot;. i just want a clean code base and am<br>trying to instill those values into the project. honestly, i don&#39;t<br>alphabetize my sock drawer or anything like that in real life ;)
<br><div><br>&gt; &gt; * do we need to offer a separate hook for media icons? hm.. i suppose so<br>&gt; &gt; if<br>&gt; &gt; only because someone might want<br>&gt;<br>&gt; That&#39;s next to be implemented.
<br><br></div>cool.<br><div><br>&gt; &gt; * i assume the icon.h and icon.cpp bits aren&#39;t intended as part of the<br>&gt; &gt; diff?<br>&gt;<br>&gt; Oops! &nbsp;Thats a start at my attempt to allow dragging of icons by clicking
<br>&gt; and dragging from the middle. &nbsp;I guess I need to do an &quot;svn diff&quot; from a<br>&gt; little farther down in the directory.<br><br></div>;) yeah, i often end up having to specify the exact files when doing diffs
<br>myself. oh well =)<br><font color="#888888"><br>--<br></font><div><div></div><div>Aaron J. Seigo<br>humru othro a kohnu se<br>GPG Fingerprint: 8B8B 2209 0C6F 7C47 B1EA &nbsp;EE75 D6B7 2EB1 A7F1 DB43<br><br>KDE core developer sponsored by Trolltech
<br></div></div><br></div></div><div class="Ih2E3d">_______________________________________________<br>Panel-devel mailing list<br><a href="mailto:Panel-devel@kde.org" target="_blank">Panel-devel@kde.org</a><br><a href="https://mail.kde.org/mailman/listinfo/panel-devel" target="_blank">

https://mail.kde.org/mailman/listinfo/panel-devel</a><br><br></div></blockquote></div>Attached is the a patch with changes to include:<br>- Removed the &quot;lock icons&quot; feature<br>- Fixed Style violations<br>- Removed the readPos and writePos methods since they weren&#39;t used anyway.
<br><br>
</blockquote></div>Just wondering if anybody had taken a look at this to see if&nbsp; it was good to commit.&nbsp; I just want to make sure I meet the deadline.<br>Chris<br>