528074 - Possible Bespin issues on Jetpack Gallery

Status: RESOLVED WONTFIX

(Websites Graveyard :: jetpackgallery.mozillalabs.com, defect)

Description

[Ryan Snyder [:ryansnyder] [:rsnyder] [:rysny]]

I was notified via Twitter the following by @ChristianSonne who worked on Bespin integration with Ubiquity.

@ryansnyder First(?) bug for you - it is unsafe to have Bespin load content from div, when content is likely to contain html (like </div>)

@ryansnyder http://tinyurl.com/yfgc3u7 and http://tinyurl.com/ylj385r is how we do it in Ubiquity - faced the same bug at first...

Comment 1

[Ryan Snyder [:ryansnyder] [:rsnyder] [:rysny]]

@ChristianSonne also mentioned that the issue was brought up in the Google Group:

Noticed the problem after this msg to groups http://groups.google.com/group/mozilla-labs-jetpack/browse_thread/thread/575aba497cf4b9cb

Comment 2

[Ryan Snyder [:ryansnyder] [:rsnyder] [:rysny]]

Worked with morgamic to remove the Bespin editor temporarily.  Now is a textarea.  Tested and pushed to production.  Should be reimplemented as Bespin, but not worried about that at the moment.

Comment 3

[Daniel Buchner [:dbuc]]

Fix the Bespin editor implementation and put back in edit jetpack page.

Comment 4

[Brian King [:kinger]]

I'm still seeing tags being ripped out from my code. See:

http://jetpackgallery.mozillalabs.com/jetpacks/312/code

Notice the JS and CSS in CDATA blocks. These should wrapped in script and style tags respectively, but when I save they are removed, rendering the Jetpack useless.

Comment 5

[Brian King [:kinger]]

Ok something I did made it magically work, but I am not sure what! I'll keep an eye out for further anomalies.

Comment 6

[Ryan Snyder [:ryansnyder] [:rsnyder] [:rysny]]

Per Daniel - We are holding off on Bespin integration.

Comment 7

[Ryan Snyder [:ryansnyder] [:rsnyder] [:rysny]]

Closing as wont fix; shutting down Jetpack Gallery at the end of the month and redirecting all traffic to addons.mozilla.org.