Allow negation/exclusion for searchable fields

chop-dbhi / cilantro

Official client for Harvest (http://harvest.research.chop.edu)

http://cilantro.harvest.io

Other

28 stars 8 forks source link

Closed naegelyd closed 10 years ago

naegelyd commented 10 years ago

Fix #678.

Signed-off-by: Don Naegely naegelyd@gmail.com

bruth commented 10 years ago

Can you post a screenshot?

naegelyd commented 10 years ago

Sure, here it is. Definitely open to change how this looks just wanted to get this in there and start the conversation.

screen shot 2014-10-14 at 2 13 23 pm

bruth commented 10 years ago

I think the text and checkbox would look more natural above the textarea. Likewise, I think the "Clear" button would look better there as well.

murphyke commented 10 years ago

+1 for this control and the Clear button above the textarea

naegelyd commented 10 years ago

Now it looks like this:

screen shot 2014-10-14 at 3 02 37 pm

bruth commented 10 years ago

Looks good. Much more clear in my opinion.

On Tue, Oct 14, 2014 at 3:04 PM, Don Naegely notifications@github.com wrote:

Now it looks like this:

Reply to this email directly or view it on GitHub: https://github.com/cbmi/cilantro/pull/726#issuecomment-59098455