Sorting requires clicking sort icon, not column header

[lukasolson]

1. Create a new search in discover
2. Add some columns
3. Attempt to sort on one of the columns other than time

Nothing happens when you click on the column headers. When you click on the time header, it sorts successfully.

[lukasolson]

Hmm... just kidding, you can sort on the columns, but you have to click on the actual tiny little sort icon, whereas with the time field, you just have to click on the column name.

[lukasolson]

We should probably just standardize how this works. Either we should allow you to click on the header in both cases, or disallow it in both cases.

[rashidkpc]

I changed this when I added the ability to remove columns from the header for 2 reasons:

1. There were a ton of buttons up there. It felt odd that clicking the header had meaning over just clicking the sort icon
2. We were showing the pointer finger for un-sortable columns (eg, _id), so it looked like clicking on the header should do something, but it didn't. I could've added a class to override the cursor on un-sortable things but just changing the sort to use the icon felt cleaner.

[lukasolson]

Yeah, makes total sense. Can we just make the same changes for the Time column as well? I think as long as it is consistent, it would be good. :)

[rashidkpc]

++ I missed that one