Skip to content

bugzilla
bugzilla
Commit 5d5f0e06 authored by timeless%mac.com's avatar timeless%mac.com
Browse files
Options

Bugzilla Bug 97185 make_select should add a newline after each option

patch by kiko. r=zach, gerv
parent 1178d9a0
Hide whitespace changes
Inline Side-by-side
Showing with 5 additions and 5 deletions
CGI.pl
View file @ 5d5f0e06
...@@ -557,7 +557,7 @@ sub make_selection_widget { ...@@ -557,7 +557,7 @@ sub make_selection_widget {
} elsif ($type eq "RADIO") { } elsif ($type eq "RADIO") {
$popup .= "<INPUT NAME=$groupname type=radio VALUE=\"$item\" check>$displaytext<br>"; $popup .= "<INPUT NAME=$groupname type=radio VALUE=\"$item\" check>$displaytext<br>";
} else { } else {
$popup .= "<OPTION SELECTED VALUE=\"$item\">$displaytext"; $popup .= "<OPTION SELECTED VALUE=\"$item\">$displaytext\n";
} }
$found = 1; $found = 1;
} else { } else {
...@@ -578,7 +578,7 @@ sub make_selection_widget { ...@@ -578,7 +578,7 @@ sub make_selection_widget {
} elsif ($type eq "RADIO") { } elsif ($type eq "RADIO") {
$popup .= "<INPUT NAME=$groupname type=radio checked>$default"; $popup .= "<INPUT NAME=$groupname type=radio checked>$default";
} else { } else {
$popup .= "<OPTION SELECTED>$default"; $popup .= "<OPTION SELECTED>$default\n";
} }
} }
if ($type eq "LIST") { if ($type eq "LIST") {
...@@ -605,10 +605,10 @@ sub make_options { ...@@ -605,10 +605,10 @@ sub make_options {
} }
$last = $item; $last = $item;
if ($isregexp ? $item =~ $default : $default eq $item) { if ($isregexp ? $item =~ $default : $default eq $item) {
$popup .= "<OPTION SELECTED VALUE=\"$item\">$item"; $popup .= "<OPTION SELECTED VALUE=\"$item\">$item\n";
$found = 1; $found = 1;
} else { } else {
$popup .= "<OPTION VALUE=\"$item\">$item"; $popup .= "<OPTION VALUE=\"$item\">$item\n";
} }
} }
} }
...@@ -631,7 +631,7 @@ sub make_options { ...@@ -631,7 +631,7 @@ sub make_options {
exit 0; exit 0;
} else { } else {
$popup .= "<OPTION SELECTED>$default"; $popup .= "<OPTION SELECTED>$default\n";
} }
} }
return $popup; return $popup;
......
Markdown is supported
0% or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment