Hello P5: song example breaks the preview window

[rasagy]

What I tried to do

Opened the first Example in the list — Hello P5 : song, and clicked on play song automatically. Here’s the project: http://alpha.editor.p5js.org/p5/sketches/r1SLhBE4mZ-

What happened

The sketch played two notes, and then reloaded the page with editor inside the preview section:

Other attempts

I read on #414 (comment) that there are some errors with the Preview section that don’t show up on Full screen, so I went to Share, got the link as http://alpha.editor.p5js.org/full/r1SLhBE4mZ- and tried this in the full screen link.

• First time, it played a note and appended a / to the link: http://alpha.editor.p5js.org/full/r1SLhBE4mZ-/#
• After clicking on the play song automatically link, it appended another / to the link: http://alpha.editor.p5js.org/full/r1SLhBE4mZ-//# and gave a 404.

Here’s a gif and screenshot of the console

GIF:

Errors in console:

Errors in full screen view:

---

Platform details

I’m on Google Chrome Version 60.0.3112.101 (Official Build) (64-bit) / Mac Sierra.

Let me know in case this is already being discussed in another issue/you need more info to replicate it!

[catarak]

Thanks for the detailed breakdown! This is a known issue, discussed in #145. The example actually got fixed, I just need to rerun the task in production. I'll do that right now 😄

[catarak]

This is now fixed in production. Dom links still cause the editor to render in the iframe, so I will leave #145 open.