[sugar] Could someone test the Home search entry...?
Simon Schampijer
simon at schampijer.de
Fri Aug 29 05:56:48 EDT 2008
Eben Eliason wrote:
> Hello all -
>
> I've recently experienced some very buggy behavior with the search
> entry in Home in my jhbuild (master). I want to ensure that this gets
> properly tested in the upcoming joyride build (as I'll be moving/on
> vacation myself), so I'm going to leave a couple test cases here for
> someone to take on.
>
> 1. Switch to the list view of Home and enter 'bro' into the search
> entry. The Browse activity should turn up as the only result in the
> filtered list.
> 2. Append a nonsense string to your search eg. 'broasdf'. If all goes
> well, you should have an empty list, though in my jhbuild testing,
> Browse remained visible.
> 3. Clear the search entry (select text and delete, or click the little
> 'x' button within the entry). The full list of installed activities
> should return. In my jhbuild testing, nothing could be done to
> restore the full list.
>
> If the above all work as expected, please verify the test case
> attached to http://dev.laptop.org/ticket/7874 as well.
>
> Finally, if this is all verified as working in joyride, it's probably
> still worth testing on the master branch in jhbuild, since I can't
> seem to get it to work there. Oddly, I tried reverting both sugar and
> sugar-toolkit to the last commits on August 13th (the day I submitted
> the patch attached to the aforementioned ticket, at which time that
> test case DID work fine in jhbuild), but the problem remained after
> rebuilding. Perhaps git just hates me.
>
> In any case, thanks for testing for me! If this IS broken, I nominate
> it as a blocker for 8.2.0, since it's easy to trigger a search, and
> impossible to clear it to reveal all installed activities, thus making
> it impossible to set favorites or launch some activities at all
> without a reboot.
>
> - Eben
>
Eben I think a simple explanation is that there is old code in master :)
def __entry_activated_cb(self, entry):
if self._autosearch_timer:
gobject.source_remove(self._autosearch_timer)
new_query = entry.props.text
if self._query != new_query and new_query is not '':
That part should not be there:
and new_query is not '':
And maybe other parts as well. Will have a look once d.l.o is up again
and verify your test case for master.
Enjoy your holidays,
Simon
More information about the Sugar
mailing list