[Seaside] Force lowercase for attributes

Boris Popov boris at deepcovelabs.com
Tue Feb 13 17:37:28 UTC 2007


This would be a whole lot easier to discuss over a pitcher, don't you
think?

:)

-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


More information about the Seaside mailing list