jnf / blinder

We use Blinder to collect and review responses to event RFPs.
MIT License
6 stars 8 forks source link

Cln/sortable #76

Closed carols10cents closed 10 years ago

carols10cents commented 10 years ago

What do you think of this to address #74? I changed the seeds file to make Talk Focus sortable but I'd imagine you'd change that in the console on the live site.

jnf commented 10 years ago

Pulled the branch and walked through it. Overall, I love it.

I don't like the jquery plugin, but I never like jquery plugins. I noticed that even though some ths don't have the sorting arrows, they're still watched for sorting. Clicking on any th kicks off sorting. Not a big deal, but I think it'd be better to just have the sorting indicators on every th so people aren't caught off guard if they accidentally click on one.

Also, I went one step further and set sortable to true for both Talk Focus and Talk Title. It worked perfectly. :yellow_heart: