2 Aug 2010 09:39
Re: Seafox HTML -> Seaside render methods translator
Nick Ager <nick.ager <at> gmail.com>
2010-08-02 07:39:28 GMT
2010-08-02 07:39:28 GMT
Hi Philippe,
The algorithm you use seems to work well for body tags but have
problems with header tags. For example look at the style tag:
- there's no such tag, simply the link tag which explains the selector
- it says the rel attribute is set with #beAlternateStylesheet
- it says the media attribute is set with #addHandheld
- it misses the attributes from the superclass
- it says the type attribute is set with #beRss
Apart from that, very nice work, thank you.
Thanks for the feedback. You're right I didn't spend as long on the header mappings, though I added a special case in the parser for the link tag so that it picks up some of the common link to stylesheet mappings. See SFLibrary>>#browserJs function createLinkHTMLElementBrush (aNode). The problem with putting special cases in the parser is that it leads to a discrepancy between what the web app shows the mappings to be and the results the parser generates. Though hopefully the parser output is more interesting to people.
Perhaps we can come up with a better approach at ESUG?
Cheers
Nick
_______________________________________________ seaside mailing list seaside <at> lists.squeakfoundation.org http://lists.squeakfoundation.org/cgi-bin/mailman/listinfo/seaside
RSS Feed