Login | Register For Free | Help
Search for: (Advanced)

Mailing List Archive: Request Tracker: Devel

[3.8beta PATCH 1/3] xhtml: uppercase element names/attributes

 

 

Request Tracker devel RSS feed   Index | Next | Previous | View Threaded


jlong at messiah

Apr 10, 2008, 1:01 PM

Post #1 of 3 (255 views)
Permalink
[3.8beta PATCH 1/3] xhtml: uppercase element names/attributes

This patch finds a bunch of places where uppercase HTML attributes
were used and replaces them with the XHTML lowercase equivalents.
---

html/Admin/Elements/SelectCustomFieldLookupType | 2 +-
html/Admin/Elements/SelectCustomFieldType | 2 +-
html/Admin/Elements/SelectScrip | 4 ++--
html/Admin/Elements/SelectScripAction | 4 ++--
html/Admin/Elements/SelectScripCondition | 4 ++--
html/Admin/Elements/SelectSingleOrMultiple | 8 ++++----
html/Admin/Elements/SelectStage | 2 +-
html/Admin/Elements/SelectTemplate | 6 +++---
html/Elements/EditCustomFieldSelect | 6 +++---
html/Elements/SelectLang | 2 +-
html/Elements/SelectMatch | 16 ++++++++--------
html/Elements/SelectOwner | 4 ++--
html/Elements/SelectSortOrder | 2 +-
html/Elements/SelectStatus | 4 ++--
html/Elements/SelectTicketSortBy | 2 +-
html/Elements/SelectTicketTypes | 2 +-
html/Elements/SelectWatcherType | 2 +-
html/Search/Elements/SelectChartType | 2 +-
html/Search/Elements/SelectGroup | 2 +-
html/Search/Elements/SelectPersonType | 4 ++--
html/Ticket/Elements/ShowDates | 2 +-
html/Ticket/Update.html | 8 ++++----
lib/RT/Interface/Web/QueryBuilder/Tree.pm | 2 +-
23 files changed, 46 insertions(+), 46 deletions(-)

(Patch is attached)
Attachments: xhtml-uppercase-elements.patch (16.8 KB)


ruz at bestpractical

Apr 10, 2008, 1:51 PM

Post #2 of 3 (241 views)
Permalink
Re: [3.8beta PATCH 1/3] xhtml: uppercase element names/attributes [In reply to]

I'm going to apply, but please send me patch that returns back space
in "LIKE"<%...
It's very hard to read such code without spaces. I don't think there
are any problems with tag <xxx > or with many spaces between
attributes.

2008/4/11 Jason Long <jlong[at]messiah.edu>:
> This patch finds a bunch of places where uppercase HTML attributes
> were used and replaces them with the XHTML lowercase equivalents.
> ---
>
> html/Admin/Elements/SelectCustomFieldLookupType | 2 +-
> html/Admin/Elements/SelectCustomFieldType | 2 +-
> html/Admin/Elements/SelectScrip | 4 ++--
> html/Admin/Elements/SelectScripAction | 4 ++--
> html/Admin/Elements/SelectScripCondition | 4 ++--
> html/Admin/Elements/SelectSingleOrMultiple | 8 ++++----
> html/Admin/Elements/SelectStage | 2 +-
> html/Admin/Elements/SelectTemplate | 6 +++---
> html/Elements/EditCustomFieldSelect | 6 +++---
> html/Elements/SelectLang | 2 +-
> html/Elements/SelectMatch | 16 ++++++++--------
> html/Elements/SelectOwner | 4 ++--
> html/Elements/SelectSortOrder | 2 +-
> html/Elements/SelectStatus | 4 ++--
> html/Elements/SelectTicketSortBy | 2 +-
> html/Elements/SelectTicketTypes | 2 +-
> html/Elements/SelectWatcherType | 2 +-
> html/Search/Elements/SelectChartType | 2 +-
> html/Search/Elements/SelectGroup | 2 +-
> html/Search/Elements/SelectPersonType | 4 ++--
> html/Ticket/Elements/ShowDates | 2 +-
> html/Ticket/Update.html | 8 ++++----
> lib/RT/Interface/Web/QueryBuilder/Tree.pm | 2 +-
> 23 files changed, 46 insertions(+), 46 deletions(-)
>
> (Patch is attached)
>
> _______________________________________________
> List info: http://lists.bestpractical.com/cgi-bin/mailman/listinfo/rt-devel
>
>



--
Best regards, Ruslan.
_______________________________________________
List info: http://lists.bestpractical.com/cgi-bin/mailman/listinfo/rt-devel


ruz at bestpractical

Apr 10, 2008, 1:56 PM

Post #3 of 3 (241 views)
Permalink
Re: [3.8beta PATCH 1/3] xhtml: uppercase element names/attributes [In reply to]

done with
find html/ -type f | xargs perl -i -pwe 's/(\w")<%/$1 <%/g'

please check

On Fri, Apr 11, 2008 at 12:51 AM, Ruslan Zakirov <ruz[at]bestpractical.com> wrote:
> I'm going to apply, but please send me patch that returns back space
> in "LIKE"<%...
> It's very hard to read such code without spaces. I don't think there
> are any problems with tag <xxx > or with many spaces between
> attributes.
>
> 2008/4/11 Jason Long <jlong[at]messiah.edu>:
>
>
> > This patch finds a bunch of places where uppercase HTML attributes
> > were used and replaces them with the XHTML lowercase equivalents.
> > ---
> >
> > html/Admin/Elements/SelectCustomFieldLookupType | 2 +-
> > html/Admin/Elements/SelectCustomFieldType | 2 +-
> > html/Admin/Elements/SelectScrip | 4 ++--
> > html/Admin/Elements/SelectScripAction | 4 ++--
> > html/Admin/Elements/SelectScripCondition | 4 ++--
> > html/Admin/Elements/SelectSingleOrMultiple | 8 ++++----
> > html/Admin/Elements/SelectStage | 2 +-
> > html/Admin/Elements/SelectTemplate | 6 +++---
> > html/Elements/EditCustomFieldSelect | 6 +++---
> > html/Elements/SelectLang | 2 +-
> > html/Elements/SelectMatch | 16 ++++++++--------
> > html/Elements/SelectOwner | 4 ++--
> > html/Elements/SelectSortOrder | 2 +-
> > html/Elements/SelectStatus | 4 ++--
> > html/Elements/SelectTicketSortBy | 2 +-
> > html/Elements/SelectTicketTypes | 2 +-
> > html/Elements/SelectWatcherType | 2 +-
> > html/Search/Elements/SelectChartType | 2 +-
> > html/Search/Elements/SelectGroup | 2 +-
> > html/Search/Elements/SelectPersonType | 4 ++--
> > html/Ticket/Elements/ShowDates | 2 +-
> > html/Ticket/Update.html | 8 ++++----
> > lib/RT/Interface/Web/QueryBuilder/Tree.pm | 2 +-
> > 23 files changed, 46 insertions(+), 46 deletions(-)
> >
> > (Patch is attached)
> >
> > _______________________________________________
> > List info: http://lists.bestpractical.com/cgi-bin/mailman/listinfo/rt-devel
> >
> >
>
>
>
> --
> Best regards, Ruslan.
>



--
Best regards, Ruslan.
_______________________________________________
List info: http://lists.bestpractical.com/cgi-bin/mailman/listinfo/rt-devel

Request Tracker devel RSS feed   Index | Next | Previous | View Threaded
 
 


Interested in having your list archived? Contact lists@gossamer-threads.com
 
  Web Applications & Managed Hosting Powered by Gossamer Threads Inc.