aslagle / reactive-table

A reactive table designed for Meteor
https://atmospherejs.com/aslagle/reactive-table
Other
328 stars 137 forks source link

Cannot search on object field #354

Open Ragsboss opened 8 years ago

Ragsboss commented 8 years ago

I have a collection that has a tags field, which is an object. It's key has tag name and value is an object that in turn has two keys - actualValue and lowerValue to represent tag value.

In reactive table, I have a field defined to represent the tags but what I'm noticing is filter is not searching tags. I'm not sure if this is by design/bug and if there is any workaround for this..

{ fieldId: 'tags', key: 'tags', sortable: false, label: 'Tags' tmpl: Template.RTableTags }

Any help is appreciated.

aslagle commented 8 years ago

The filter doesn't work with an object. You can make the field key point to one of the values in the object, but I'm not sure how you could do it if the tag name is different for each row.

Ragsboss commented 8 years ago

I'm using the reactive table to display the list of main objects, each of which has a set of tags. The tags column in reactive table shows first few tags and hover shows the full list of tags.

Perhaps I can stringify the whole tags and add a hidden field with sole purpose of enable tag search. The value of this field is this string representation of all strings. Would it then work? I'll try it soon and update you too but do share quick thoughts, if any..

Sent from my iPhone Email maybe terse

On Feb 13, 2016, at 2:05 PM, Amy Slagle notifications@github.com<mailto:notifications@github.com> wrote:

The filter doesn't work with an object. You can make the field key point to one of the values in the object, but I'm not sure how you could do it if the tag name is different for each row.

Reply to this email directly or view it on GitHubhttps://github.com/aslagle/reactive-table/issues/354#issuecomment-183764167.

aslagle commented 8 years ago

Yeah, that should work fine.

Ragsboss commented 8 years ago

This does work.. But I'm not sure if there is a better way or if this will cause any performance/scalability problems... Anyways I'm fine for now..