Skip Menu |

This queue is for tickets about the Jifty CPAN distribution.

Report information
The Basics
Id: 69204
Status: new
Priority: 0/
Queue: Jifty

People
Owner: Nobody in particular
Requestors: github [...] bestpractical.com
Cc:
AdminCc:

Bug Information
Severity: (no value)
Broken in: (no value)
Fixed in: (no value)



CC:
Subject: Fwd: [bestpractical.com #45121] Re: [jifty] Don't require that Popout region links be in a region already or specify an element (a632c54)
Date: Thu, 30 Jun 2011 11:28:43 -0400
To: bug-Jifty [...] rt.cpan.org
From: github [...] bestpractical.com
This is forward of transaction #842266 of a ticket #45121

Message body is not shown because sender requested not to inline it.

Subject: Re: [jifty] Don't require that Popout region links be in a region already or specify an element (a632c54)
Date: Wed, 8 Jun 2011 14:14:55 -0700
To: github [...] bestpractical.com
From: tsibley <reply+c-421369-f233327f23732bb6f8af8c5bdf9da5dea913c745 [...] reply.github.com>
I just discovered that this introduces a bug when using Popout and the toggle => 1 option. On the first click, the popout will display. On the second click, the content of your entire page disappears since jifty.js does some shortcutting and calls jQuery(element).empty(). Since we hardcode the target element as "body" in Element.pm, the <body> node is emptied. Whoops! -- Reply to this email directly or view it on GitHub: https://github.com/bestpractical/jifty/commit/a632c54df8d893fca1dbafbbf528f48a6ed13e50#commitcomment-421369