[aur-dev] [PATCH] Implemented typeahead suggest

Dan McGee dpmcgee at gmail.com
Wed Dec 12 18:41:37 EST 2012


On Wed, Dec 12, 2012 at 4:25 PM, canyonknight <canyonknight at gmail.com> wrote:
> On Tue, Dec 11, 2012 at 12:36 PM, Marcel Korpel <marcel.lists at gmail.com> wrote:
>> ---
>
> So I played with this patch again. The source of my problem was the
> line breaks that were added when you sent it. After going through and
> fixing them a second time, it does indeed work and looks really nice!
>
> I added a couple of thoughts inline...
>
>> +        matcher: function(item) { return true; },
>> +        sorter: function(items) { return items; },
>> +        menu: '<ul class="pkgsearch-typeahead"></ul>',
>> +        items: 20
>
> Any reason to display 20 items while archweb only displays 10?

Remember that archweb has ~10K packages, while the AUR has ~40K. There
might be a lot of noise in the signal, and showing more items would at
least help you see it.

-Dan


More information about the aur-dev mailing list