no, it's fixed after all..
authorjoey <joey@0fa5a96a-9a0e-0410-b3b2-a0fd24251071>
Sat, 28 Jul 2007 20:59:49 +0000 (20:59 +0000)
committerjoey <joey@0fa5a96a-9a0e-0410-b3b2-a0fd24251071>
Sat, 28 Jul 2007 20:59:49 +0000 (20:59 +0000)
doc/bugs/Problems_with_graphviz.pm_plug-in.mdwn

index d3629f840a15b5158c366cf988613f5e28196513..1ec3e32bceb5f2cd064ff25e5d4553878b221759 100644 (file)
@@ -17,11 +17,10 @@ It also generates image URLs relative to the page being rendered, which means th
 >> the same, and the patch is still needed for making the graphviz
 >> plug-in work in preview mode here.
 
 >> the same, and the patch is still needed for making the graphviz
 >> plug-in work in preview mode here.
 
->>> You're right, it doesn't preview properly. However, your patch doesn't
->>> really fix it, since it only makes it show any old files that are lying
->>> around, and not a current one updated for the current page. Fixing it
->>> really right calls for embedding the file in a data uri in the preview
->>> page, as the sparkline plugin does. --[[Joey]]
+>>> No, the relevant code is just above the bit you patched. Notice that in
+>>> preview mode, it returns an img tag with the graphviz drawing embedded
+>>> in it, using base64 encoding. Unless you can show me why that doesn't
+>>> work, I still think this bug is done. --[[Joey]]
 
 >> Here is an updated patch againt ikiwiki-2.5:
 
 
 >> Here is an updated patch againt ikiwiki-2.5: