--- trunk/lib/Frey/Action.pm	2008/12/06 00:25:19	731
+++ trunk/lib/Frey/Action.pm	2008/12/09 20:31:43	783
@@ -26,6 +26,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;
@@ -106,13 +114,51 @@
 
 	sub select_values {
 		my ( $name, $attr_type, $values ) = @_;
-		my $options = join("\n",
-			map {
-				my $v = ref($_) eq 'HASH' ? $_->{$name} : $_;
-				qq|<option value="$v">$v</option>| if $v;
-			} @$values
-		);
-		qq|<select title="$attr_type" name="$name">$options</select>| if $options;
+
+		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";
+
+		if ( $#values > 3 ) {
+			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'} ) ) {
@@ -125,13 +171,6 @@
 
 	my @checkboxes;
 
-	my $skip_prefix;
-	if ( $self->class->can('action') ) {
-		my $action = eval $self->class . '->action';
-		$skip_prefix->{$_}++ foreach eval $self->class .'->action_order';
-		warn "# skip_prefix = ", $self->dump( $skip_prefix );
-	}
-
 	my $label_width = 1; # minimum
 
 	foreach my $name (
@@ -155,7 +194,7 @@
 			$attr->has_default        ? $attr->default( $name ) :
 			'';
 
-		if ( ref($params_config) eq 'HASH' ) {
+		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 );
@@ -177,7 +216,8 @@
 
 		$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<br>|;
@@ -206,8 +246,8 @@
 
 	my $html;
 
-	$form =~ s{<([^>]+)(name=")([^"]+)(")([^>]*)>}
-		{<$1$2$3$4 id="$3" $5}gs; # http://www.quirksmode.org/oddsandends/forms.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>