<div><span class="gmail_quote">On 5/24/08, <b class="gmail_sendername">Fredrik Höglund</b> <<a href="mailto:fredrik@kde.org">fredrik@kde.org</a>> wrote:</span><blockquote class="gmail_quote" style="margin:0;margin-left:0.8ex;border-left:1px #ccc solid;padding-left:1ex">
On Saturday 24 May 2008 14:53, Riccardo Iaconelli wrote:<br> > SVN commit 812032 by ruphy:<br> ><br> > Before we make a contaiment out of it (are we going to make it for 4.1 with the freeze?), and we're able to choose it,<br>
> at least add an option to make folderview's background transparent, so it doesn't hurt your eyes. ;-)<br> > Not sure about what doing by default, but I guess I'll keep it opaque (as it is now), so people understand that is a<br>
> plasmoid.<br> ><br> > ACK'd on review board by logixoul.<br> <br> <br>Hi Riccardo,<br> <br> When we discussed this patch on IRC yesterday, I pointed out that:<br> <br> 1) You can't add new UI strings when we're in message freeze.<br>
2) The applet should call setBackgroundHints() in constraintsEvent(),<br> not init(), and that there's already a commented out call in that function<br> that disables the default background.<br> <br> Aaron also said that he preferred it if we solved this problem by making<br>
the applet background translucent in the default theme.</blockquote><div></div><div>That's art issue, I don't want to sound harsh... but it's not really his field. =)</div><div>Plus, we're currently redoing that background...</div>
<br><blockquote class="gmail_quote" style="margin:0;margin-left:0.8ex;border-left:1px #ccc solid;padding-left:1ex"> Since you were present and participated in that discussion I don't<br> understand why you went ahead and committed this patch anyway<br>
without even addressing the technical issues with it.<br> </blockquote></div><p>To be sincere, I don't remember at all your objections, just aaron's, and I thought to have answered those in the associated bug. I've really missed those... sorry. :\</p>
<p>However, I did completely forget about the string freeze when committing this patch, and that is a sensible reason why this patch should be reverted. =)</p><p>I will revert this patch now... anyways, what should we do for 4.1? Translucent folderview by default? Put this patch back in 4.2? Make this patch default without the GUI option? Leave everything as it is? or...?</p>
<p>Bye,</p>-Riccardo<br>-- <br>GPG key:<br>3D0F6376<br>When encrypting, please encrypt also for this subkey:<br>9EBD7FE1<br>-----<br>Pace Peace Paix Paz Frieden Pax Pokój Friður Fred Béke 和平<br>Hasiti Lapé Hetep Malu Mир Wolakota Santiphap Irini Peoch שלום<br>
Shanti Vrede Baris Rój Mír Taika Rongo Sulh Mir Py'guapy 평화