HaxeFlixel / flixel-docs

Documentation for HaxeFlixel
71 stars 135 forks source link

API docs target dropdown #132

Open Gama11 opened 9 years ago

Gama11 commented 9 years ago

Currently we only have flash there, would be nice to have the remaining targets as well (html5, cpp, neko, android, ios...)

Gama11 commented 9 years ago

Generating the xml files works fine, but dox has some issues with them when trying to generate the docs:

Parsing ./flash.xml
Parsing ./neko.xml
Called from ? line 1
Called from dox/Dox.hx line 137
Called from dox/Dox.hx line 130
Called from C:\HaxeToolkit\haxe\std/haxe/rtti/XmlParser.hx line 84
Called from C:\HaxeToolkit\haxe\std/haxe/rtti/XmlParser.hx line 311
Called from C:\HaxeToolkit\haxe\std/haxe/rtti/XmlParser.hx line 278
Uncaught exception - Incompatibilities between UInt in flash and neko (documenta
tion is different)
Gama11 commented 8 years ago

Since https://github.com/HaxeFlixel/flixel/pull/1668, there really isn't all that much API that exclusively exists on a specific target, so this is probably not worth the time investment.

ashes999 commented 8 years ago

This is interesting. I was going to suggest (since the issue I ran into is "field missing in docs") pick a target that has the most fields/methods and publish those, instead. (You can use XML size as a rough gauge of this.)

Is it worth pursuing this?

Should we see if the (downstream) exception can be fixed in Dox?

Gama11 commented 8 years ago

I don't think this is a dox issue.

ashes999 commented 8 years ago

Is it worth digging down to the root cause to see if this is fixable?