Allow initial sort column to be specified about sortable HOT 10 OPEN

@zackbloom thoughts on the right approach here? Triggering a th.click() is the current strawman.

from sortable.

Implemented a lazy proof of concept of this here for my own use:

https://github.com/jmduke/sortable

I think refactoring the main function so its less anonymous would be a more semantic way to do this -- that way we could pass it a th and maybe a direction parameter, and access it outside the scope of the onClick handler. Thoughts?

from sortable.

@jmduke thanks for the POC. I think you're right that a refactor is probably needed to do this the right way.

from sortable.

+1

from sortable.

+1

from sortable.

Hey @adamschwartz, I was thining about submitting a PR for this today. My idea was to extract the sorting logic in the event handler to a sort method and bind it to Sortable inside the event handler. Then we can call it from the init method like @jmduke did in his implementation. Instead of calling click we could just call that method.
I'm working on the PR right now. I'll submit it in a few minutes.

from sortable.

Thanks @visola we’ll take a look!

from sortable.

Any updates? Would be nice

from sortable.

@adamschwartz Anything happening with the PR?

from sortable.

@interrogator Unfortunately I have not had time to follow up on this issue. Please feel free to reach out to other members of the @HubSpot org who may be able to help.

from sortable.