Search in contacts on compose page

[alec]

The list of contacts in compose screen isn't really useful if there's no search feature.

[thomasb]

The autocompletion is the search feature here. It's meant to be a listing in addition to the already existing autocompletion.

[alec]

You're saying we don't need searching here? Where's the point in having the list here then?

[vanmeeuwen]

I think a search is useful - perhaps another type of search than is used in other places - starting to type would just filter out the results from the list of contacts displayed that would not match.

[vanmeeuwen]

FWIW, ​http://webmail.klab.cc/~vanmeeuwen/roundcubemail-trunk.svn/roundcubemail/, in the compose screen, with the third party 'compose_addressbook' plugin enabled, the icon just above the 'Cancel' button shows something similar to what we would want, I suppose.

[thomasb]

Replying to alec:

You're saying we don't need searching here? Where's the point in having the list here then?

The point of having the list is to literally see a list of (possible) recipients. Let's say I wanna invite some friends to a party so I need to go through a list of addresses to pick the right persons without forgetting anyone. As opposed to writing a message to a specific person (which is perfectly possible with autocompletion) the list is there for "browsing". I don't say that searching isn't necessary at all here but I'd not consider it a showstopper.

[thomasb]

Replying to vanmeeuwen:

I think a search is useful - perhaps another type of search than is used in other places - starting to type would just filter out the results from the list of contacts displayed that would not match.

Filtering a large LDAP directory with 600K entries? Good luck!

[alec]

This should work the same as search in addressbook. We could even add identical searchbox above the list.

[thomasb]

Yes, I definitely vote for that.

[thomasb]

Added search box in commit [bd0daee7fd8d].

[alec]

Please, use splitter to change size of addressbook widget to see style issues with search box element.

[alec]

Also, implemented search doesn't find groups, but maybe should.

[thomasb]

Replying to alec:

Please, use splitter to change size of addressbook widget to see style issues with search box element.

What browser? Works for me.

[alec]

Firefox 18.0.1 (see screenshots), the same in Opera. There's no issue in Chrome indeed.

[thomasb]

Scaling fixed in [bcbfe0ba457f6fce752] groups are a bit trickier to implement...

[alec]

In Larry skin there's no indication about which addressbook is selected.

[thomasb]

Replying to alec:

In Larry skin there's no indication about which addressbook is selected.

That could be a side-effect of my recent changes to the address book listing using the new treelist.js widget. I'm looking into it.

[thomasb]

Selection bug fixed in [4223b1e38ff38d7]

I'm closing this for now. If searching in groups is important, please create a new ticket.