Code review comment for lp:~jeremywootten/slingshot/fix-1213321

Thanks for this. I have uploaded a different fix already - I dealt with
the Enter key-press by connecting to the activate signal of the searchbar
instead. I am waiting to see whether anyone finds a flaw in this approach
and will bear your alternative solution in mind.

best regards

On 7 October 2013 19:36, if <email address hidden> wrote:

> what i did for fixing this, I'm not familiar with vala, is keep the code
> handles the KP_Enter remains in the
> public override bool key_press_event (Gdk.EventKey event)
> {
>
> // only deal with KP_Enter
>
> }
> and other code is exactly you fixed. I know it is bad fix, but that seems
> work fine.
>
>
> 2013/10/2 Jeremy Wootten <email address hidden>
>
> > You are right! I always selected the application with the mouse so did
> not
> > notice this. I will look into this.
> >
> >
> > On 1 October 2013 08:43, if <email address hidden> wrote:
> >
> > > Review: Needs Fixing
> > >
> > > hi,I tested this fix, and i found out it doesn't response to ENTER key,
> > > cause public bool on_key_press (Gdk.EventKey event) will never be
> called
> > > when only ENTER pressed
> > > --
> > >
> > >
> >
> https://code.launchpad.net/~jeremywootten/slingshot/fix-1213321/+merge/182058
> > > You are the owner of lp:~jeremywootten/slingshot/fix-1213321.
> > >
> >
> >
> >
> > --
> > Jeremy Wootten
> > GPG Key ID CB585BCD
> > Key Fingerprint 37C0 3C2A A6D4 E45B BA7C 4328 2DF2 1882 CB58 5BCD
> >
> >
> >
> https://code.launchpad.net/~jeremywootten/slingshot/fix-1213321/+merge/182058
> > You are reviewing the proposed merge of
> > lp:~jeremywootten/slingshot/fix-1213321 into lp:slingshot.
> >
>
> --
>
> https://code.launchpad.net/~jeremywootten/slingshot/fix-1213321/+merge/182058
> You are the owner of lp:~jeremywootten/slingshot/fix-1213321.
>

--
Jeremy Wootten
GPG Key ID CB585BCD
Key Fingerprint 37C0 3C2A A6D4 E45B BA7C 4328 2DF2 1882 CB58 5BCD