--- trunk/lib/Frey/Action.pm	2008/11/20 14:39:43	480
+++ trunk/lib/Frey/Action.pm	2009/01/06 13:18:43	940
@@ -12,6 +12,11 @@
 Invoke any L<Frey> object creating html for with various default parameters
 if not supplied at invocation.
 
+You can force rendering of fields if you define C<render_attribute> sub with
+desired rendering as in:
+
+  sub render_pipe { 'radio' }
+
 =cut
 
 has 'class' => (
@@ -26,6 +31,14 @@
 	default => sub { {} },
 );
 
+has 'input_step_size' => (
+	documentation => 'Resize input fields by this step',
+	is => 'rw',
+	isa => 'Int',
+#	required => 1,
+	default => 20,
+);
+
 =head2 required
 
   my @required_attributes = $self->required;
@@ -36,6 +49,7 @@
 sub required {
 	my ( $self ) = @_;
 	$self->load_class( $self->class );
+
 	my @required = 
 		grep {
 			defined $_ && $_->can('name') &&
@@ -97,64 +111,176 @@
 
 	my $default = clone $self->params; # XXX we really don't want to modify params!
 
-	my $config_params = {};
-	$config_params = $self->config($class);
-	warn "# $class config = ",dump( $config_params ) if $self->debug;
+	my $params_config = {};
+	$params_config = $self->config($class);
+	warn "# $class config = ",dump( $params_config ) if $self->debug;
 
 	my $form;
 
-	foreach my $name ( grep { ! $class->meta->get_attribute($_)->is_lazy } $self->attributes ) {
+	sub select_values {
+		my ( $name, $attr_type, $values ) = @_;
+
+		my $max_value_len = 0;
+		my @values;
+		my $html = '';
+
+		foreach ( @$values ) {
+			my $v = ref($_) eq 'HASH' ? $_->{$name} : $_;
+			warn "## value '$v'";
+			push @values, $v;
+			$max_value_len = length($v) if length($v) > $max_value_len;
+		}
+
+		warn "# max_value_len: $max_value_len";
+		my $render = eval $class . '->render_' . $name;
+		warn "## render $@";
+
+		if ( $#values > 3 && $render !~ m{radio} ) {
+			my $options = join("\n",
+				map {
+					qq|<option value="$_">$_</option>|;
+				} @values
+			);
+			# onchange="alert(this.options[this.selectedIndex].value);"
+			$html = qq|
+				<select title="$attr_type" name="$name">
+				$options
+				</select>
+			| if $options;
+		} else {
+			my $delimiter = $max_value_len > $self->input_step_size ? qq|<br>| : '';
+			my $radio =
+#				$delimiter .
+				join("\n",
+				map { strip(qq|
+						<span title="$attr_type">
+						<input type="radio" name="$name" value="$_">
+						$_
+						</span>
+						$delimiter
+				|) } @values
+			);
+			$html = qq|<div style="display: block;">$radio</div>|;
+		}
+
+		return
+#			qq|<input type="text" name="$name">| .
+			$html
+	}
+
+	foreach my $checkbox ( split(/\s+/, $default->{'frey-checkboxes'} ) ) {
+		next if defined $default->{ $checkbox };
+
+		$default->{ $checkbox } = 0;
+		$self->params->{ $checkbox } = 0;
+		warn "# checkbox $checkbox not ticked";
+	}
+
+	my @checkboxes;
+
+	my $label_width = 1; # minimum
+
+	foreach my $name (
+		grep {
+			die "$_ doesn't have meta" unless $class->can('meta');
+			! $class->meta->get_attribute($_)->is_lazy
+			&& ! defined $default->{$_}
+			&& ! m{^_} # skip _private
+		} $self->attributes
+	) {
 		my $attr_type = '';
 		my $type = $name =~ m/^pass/ ? 'password' : 'text';
 		my $label = $name;
-		my $value = '';
 		my $label_title = '';
 		my $value_html = '';
-		if ( ref($config_params) eq 'HASH' ) {
-			$value = $config_params->{$name};
-		} elsif ( ref($config_params) eq 'ARRAY' ) {
-			$value_html = qq|<select name="$name">| . join("\n",
-				map {
-					my $v = $_->{$name};
-					qq|<option value="$v">$v</option>|
-				} @$config_params
-			) . qq|</select>|;
-			$default->{$name} = $config_params->[0]->{$name};
-		} elsif ( my $attr = $class->meta->get_attribute( $name ) ) {
-			if ( $attr->has_type_constraint ) {
-				$attr_type = $attr->type_constraint->name;
-				if ( $attr->type_constraint->can('values') ) {
-					$value_html = qq|<select name="$name">| . join("\n",
-						map { qq|<option value="$_">$_</option>| } @{ $attr->type_constraint->values }
-					) . qq|</select>|;
-				} elsif ( $attr_type !~ m{^(Str|Int)$} ) {
-					$value_html = qq|<textarea name="$name" title="$attr_type">$value</textarea>|;
-				}
-			}
-			$value = $attr->default( $name ) if ! $value && $attr->has_default;
-			$label_title = qq| title="| . $attr->documentation . qq|"| if $attr->has_documentation;
-		} else {
-			warn "wired attribute $name";
+
+		my $attr = $class->meta->get_attribute( $name );
+		$attr_type = $attr->type_constraint->name if $attr->has_type_constraint;
+
+		my $value =
+			defined $default->{$name} ? $default->{$name}       :
+			$attr->has_default        ? $attr->default( $name ) :
+			'';
+
+		if ( ref($params_config) eq 'HASH' && defined $params_config->{$name} ) {
+			$value = $params_config->{$name};
+		} elsif ( ref($params_config) eq 'ARRAY' ) {
+			$value_html = select_values( $name, $attr_type, $params_config );
+			$default->{$name} = $params_config->[0]->{$name};
+		} elsif ( $attr->has_type_constraint && $attr->type_constraint->can('values') ) {
+			$value_html = select_values( $name, $attr_type, $attr->type_constraint->values );
+		} elsif ( $attr_type =~ m{^Bool} ) {
+			my $suffix = '';
+			$suffix = ' checked' if $value;
+			$value_html = qq|<input type="checkbox" name="$name" title="$attr_type" value="$value"$suffix>|;
+			push @checkboxes, $name;
+		} elsif ( ! defined $value ) {
+			$value_html = qq|<tt id="$name">undef</tt><!-- $name = undef -->|; # FIXME if $self->debug
+		} elsif ( $attr_type !~ m{^(Str|Int)$} || $value =~ $Frey::Web::re_html || $name =~ m{text} ) {
+			$value_html = qq|<textarea name="$name" title="$attr_type">$value</textarea>|;
 		}
 
+		$label_title = qq| title="| . $attr->documentation . qq|"| if $attr->has_documentation;
+
 		$default->{$name} = $value unless defined $default->{$name};
 
-		$value_html = qq|<input type="$type" name="$name" title="$attr_type" value="$value">| unless $value_html;
+		my $size = ( int( length($value) / $self->input_step_size ) + 1 ) * $self->input_step_size;
+		$value_html = qq|<input type="$type" name="$name" title="$attr_type" value="$value" size="$size">| unless $value_html;
 
 #		warn "# required $name ", $class->meta->get_attribute( $name )->dump( 2 );
-		$form .= qq|<label for="$name"$label_title>$label</label>| . $value_html;
+		$form .= qq|<label for="$name"$label_title>$label</label>$value_html<br>|;
+		my $ll = length($label);
+		$label_width = $ll if $ll > $label_width;
 	}
-	my $html = qq|<h1>$class params</h1><form method="post">$form<input type="submit" value="Run $class"></form>|;
-	push @{ $self->status }, {
-		$self->editor( $self->class ) => {
+	$form .= qq|<input type="hidden" name="frey-checkboxes" value="| . join(' ', @checkboxes) . qq|">| if @checkboxes;
+
+	$self->add_css(qq|
+		label,input {
+			display: block;
+			float: left;
+			margin-bottom: 10px;
+		}
+
+		label {
+			text-align: right;
+			width: ${label_width}ex;
+			padding-right: 20px;
+		}
+
+		br {
+			clear: left;
+		}
+	|);
+
+	my $html;
+
+	# http://www.quirksmode.org/oddsandends/forms.html
+#	$form =~ s{<([^>]+)(name=")([^"]+)(")([^>]*)>}{<$1$2$3$4 id="$3" $5}gs;
+
+	$html = qq|
+		<h1>$class params</h1>
+		<form method="post">
+		$form
+		<input type="submit" value="Run $class">
+		</form>
+	| if $form;
+
+	$self->add_status({
+		$self->class => {
 			params  => $self->params,
-			config_params  => $config_params,
-			default => $default
+			params_config  => $params_config,
+			default => $default,
 		},
-	};
+	});
 
 	return ($html,$default) if wantarray;
 	return $html;
 }
 
+=head1 SEE ALSO
+
+L<http://www.quirksmode.org/css/forms.html> for info on CSS2 forms
+
+=cut
+
 1;