Yeah, sorry for that.

I would suggest rejecting my pull request and just fixing it on your master
branch, because as you noticed, the missing line break will break your gem.

Thanks for sharing your gem with community, btw :)

2013/6/14 Adam McCrea <notifications@github.com>

> In lib/activeadmin-sortable.rb:
>
> > @@ -18,8 +18,7 @@ module TableMethods
> >
> >        def sortable_handle_column
> >          column '' do |resource|
> > -          sort_url = "#{resource_path(resource)}/sort"
> > -          content_tag :span, HANDLE, :class => 'handle', 'data-sort-url' => sort_url
> > +          sort_url =   polymorphic_path([:sort, :admin, active_admin_config.resource_label.parameterize.underscore],{:id => resource.id})          content_tag :span, HANDLE, :class => 'handle', 'data-sort-url' => sort_url
>
> Thanks for this. The diff makes it look like we lost a line break here,
> though.
>
> --
> Reply to this email directly or view it on GitHub<https://github.com/neo/activeadmin-sortable/pull/5/files#r4700466>
> .
>



-- 
Kind regards,
Radek Jêdryszczak
www.selleo.com