Skip Menu |

This queue is for tickets about the Tree-Simple-View CPAN distribution.

Report information
The Basics
Id: 16064
Status: resolved
Priority: 0/
Queue: Tree-Simple-View
People
Owner: stevan.little [...] gmail.com
Requestors: simonw [...] digitalcraftsmen.net
Cc:
AdminCc:
Bug Information
Severity: Wishlist
Broken in: (no value)
Fixed in: (no value)

History Show all quoted text
  Wed Nov 23 13:04:00 2005 Guest - Ticket created
Subject: Patch to provide optional xhtml compliance
Please consider the attached patch to add xhtml compliance (all tags and attribute names in lower case) to Tree::Simple::View::HTML via a tag_style option. A new test file t/22_Tree_Simple_View_HTML_tag_style.t provides tests and the documentation is updated. I've not patched Tree::Simple::View::DHTML as yet as there are more tags to consider and it's not an itch I need scratched right now :-) I did consider just patching the whole module into lowercase but decided there was probably a reason for you wanting capitals. All feedback welcome. Many thanks, Simon.

Message body is not shown because it is too large.

  Wed Nov 23 13:54:45 2005 stevan.little [...] gmail.com - Given to STEVAN
  Wed Nov 23 13:54:45 2005 stevan.little [...] gmail.com - Correspondence added
Simon, Thanks for the patch, I will review it and apply it this weekend. I will let you know when the new version is available. Thanks, Stevan [guest - Wed Nov 23 13:04:00 2005]: Show quoted text
> Please consider the attached patch to add xhtml compliance (all tags > and attribute names in lower case) to Tree::Simple::View::HTML via > a tag_style option. > > A new test file t/22_Tree_Simple_View_HTML_tag_style.t provides tests > and the documentation is updated. > > I've not patched Tree::Simple::View::DHTML as yet as there are more > tags to consider and it's not an itch I need scratched right now :- > ) > > I did consider just patching the whole module into lowercase but > decided there was probably a reason for you wanting capitals. > > All feedback welcome. > > Many thanks, > > Simon.
  Mon Nov 28 14:35:51 2005 stevan.little [...] gmail.com - Status changed from 'new' to 'resolved'
  Mon Nov 28 14:35:51 2005 stevan.little [...] gmail.com - Correspondence added
Simon, Patch has been applied and has been uploaded to PAUSE as version 0.14. Thanks again, Stevan [guest - Wed Nov 23 13:04:00 2005]: Show quoted text
> Please consider the attached patch to add xhtml compliance (all tags > and attribute names in lower case) to Tree::Simple::View::HTML via a > tag_style option. > > A new test file t/22_Tree_Simple_View_HTML_tag_style.t provides tests > and the documentation is updated. > > I've not patched Tree::Simple::View::DHTML as yet as there are more > tags to consider and it's not an itch I need scratched right now :-) > > I did consider just patching the whole module into lowercase but > decided there was probably a reason for you wanting capitals. > > All feedback welcome. > > Many thanks, > > Simon.