{"issue":{"user":{"id":1240282,"login":"jsamuel","url":"https://api.github.com/users/jsamuel","avatar_url":"https://secure.gravatar.com/avatar/bc1d195cedb959a3946572b08ce9e7c7?d=https://a248.e.akamai.net/assets.github.com%2Fimages%2Fgravatars%2Fgravatar-140.png","gravatar_id":"bc1d195cedb959a3946572b08ce9e7c7"},"milestone":null,"title":"Show blocked embeds","created_at":"2011-12-22T04:12:14Z","labels":[{"name":"extension","url":"https://api.github.com/repos/RequestPolicy/requestpolicy/labels/extension","color":"ededed"}],"updated_at":"2011-12-22T04:12:16Z","state":"open","id":2633138,"closed_at":null,"pull_request":{"patch_url":null,"html_url":null,"diff_url":null},"html_url":"https://github.com/RequestPolicy/requestpolicy/issues/201","body":"When for example a Youtube video is blocked it currently does not show an indication like it does for images.\r\n\r\n{{{\r\n    <object width=\"640\" height=\"390\">\r\n        <param name=\"movie\" value=\"http://www.youtube.com/v/cs2DsoiA54Y?fs=1&amp;hl=en_US&amp;rel=0\"></param>\r\n        <param name=\"allowFullScreen\" value=\"true\"></param>\r\n        <param name=\"allowscriptaccess\" value=\"always\"></param>\r\n        <embed src=\"http://www.youtube.com/v/cs2DsoiA54Y?fs=1&amp;hl=en_US&amp;rel=0\" type=\"application/x-shockwave-flash\" allowscriptaccess=\"always\" allowfullscreen=\"true\" width=\"640\" height=\"390\"></embed>\r\n    </object>\r\n}}}\r\n\r\nInstead of replacing the embed src with a blank swf, one solution would be to replace the embed tag with an image tag similar to how the blocked images are currently handled.\r\n\r\n\r\n{{{\r\n    <object height=\"390\" width=\"640\">\r\n        <param name=\"movie\" value=\"http://www.youtube.com/v/cs2DsoiA54Y?fs=1&amp;hl=en_US&amp;rel=0\">\r\n        <param name=\"allowFullScreen\" value=\"true\">\r\n        <param name=\"allowscriptaccess\" value=\"always\">\r\n        <img src=\"snip\" title=\"[youtube.com]\" style=\"border: 1px solid rgb(255, 204, 204); background-repeat: no-repeat; background-position: center center; background-image: url(snip);\" height=\"390\" width=\"640\">\r\n    </object>\r\n}}}\r\n\r\nThis could also be expanded for blocked iframes.","number":201,"url":"https://api.github.com/repos/RequestPolicy/requestpolicy/issues/201","comments":1,"assignee":null},"comment":{"user":{"id":1240282,"login":"jsamuel","url":"https://api.github.com/users/jsamuel","avatar_url":"https://secure.gravatar.com/avatar/bc1d195cedb959a3946572b08ce9e7c7?d=https://a248.e.akamai.net/assets.github.com%2Fimages%2Fgravatars%2Fgravatar-140.png","gravatar_id":"bc1d195cedb959a3946572b08ce9e7c7"},"created_at":"2011-12-22T04:12:16Z","updated_at":"2011-12-22T04:12:16Z","id":3244120,"body":"[justin] Thanks for the patch. I'll target this for 0.5.21 because 0.5.20 contains a security fix (#204) and I feel this change will need to spend some time in beta. Hopefully 0.5.21 can bake longer than 0.5.20.","url":"https://api.github.com/repos/RequestPolicy/requestpolicy/issues/comments/3244120"},"action":"created"}