Hi Radek,<br><br><div class="gmail_quote">On Sat, Jun 25, 2011 at 3:35 AM, Radek Doulik <span dir="ltr">&lt;<a href="mailto:rodo@novell.com">rodo@novell.com</a>&gt;</span> wrote:<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex;">

<br>
At least I am not reaching the part you modified when adding custom<br>
animation thru custom animation pane (using the add button and custom<br>
animation dialog). If we move it to the loop, it will be reached always<br>
when selecting an entry - the select method is called recursively when<br>
adding new pEntry in:<br>
<br>
    if( !pEntry &amp;&amp; bSelect )<br>
    {<br>
        append( pEffect );<br>
        select( pEffect );<br>
    }<br>
<br>
I wonder how do you reach that part of code?</blockquote><div><br></div><div>Hmm. I think the code is incorrect here. That last if statement should really only be used if the effect wasn&#39;t in the list &amp; needs to be added. But when I added the MakeVisible in the if statement, it made new animations be scrolled to, and I was doing it via add-&gt;custom animation dialog too. What do you think of my attached patch? Once we have found the given entry, we should select it and then we are done, aren&#39;t we?</div>
<div><br></div><div>Kind regards,</div><div><br></div><div>Luke.</div></div>