[Radiant] (yet another) sorting patch

[Erik B. Ordway]

I thought about doing the list convertion, it would have been  
cleaner.  I decided to skip the AJAX for and do the page refresh as I  
did not want ot bite off more that I could chew.  It does have a  
speed issue going this route.  It feels a little slow one one site  
with 132 pages.

Since you have implemented the page to use lists I should be able to  
convert my re-parenting stuff to make use of that and avoid the  
refresh issue.  Would you be interested in a patch on your patch that  
does that?

On Aug 3, 2006, at 11:12 PM, bodhi wrote:

> Hi All,
>
> Sorry to step on your toes Erik, but I was halfway through this when
> you posted your patch (which I haven't had a chance to look at just
> yet). I converted the page list from tables to lists, and used
> scriptaculous to order pages. It doesnt do reparenting, so only allows
> you to sort children, not move children to different sections.
>
> It's a pretty invasive patch, but I did get to remove most of the
> javascript from admin/pages/index.rhtml (I enjoy deleting  
> code :) ). If
> anyone could take a look at it and see how it works for them, it would
> be much appreciated:
>
> http://dev.radiantcms.org/radiant/ticket/199
>
> It breaks a few of the unit tests (due to the sorting) and I haven't
> tested it on Internet Explorer yet.
>
> Bodhi
>
> _______________________________________________
> Radiant mailing list
> Radiant at lists.radiantcms.org
> http://lists.radiantcms.org/mailman/listinfo/radiant