X-Git-Url: http://git.vanrenterghem.biz/git.ikiwiki.info.git/blobdiff_plain/9645f0ccbc970050c5cccc2d03e277614af36d8a..f55e219d4f63d37a9b47c5f018e7245173be73f3:/doc/todo/html.mdwn

diff --git a/doc/todo/html.mdwn b/doc/todo/html.mdwn
index ba167ea62..46d4815ff 100644
--- a/doc/todo/html.mdwn
+++ b/doc/todo/html.mdwn
@@ -1,13 +1,44 @@
-Make the html valid. Add css and prettify. Make RecentChanges use table for
+Add css and prettify. Make RecentChanges use table for
 formatting, and images to indicate web vs svn commits and to link to diffs.
 
 All of this should be doable w/o touching a single line of code, just
 editing the [[templates]] BTW.
 
-Current problems:
+## html validation
+
+  * A doctype should be added: do we want XHTML 1.0 or HTML 4.01 Trans as
+    default?
+    
+	Need to choose a doctype that corresponds to what [[MarkDown]]
+	generates. For example, it does generate <hr />
+	So xhtml seems a good fit and I'm going to add that to the
+	templates. 
+	
+	One consideration of course is that regular users might embed html
+	that uses deprecated presentational elements like <center>. At
+        least firefox seems to handle that mixture ok. 
+	--[[Joey]]
+	
+  * If XHTML: In templates <hr> should become <hr /> etc.
+        
+    Done --[[Joey]]
 
-  * A doctype should be added: do we want XHTML 1.0 or HTML 4.01 Trans as default?
-  * If XHTML: In templates <hr> should become <hr\> etc.
   * Image wikilinks should provide an alt text (maybe '$filname wiki-image'?).
+
+  	Now it will be the text of the [[WikiLink]], which even allows
+	setting a custom alt text like this: \[[my_alt_text|image.png]]
+	--[[Joey]]
+	
   * &'s in (cgi-)url's must be escaped as &.
+  
+  	Fixed --[[Joey]]
+	
   * [ [inlinepage] ] gets wrapped in <p>...</p> which has a high chance of invalidating the page.
+
+  	Since markdown does this, the only way I can think to fix it is to
+	make the inlined page text start with </p> and end with
+	<p>. Ugly, and of course there could be problems with
+	markdown enclosing it in other spanning tags in some cases.
+	I've implemented this hack now. :-/ --[[Joey]]
+
+Test: [validate this page](http://validator.w3.org/check?url=referer)