--- trunk/lib/Frey/SVK.pm	2008/11/28 16:39:37	591
+++ trunk/lib/Frey/SVK.pm	2008/11/28 19:23:38	603
@@ -7,7 +7,7 @@
 has commit_path => (
 	documentation => 'path to commit',
 	is => 'rw',
-	isa => 'Str',
+	isa => 'ArrayRef|Str',
 );
 
 has message => (
@@ -57,10 +57,20 @@
 sub status_as_markup {
 	my ($self) = @_;
 	my $status = `svk status -q`;
-	$status =~ s{^(\w+\s+)(\S+)$}{$1<a href="#$2">$2</a>}gm;
-	$self->add_css( qq| pre.l a { text-decoration: none; } | );
-	$status = qq|<pre class="l">$status</pre>|;
-	$self->add_status( $status );
+	$status =~ s{^(\w+\s+)(\S+)$}{$1<input name="commit_path" value="$2" type="checkbox"><a href="#$2">$2</a>}gm;
+	if ( $status ) {
+		$self->add_css( qq| pre.l a { text-decoration: none; } | );
+		$status = qq|
+			<form>
+			<div style="background: #ffd; float: right; padding: 1em;">
+				<textarea name="message" width=20 height=4></textarea>
+				<br><input type="submit" value="Commit">
+			</div>
+			<pre class="l">$status</pre>
+			</form>
+		|;
+		$self->add_status( $status );
+	}
 	warn "status_as_markup ",length($status)," bytes";
 	return $status;
 }
@@ -92,7 +102,10 @@
 		$self->TODO( "Frey::Web role missing" );
 	}
 
-	my $html = $self->status_as_markup . $self->diff_as_markup;
+	my $html
+		= ( $self->status_as_markup || $self->error('no status_or_markup output') )
+		. ( $self->diff_as_markup   || $self->error('no diff_as_markup output') )
+		;
 	warn "as_markup ",length($html)," bytes";
 
 	return $html;