--- trunk/lib/Frey/ObjectBrowser.pm	2008/07/09 20:47:27	75
+++ trunk/lib/Frey/ObjectBrowser.pm	2008/07/13 12:40:45	107
@@ -1,8 +1,14 @@
 package Frey::ObjectBrowser;
 use Moose;
-use MooseX::Method;
 
 extends 'Frey';
+with 'Frey::Web';
+
+has 'fey_class' => (
+	is => 'rw',
+	isa => 'Str',
+	required => 1,
+);
 
 my @callbacks;
 
@@ -24,53 +30,84 @@
 #use Data::Dumper;
 #$Data::Dumper::Deparse = 1;
 use Data::Dump qw/dump/;
-use Data::Structure::Util qw(unbless);
 
+use Frey::Web::Row;
 
-#method 'html' => positional(
-#	isa => 'Continuity::RequestHolder',
-#	required => 1,
-#) => sub {
 sub html {
 	my ( $self, $req ) = @_;
 
-	my $params = $req->param;
-	warn "# params = ",dump( $params );
-
-	use Strix::User;
-	use Strix::View::User;
+	my $f = { $req->params };
+	my $layout = delete( $f->{layout} ) || 'div';
 
-	my $s = Strix::User->rows();
-	warn "# s = ",dump( $s );
+	my $s = $self->fey_class->collection();
+	warn "# s ",$self->fey_class," = ",dump( $s ) if $self->debug;
 
-	my $f = {};
 	my @w;
 
 	while ( my $u = $s->next ) {
-		warn "# u = ",dump( $u );
-		my $w = Strix::View::User->new(
-			#%{ unbless( $u ) }, # FIXME this is cludge
-			%{ $u }	# much better :-)
-		);
-		$w->_fey( $u );
-		warn "# w = ",dump( $w );
+		my $w = Frey::Web::Row->new( fey_class => $self->fey_class, fey => $u, layout => $layout );
+		warn "# w = ",dump( $w ) if $self->debug;
 		push @w, $w;
 	}
 
-	push @w, Strix::View::User->new( render_as => 'edit' );
-
 	while ( 1 ) {
 	
-		warn "# got $#w elements\n";
+		warn "# got $#w elements: ",dump( map { $_->id } @w );
+		warn "<<< GOT REQUEST ",dump( $f );
 
-		my $form = join('<hr/>', map { $_->process($f) } @w );
-	    my $doc = Continuity::Widget::DomNode->create(
+		# re-apply (changed?) layout to elements
+		if ( defined $f->{layout} ) {
+			$layout = delete( $f->{layout} );
+			warn "# new layout = $layout\n";
+		}
+
+		my $rows_delimiter = '';
+		$rows_delimiter = '<hr/>' if $layout eq 'div';
+
+		my $form = join($rows_delimiter, map {
+			$_->layout( $layout );		# FIXME do we need this still?
+			$_->process($f) || ''
+		} @w );
+
+		if ( $w[ $#w ]->fey ) {
+			warn "+++ added new item widget\n";
+			my $w = Frey::Web::Row->new( fey_class => $self->fey_class, render_as => 'edit', layout => $layout );
+			$form .= $rows_delimiter . $w->process;
+			push @w, $w;
+		}
+
+		$form = qq|<table>$form</table>| if $layout eq 'table';
+
+		if ( $layout eq 'columns' ) {
+			# add header to table
+			my $header = $w[0]->render_iterator( sub {
+				my ( $name, $field_name, $label, $val ) = @_;
+				return if $name =~ /^_/;
+				return qq|<th>$label</th>|;
+			});
+			$form = qq|<table><tr>$header</tr>$form</table>|;
+		}
+
+		my $layout_picker = join(' ',
+			qq|<div class="layout">layout:|,
+			map {
+				if ( $layout eq $_ ) {
+					qq|<em>$_</em>|;
+				} else {
+					qq|<a href="?layout=$_">$_</a>|;
+				}
+			} ( 'div', 'table', 'columns' ),
+			qq|</div>|,
+		);
+
+	    my $html = dom2html(
 			html => [
 				head => [
-					link  => { rel=>"stylesheet", href=>"form.css", type=>"text/css" }
+					link  => { rel=>"stylesheet", href=>"/static/form.css", type=>"text/css" }
 				],
 				body => [
-					h1 => [ 'Strix::User' ],
+					h1 => [ $self->fey_class ],
+					$layout_picker,
 					form => { method=>'post' } => [
 						$form,
 					],
@@ -78,10 +115,14 @@
 			]
 		);
 
-		$req->print( $doc->to_string );
+		warn ">>> $layout ",length( $html ),"\n";
+
+		$req->print( $html );
 		$req->next;
 
 		$f = { $req->params };
+
+		process_links( $req );
 	}
 
 	warn "NO WAY OUT!";