Respect the "select friend colors" checkbox, so that if you select 
custom friends colors, then change your mind and uncheck the checkbox, we 
*don't* add the colors

        Patch by afuna. 

diffs (23 lines):

diff -r a6fda2a24dea -r 12fbf1b192f0 htdocs/manage/circle/add.bml
--- a/htdocs/manage/circle/add.bml      Sat Nov 21 16:40:05 2009 -0600
+++ b/htdocs/manage/circle/add.bml      Mon Nov 23 16:26:55 2009 +0000
@@ -76,8 +76,8 @@
         }
 
         if ( $POST{add_watch} ) {
-            my $fg = LJ::color_todb( $POST{editfriend_add_1_fg} );
-            my $bg = LJ::color_todb( $POST{editfriend_add_1_bg} );
+            my $fg = $POST{color_switch} ? LJ::color_todb( 
$POST{editfriend_add_1_fg} ) : "";
+            my $bg = $POST{color_switch} ? LJ::color_todb( 
$POST{editfriend_add_1_bg} ) : "";
             $remote->add_edge( $u, watch => {
                 fgcolor => $fg,
                 bgcolor => $bg,
@@ -270,7 +270,7 @@
 
     ## let them pick the colors
     my $color_text = "<?p <table><tr><td 
valign=\"top\"><i>$ML{'.optional'}:</i> ".
-             "<input type=\"checkbox\" id=\"color_switch\" value=\"\" 
onClick=\"color_display(this);\"> </td><td><label for=\"color_switch\">" .
+             "<input type=\"checkbox\" id=\"color_switch\" 
name=\"color_switch\" onClick=\"color_display(this);\"> </td><td><label 
for=\"color_switch\">" .
              BML::ml( '.colors.text', { user => $username } ) .
              "</label><br /><span style=\"font-size: 
7pt;\">($ML{'.colors.disclaimer'})</span></td></tr></table> p?>";
     $body .= "<script>\n";
_______________________________________________
dw-changelog mailing list
[email protected]
http://lists.dwscoalition.org/mailman/listinfo/dw-changelog

Reply via email to