see more.. shown with < 5 values

[grtjn]

if nr values smaller than facet limit, suppress see more..

[joemfb]

This should be on ml-search-ng, the functionality is totally contained there.

[grtjn]

Yes. Best to link to a ticket there, and keep this to update the bower dep..

[joemfb]

To be clear, you're asking that the stored search options be loaded initially, and the facet values be compared to their respective facet-limit property?

[grtjn]

Yeah, I guess that is how it needs to be done. Would be convenient if REST api would return a total facet count itself, but it doesn't. Although, we also have this includeAggregates that can provide a (total) count for a facet?

[hunterwilliams]

I realize this should be moved over to ml-search-ng but would the behavior be too odd if show more only appeared on facets with 5 or more values? Unfortunately 5 would look sorta weird, but it'd eliminate the awkwardness of 2-4 valued facets

[grtjn]

Crossed my mind too, but reluctant on a solution like that..

[grtjn]

If we can't find a solution for this in 0.2.0, I suggest we disable the show more for the moment. It looks confusing like it is now, despite the fact that it is a really nice feature..

[joemfb]

I don't think show-more needs to be on by default. It's trivial to add to a demo if requested.

I think loading search-options when a new mlsearch context is initiated could be useful for lots of things, not just this, but I'd like to take time and get it right. I won't be free to work on it for awhile.

[grtjn]

Pushing this into next milestone..