Hi Peter,<br><br><div class="gmail_quote">On Wed, Mar 25, 2009 at 9:59 AM, Peter Penz <span dir="ltr"><<a href="mailto:peter.penz@gmx.at" target="_blank">peter.penz@gmx.at</a>></span> wrote:<blockquote class="gmail_quote" style="border-left: 1px solid rgb(204, 204, 204); margin: 0pt 0pt 0pt 0.8ex; padding-left: 1ex;">

<div>
<br>
<br>
</div>I think the patch looks good and should be committed. But I'd like to wait for David Faure's input too, as I also think we must prevent code duplication and Konqueror should just use the Dolphin KPart for this.</blockquote>
<div> </div>
<div>I already have other patchs: one makes dolphinpart to use remoteencoding class, the other simply wipe KRemoteEncodingPlugin from Konqueror source dir. As KRemoteEncodingPlugin is aKparts::Plugin, Konqueror has zero line of code about<br>

it. So just removing plugin dir is enough for Konqueror, as it uses DolphinPart. And I met David at #plasma yesterday (actually I was looking for him:)) He said he don't know much about remote encoding, want me to ask Tiago. <br>
<br>
</div><div><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;">
- dolphinviewactionhandler.cpp: You include "dolphinnewmenuobserver.h", but have not used it (maybe a relict from the recent "Create New..." patch?).</blockquote><div> </div><div>Ahh, didn't notice it. After David commited the KnewMenu stuff that you asked him, I wanted to make f10 action to use it. But actually didn't get it :) I understand that how to attach an existing nemMenu instance but don't understand how we will get the attached instance from dolphinnewmenuobserver. I can make f10 action work, if you give some hint.<br>

<br> <br>Thanks.<br></div></div><br clear="all"><br>-- <br>Rahman Duran<br><br>Software Engineer<br>Turkey<br><br>How many apples fell on Newton's head before he took the hint! <br><br>