Double-clic on a page to edit it.

[lenoir]

I often try to edit a page by double-clicking on it's title in the tree. Intuitively.

I think that would be a nice benefit for clients editing many pages.

[ryan]

I agree, and I actually have tried to implement this before. I could never get jQuery's double click event register and work here. I suppose it's because we are already using the click event. But if someone else figures out how to get both double click and click working for it, I think it would be worthwhile to have. 

[diogo]

Here is a solution https://gist.github.com/ncr/399624

And a fiddle for testing http://jsfiddle.net/mBYPD/

[Soma]

Why so complicated? It's in jQuery

$('body').on("dblclick", function(){
    alert("double clicked");
});

Ah, just read again: click and double click on same. It does work with both.

[Pete]

diogo's code means we could have it delete the site on triple-clicks though and destroy the world on quadruple clicks with just a few adjustments

[Joss]

diogo's code means we could have it delete the site on triple-clicks though and destroy the world on quadruple clicks with just a few adjustments

I could live with that .... or not...

[lenoir]

Detroy the world if you wish, just let us know when you got a version running.

[diogo]

Maybe we don't need to complicate. Since a new page will open, we don't need to prevent the click action because it won't be seen. I did this change on ProcessPageList.js and it works pretty well:

//line: 403

$("a.PageListPage", $ul).click(clickChild)
    .dblclick(function(){
        window.open($(this).siblings('ul').find('li.PageListActionEdit a').attr('href'), "_self");
    });

[ryan]

Thanks Diogo! Seems to work very well. I've added to the source and will test locally for a day or two, then push it to the dev branch. 

[lenoir]

Thumbs up Diogo: works perfectly, feels like it's always been there ;-)

@ryan: reckon it could be by default on the next release?

[ryan]

It's in the dev branch now