[Seaside] Conditional comments types and VirtualEarth example

[Philippe Marschall]

Hi

I updated the VirtualEarth example with this and it runs fine on
FireFox and Opera with the latest Seaside (2.8 or 2.9). The code also
validates.

Cheers
Philippe

2007/11/25, Philippe Marschall <philippe.marschall at gmail.com>:
> Hi
>
> Can you check out Seaside2.8a1-pmm.532
>
> html revealedIf not; ie7; do: [
>         html script url: 'http://local.live.com/JS/AtlasCompat.js' ]
>
> Cheers
> Philippe
>
> 2007/11/21, astares at gmx.de <astares at gmx.de>:
> > Hi,
> >
> > according to http://msdn2.microsoft.com/en-us/library/ms537512.aspx
> >
> > there are two types of conditional comments: "downlevel-hidden" and
> > "downlevel-revealed". Currently the seaside class WAOpeningConditionalComment seems to support only downlevel-hidden (see WAOpeningConditionalComment>>encodeOn:)
> >
> >
> > This makes some trouble to make my VirtualEarth example working with Firefox 2.0.0.9 on Windows.
> > The VirtualEarth control is by default working with IE and needs a patch for firefox. (see http://blogs.msdn.com/virtualearth/archive/2006/02/13/531896.aspx)
> >
> > updateRoot: anHtmlRoot
> >
> >    anHtmlRoot if not; ie; do: [
> >       anHtmlRoot script url: 'http://local.live.com/JS/AtlasCompat.js']
> >
> > See http://www.squeaksource.com/SeasideExamples/VirtualEarthDemo-tbn.1.mcz
> > for the complete code and a sample. Note that the "not" operator is an extension I would like to see in base seaside.
> >
> > But with Firefox 2.0.0.9 on Windows the example is starts to work when I change WAOpeningConditionalComment>>encodeOn: by removing the "--" into
> >
> >   encodeOn: aDocument
> >     aDocument nextPutAll: '<!['; nextPutAll: self condition; nextPutAll: ']>'
> >
> > Looks like a "downlevel-revealed" is required here. Any plans to support both types of conditional comments in the "if"/WAConditionalComment.
> >
> > Any thoughts?
> >
> > Bye
> > Torsten
> >
> >
> >
> >
> > --
> > Psssst! Schon vom neuen GMX MultiMessenger gehört?
> > Der kann`s mit allen: http://www.gmx.net/de/go/multimessenger
> > _______________________________________________
> > seaside mailing list
> > seaside at lists.squeakfoundation.org
> > http://lists.squeakfoundation.org/cgi-bin/mailman/listinfo/seaside
> >
>