[Seaside] Force lowercase for attributes

Philippe Marschall philippe.marschall at gmail.com
Tue Feb 13 17:48:06 UTC 2007


2007/2/13, Boris Popov <boris at deepcovelabs.com>:
> This would be a whole lot easier to discuss over a pitcher, don't you
> think?

Yepp, Squeak Stammtisch, ESUG, ... what do you propose?

> :)
>
> -Boris
>
> --
> +1.604.689.0322
> DeepCove Labs Ltd.
> 4th floor 595 Howe Street
> Vancouver, Canada V6C 2T5
> http://tinyurl.com/r7uw4
>
> boris at deepcovelabs.com
>
> CONFIDENTIALITY NOTICE
>
> This email is intended only for the persons named in the message
> header. Unless otherwise indicated, it contains information that is
> private and confidential. If you have received it in error, please
> notify the sender and delete the entire message including any
> attachments.
>
> Thank you.
>
> -----Original Message-----
> From: seaside-bounces at lists.squeakfoundation.org
> [mailto:seaside-bounces at lists.squeakfoundation.org] On Behalf Of
> Philippe Marschall
> Sent: Tuesday, February 13, 2007 9:31 AM
> To: The Squeak Enterprise Aubergines Server - general discussion.
> Subject: Re: [Seaside] Force lowercase for attributes
>
> 2007/2/13, Boris Popov <boris at deepcovelabs.com>:
> > Sorry, Philippe, I really didn't mean to start this back and forth,
> > that's not really the point. My point was that if you feed contents of
> > the page with any of xhtml doctypes that has non-lowercase attributes,
> > it will fail validation
>
> That has now been fixed.
>
> > and my understanding was that we'd all strived
> > to have our framework produce standards compliant code to the best of
> > our abilities.
>
> We are far away from that, we could do much more. We could only allow
> the creation of elements that are defined and could check every
> attribute and it's value. We could enforce proper nesting of elements.
>
> Philippe
>
> > Browser quirks such as treatment (or non-treatment, I
> > should say) of text/xml don't play into this at all,
> >
> > Cheers!
> >
> > -Boris
> >
> > --
> > +1.604.689.0322
> > DeepCove Labs Ltd.
> > 4th floor 595 Howe Street
> > Vancouver, Canada V6C 2T5
> > http://tinyurl.com/r7uw4
> >
> > boris at deepcovelabs.com
> >
> > CONFIDENTIALITY NOTICE
> >
> > This email is intended only for the persons named in the message
> > header. Unless otherwise indicated, it contains information that is
> > private and confidential. If you have received it in error, please
> > notify the sender and delete the entire message including any
> > attachments.
> >
> > Thank you.
> >
> > -----Original Message-----
> > From: seaside-bounces at lists.squeakfoundation.org
> > [mailto:seaside-bounces at lists.squeakfoundation.org] On Behalf Of
> > Philippe Marschall
> > Sent: Tuesday, February 13, 2007 9:17 AM
> > To: The Squeak Enterprise Aubergines Server - general discussion.
> > Subject: Re: [Seaside] Force lowercase for attributes
> >
> > 2007/2/13, Boris Popov <boris at deepcovelabs.com>:
> > > But it still says XHTML, doesn't it? :)
> >
> > Doesn't matter a thing as long as the mime-type is:
> > text/html
> >
> > > Either way, this thread has deviated way far from the path I was on
> > > originally so I'll let it go as I have no way to contribute changes
> > > directly to Squeak codebase and trying to convince someone else to
> do
> > > the work just doesn't seem to be working out, understandably so, by
> > the
> > > way.
> > >
> > > Thanks!
> > >
> > > -Boris
> > >
> > > --
> > > +1.604.689.0322
> > > DeepCove Labs Ltd.
> > > 4th floor 595 Howe Street
> > > Vancouver, Canada V6C 2T5
> > > http://tinyurl.com/r7uw4
> > >
> > > boris at deepcovelabs.com
> > >
> > > CONFIDENTIALITY NOTICE
> > >
> > > This email is intended only for the persons named in the message
> > > header. Unless otherwise indicated, it contains information that is
> > > private and confidential. If you have received it in error, please
> > > notify the sender and delete the entire message including any
> > > attachments.
> > >
> > > Thank you.
> > >
> > > -----Original Message-----
> > > From: seaside-bounces at lists.squeakfoundation.org
> > > [mailto:seaside-bounces at lists.squeakfoundation.org] On Behalf Of
> > > Philippe Marschall
> > > Sent: Tuesday, February 13, 2007 9:08 AM
> > > To: The Squeak Enterprise Aubergines Server - general discussion.
> > > Subject: Re: [Seaside] Force lowercase for attributes
> > >
> > > 2007/2/13, Boris Popov <boris at deepcovelabs.com>:
> > > >
> > > >
> > > >
> > > > Why do you say its transitional?
> > >
> > > Because WAHtmlRoot >> #initialize says so and we still have support
> > > for deprecated attributes.
> > >
> > > Philippe
> > >
> > > > Our app validates as strict and Michel had
> > > > integrated quite a few changes to make it happen. Why could we not
> > > separate
> > > > the two and do the best we can to assist those who care about web
> > > standards.
> > > >
> > > >  Cheers!
> > > >
> > > >  -Boris
> > > >  (Sent from a BlackBerry)
> > > >
> > > >  ----- Original Message -----
> > > >  From: seaside-bounces at lists.squeakfoundation.org
> > > > <seaside-bounces at lists.squeakfoundation.org>
> > > >  To: The Squeak Enterprise Aubergines Server - general discussion.
> > > > <seaside at lists.squeakfoundation.org>
> > > >  Sent: Tue Feb 13 08:49:24 2007
> > > >  Subject: Re: [Seaside] Force lowercase for attributes
> > > >
> > > >  2007/2/13, Boris Popov <boris at deepcovelabs.com>:
> > > >  >
> > > >  >
> > > >  >
> > > >  > I'm away from a pc, but look for setter of onKeyDown on a body,
> > it
> > > was
> > > > cased
> > > >  > like that and failed validation,
> > > >
> > > >  Got that sucker.
> > > >
> > > >  > but my point was that so long as protocol
> > > >  > is available and manages xhtml, results should validate no
> matter
> > > what.
> > > >  > Isn't xml just a superset of xhtml?
> > > >
> > > >  It is. So attribute names are case-sensitive. But keep in mind
> that
> > > >  Seaside uses html-compatible xhtml, which is an undocumented hack
> > > that
> > > >  builds upon several undocumented hacks. As far as browsers are
> > > >  concerned seaside produces html 4 transitional.
> > > >
> > > >  > I was advocation for facilities to
> > > >  > support both but with their own respective rulesets.
> > > >
> > > >  They have both the very same ruleset:
> > > >  attribute names are case-sensitive
> > > >  Only in html are attribute names case insensitive.
> > > >
> > > >  Philippe
> > > >
> > > >  >  Cheers!
> > > >  >
> > > >  >  -Boris
> > > >  >  (Sent from a BlackBerry)
> > > >  >
> > > >  >
> > > >  >  ----- Original Message -----
> > > >  >  From: seaside-bounces at lists.squeakfoundation.org
> > > >  > <seaside-bounces at lists.squeakfoundation.org>
> > > >  >  To: The Squeak Enterprise Aubergines Server - general
> > discussion.
> > > >  > <seaside at lists.squeakfoundation.org>
> > > >  >  Sent: Mon Feb 12 22:17:24 2007
> > > >  >  Subject: Re: [Seaside] Force lowercase for attributes
> > > >  >
> > > >  >  2007/2/12, Boris Popov <boris at deepcovelabs.com>:
> > > >  >  > Then we should have WAHtmlAttributes and WAXmlAttributes,
> > > depending on
> > > >  >  > application's configuration?
> > > >  >
> > > >  >  I'd rather say deeding on the brush.
> > > >  >
> > > >  >  But you shouldn't have to use #attributeAt:put: anyway. I
> > thought
> > > I
> > > >  >  fixed all senders in Seaside. Do I have overlooked something
> or
> > do
> > > you
> > > >  >  miss certain attributes on certain elements?
> > > >  >
> > > >  >  Philippe
> > > >  >
> > > >  >  > -Boris
> > > >  >  >
> > > >  >  > --
> > > >  >  > +1.604.689.0322
> > > >  >  > DeepCove Labs Ltd.
> > > >  >  > 4th floor 595 Howe Street
> > > >  >  > Vancouver, Canada V6C 2T5
> > > >  >  > http://tinyurl.com/r7uw4
> > > >  >  >
> > > >  >  > boris at deepcovelabs.com
> > > >  >  >
> > > >  >  > CONFIDENTIALITY NOTICE
> > > >  >  >
> > > >  >  > This email is intended only for the persons named in the
> > message
> > > >  >  > header. Unless otherwise indicated, it contains information
> > that
> > > is
> > > >  >  > private and confidential. If you have received it in error,
> > > please
> > > >  >  > notify the sender and delete the entire message including
> any
> > > >  >  > attachments.
> > > >  >  >
> > > >  >  > Thank you.
> > > >  >  >
> > > >  >  > -----Original Message-----
> > > >  >  > From: seaside-bounces at lists.squeakfoundation.org
> > > >  >  > [mailto:seaside-bounces at lists.squeakfoundation.org]
> > > > On
> > > >  > Behalf Of
> > > >  >  > Philippe Marschall
> > > >  >  > Sent: Monday, February 12, 2007 1:26 PM
> > > >  >  > To: The Squeak Enterprise Aubergines Server - general
> > > discussion.
> > > >  >  > Subject: Re: [Seaside] Force lowercase for attributes
> > > >  >  >
> > > >  >  > 2007/2/12, Boris Popov <boris at deepcovelabs.com>:
> > > >  >  > > XHTML's attribute names are all lower-case, so how about
> the
> > > > following
> > > >  >  > > change?
> > > >  >  > >
> > > >  >  > > WAHtmlAttributes>>at: keyString put: valueString
> > > >  >  > >   valueString ifNotNil:
> > > >  >  > >      [self attributes
> > > >  >  > >            at: keyString asLowercase
> > > >  >  > >            put: valueString]
> > > >  >  >
> > > >  >  > Please no, we had that once and I itentionally removed it.
> The
> > > problem
> > > >  >  > is that although that all the XHTML attribute names are
> > > lowercase that
> > > >  >  > is not true for all XML like RSS.
> > > >  >  >
> > > >  >  > Philippe
> > > >  >  >
> > > >  >  > > By the way, I don't Squeak, otherwise I wouldn't have
> > bothered
> > > the
> > > >  >  > list
> > > >  >  > > with it ;)
> > > >  >  > >
> > > >  >  > > This ensure that code like,
> > > >  >  > >
> > > >  >  > >
> > > >  >  > > -Boris
> > > >  >  > >
> > > >  >  > > --
> > > >  >  > > +1.604.689.0322
> > > >  >  > > DeepCove Labs Ltd.
> > > >  >  > > 4th floor 595 Howe Street
> > > >  >  > > Vancouver, Canada V6C 2T5
> > > >  >  > > http://tinyurl.com/r7uw4
> > > >  >  > >
> > > >  >  > > boris at deepcovelabs.com
> > > >  >  > >
> > > >  >  > > CONFIDENTIALITY NOTICE
> > > >  >  > >
> > > >  >  > > This email is intended only for the persons named in the
> > > message
> > > >  >  > > header. Unless otherwise indicated, it contains
> information
> > > that is
> > > >  >  > > private and confidential. If you have received it in
> error,
> > > please
> > > >  >  > > notify the sender and delete the entire message including
> > any
> > > >  >  > > attachments.
> > > >  >  > >
> > > >  >  > > Thank you.
> > > >  >  > >
> > > >  >  > > _______________________________________________
> > > >  >  > > Seaside mailing list
> > > >  >  > > Seaside at lists.squeakfoundation.org
> > > >  >  > >
> > > >  >
> > > > http://lists.squeakfoundation.org/cgi-bin/mailman/listinfo/seaside
> > > >  >  > >
> > > >  >  > _______________________________________________
> > > >  >  > Seaside mailing list
> > > >  >  > Seaside at lists.squeakfoundation.org
> > > >  >  >
> > > >  >
> > > > http://lists.squeakfoundation.org/cgi-bin/mailman/listinfo/seaside
> > > >  >  > _______________________________________________
> > > >  >  > Seaside mailing list
> > > >  >  > Seaside at lists.squeakfoundation.org
> > > >  >  >
> > > >  >
> > > > http://lists.squeakfoundation.org/cgi-bin/mailman/listinfo/seaside
> > > >  >  >
> > > >  >  _______________________________________________
> > > >  >  Seaside mailing list
> > > >  >  Seaside at lists.squeakfoundation.org
> > > >  >
> > > > http://lists.squeakfoundation.org/cgi-bin/mailman/listinfo/seaside
> > > >  >
> > > >  > _______________________________________________
> > > >  > Seaside mailing list
> > > >  > Seaside at lists.squeakfoundation.org
> > > >  >
> > > > http://lists.squeakfoundation.org/cgi-bin/mailman/listinfo/seaside
> > > >  >
> > > >  >
> > > >  _______________________________________________
> > > >  Seaside mailing list
> > > >  Seaside at lists.squeakfoundation.org
> > > > http://lists.squeakfoundation.org/cgi-bin/mailman/listinfo/seaside
> > > >
> > > >
> > > >
> > > > _______________________________________________
> > > > Seaside mailing list
> > > > Seaside at lists.squeakfoundation.org
> > > > http://lists.squeakfoundation.org/cgi-bin/mailman/listinfo/seaside
> > > >
> > > >
> > > _______________________________________________
> > > Seaside mailing list
> > > Seaside at lists.squeakfoundation.org
> > > http://lists.squeakfoundation.org/cgi-bin/mailman/listinfo/seaside
> > > _______________________________________________
> > > Seaside mailing list
> > > Seaside at lists.squeakfoundation.org
> > > http://lists.squeakfoundation.org/cgi-bin/mailman/listinfo/seaside
> > >
> > _______________________________________________
> > Seaside mailing list
> > Seaside at lists.squeakfoundation.org
> > http://lists.squeakfoundation.org/cgi-bin/mailman/listinfo/seaside
> > _______________________________________________
> > Seaside mailing list
> > Seaside at lists.squeakfoundation.org
> > http://lists.squeakfoundation.org/cgi-bin/mailman/listinfo/seaside
> >
> _______________________________________________
> Seaside mailing list
> Seaside at lists.squeakfoundation.org
> http://lists.squeakfoundation.org/cgi-bin/mailman/listinfo/seaside
> _______________________________________________
> Seaside mailing list
> Seaside at lists.squeakfoundation.org
> http://lists.squeakfoundation.org/cgi-bin/mailman/listinfo/seaside
>


More information about the Seaside mailing list