[Rt-commit] r2451 - in rt/branches/PLATANO-EXPERIMENTAL-CSS: . html/Elements/CollectionAsTable html/Elements/RT__Ticket

[jesse at bestpractical.com]

Author: jesse
Date: Mon Mar 14 03:03:10 2005
New Revision: 2451

Modified:
   rt/branches/PLATANO-EXPERIMENTAL-CSS/   (props changed)
   rt/branches/PLATANO-EXPERIMENTAL-CSS/html/Elements/CollectionAsTable/ParseFormat
   rt/branches/PLATANO-EXPERIMENTAL-CSS/html/Elements/CollectionAsTable/Row
   rt/branches/PLATANO-EXPERIMENTAL-CSS/html/Elements/RT__Ticket/ColumnMap
Log:
 r8611 at hualien:  jesse | 2005-03-14 02:43:06 -0500
  r8499 at hualien:  jesse | 2005-03-14 02:28:22 -0500
   r6454 at hualien:  jesse | 2005-03-06 14:53:48 -0500
    r2568 at hualien:  jesse | 2005-01-06 16:57:40 -0500
    RT-Ticket: 6336
    RT-Status: resolved
    RT-Update: correspond
    
    
    The attribute "align" in COLUMN_MAP wasn't usable, because it could not be
    used for a complete column, so I added a /ALIGN:.. section in 
    CollectionAsTable/ParseFormat and use it in Row.
    
    There also was a minor error to be corrected, since the old variant of 
    ParseFormat did not allow more than one /ANYTHING:... to be use (in my 
    example /TITLE:.../ALIGN:...)  --Dirk Pape
    
    
    
   
  
 


Modified: rt/branches/PLATANO-EXPERIMENTAL-CSS/html/Elements/CollectionAsTable/ParseFormat
==============================================================================
--- rt/branches/PLATANO-EXPERIMENTAL-CSS/html/Elements/CollectionAsTable/ParseFormat	(original)
+++ rt/branches/PLATANO-EXPERIMENTAL-CSS/html/Elements/CollectionAsTable/ParseFormat	Mon Mar 14 03:03:10 2005
@@ -60,15 +60,19 @@
     }
 
     my $colref;
-    if ( $col =~ s/\/STYLE:(.*?)$//io ) {
+
+    if ( $col =~ s!/STYLE:([^/]+)!!io ) {
         $colref->{'style'} = $1;
     }
-    if ( $col =~ s/\/CLASS:(.*?)$//io ) {
+    if ( $col =~ s!/CLASS:([^/]+)!!io ) {
         $colref->{'class'} = $1;
     }
-    if ( $col =~ s/\/TITLE:(.*?)$//io ) {
+    if ( $col =~ s!/TITLE:([^/]+)!!io ) {
         $colref->{'title'} = $1;
     }
+    if ( $col =~ s!/ALIGN:([^\/]+)!!io ) {
+        $colref->{'align'} = $1;
+    }
     if ( $col =~ /__(.*?)__/gio ) {
         my @subcols;
         while ( $col =~ s/^(.*?)__(.*?)__//o ) {

Modified: rt/branches/PLATANO-EXPERIMENTAL-CSS/html/Elements/CollectionAsTable/Row
==============================================================================
--- rt/branches/PLATANO-EXPERIMENTAL-CSS/html/Elements/CollectionAsTable/Row	(original)
+++ rt/branches/PLATANO-EXPERIMENTAL-CSS/html/Elements/CollectionAsTable/Row	Mon Mar 14 03:03:10 2005
@@ -68,7 +68,8 @@
         next;
     }
     $item++;
-    $m->out('<td class="collection-as-table">');
+	my $align = ($column->{align} || "left");
+    $m->out('<td class="collection-as-table" align="' . $align . '">');
     foreach my $subcol ( @{ $column->{output} } ) {
         if ( $subcol =~ /^__(.*?)__$/o ) {
             my $col = $1;

Modified: rt/branches/PLATANO-EXPERIMENTAL-CSS/html/Elements/RT__Ticket/ColumnMap
==============================================================================
--- rt/branches/PLATANO-EXPERIMENTAL-CSS/html/Elements/RT__Ticket/ColumnMap	(original)
+++ rt/branches/PLATANO-EXPERIMENTAL-CSS/html/Elements/RT__Ticket/ColumnMap	Mon Mar 14 03:03:10 2005
@@ -293,10 +293,6 @@
     },
 
 };
-
-
-
-# }}}
 </%ONCE>
 <%init>
 $m->comp( '/Elements/Callback', COLUMN_MAP    => $COLUMN_MAP, _CallbackName => 'ColumnMap');