[Kmymoney-devel] Payee completion (was Re:Checkpoint before release)

[Alvaro Soliverez]

On Mon, May 10, 2010 at 12:31 PM, Cristian Oneţ <onet.cristian at gmail.com> wrote:
> On Mon, May 10, 2010 at 6:25 PM, Alvaro Soliverez <asoliverez at gmail.com> wrote:
>> On Mon, May 10, 2010 at 12:22 PM, Cristian Oneţ <onet.cristian at gmail.com> wrote:
>>> On Mon, May 10, 2010 at 6:14 PM, Alvaro Soliverez <asoliverez at gmail.com> wrote:
>>>> On Mon, May 10, 2010 at 11:48 AM, tonyb <tonyb.lx at ntlworld.com> wrote:
>>>>> On Monday 10 May 2010 02:19:27 Alvaro Soliverez wrote:
>>>>>> One bug I think is critical is 235618, the problem in the way payee
>>>>>> completion works, but 1 week should be enough to fix it. I hope so.
>>>>>
>>>>> I just tried out Christian's patch, attached. It performs payee completion on
>>>>> Tab or Enter, as per KDE3. However, as Christian pointed out, it is impossible
>>>>> to create a new payee which is a matching starting subset of an existing one.
>>>>> In the KDE3 version, this can be achieved with Escape, or clicking the mouse
>>>>> in another field.
>>>>>
>>>>
>>>> Does it solve the problem when selecting the payee and then pressing a
>>>> key triggering the new payee dialog?
>>>
>>> AFAIK that was solved when https://bugs.kde.org/show_bug.cgi?id=236410
>>> was closed.
>>>
>> The crash is solved, but the payee completion still has issues as per
>> https://bugs.kde.org/show_bug.cgi?id=235618
>>
>> That one is not solved, and it's pretty annoying.
>
> The bug 235618 states that a payee can't be selected using the tab key
> and you stated that there is a 'problem when selecting the payee and
> then pressing a key triggering the new payee dialog' I think that
> those are 2 different issues. Anyway selecting a payee and pressing a
> key should not bring up the new payee dialog and I don't remember
> experiencing this behavior but I'll try to reproduce it today when
> I'll get home.
>

The behaviour I see is when you select the payee, then press a key
which doesn't match any of the existing payees.
That triggers the New payee dialog.

It seems to be caused by a FocusOutEvent of the completer glimpsing
for a fraction of a second, then clearing itself and hiding.
Unfortunately, the event reason is not Popup, so it is not escaped
properly.