Added 'select' all to wishlist and moved 'select all' down on notifies

For consistency

3 files changed, 7 insertions, 1 deletions

diff --git a/ChangeLog b/ChangeLog
index 20e11ace..b67a5917 100644
--- a/ChangeLog
+++ b/ChangeLog
@@ -5,6 +5,7 @@
 	- Added spoiler warning to character revision pages
 	- Generate dbedit/dbdel notifications on character edits
 	- CSS: Hide links in [spoiler] tags
+	- Added 'select' all to wishlist and moved 'select all' down on notifies
 	- Bugfix: don't accidentally remove char traits when editing
 	- Bugfix: fixed possible SQL table name clash on history browser
 	- Bugfix: properly announce chars and traits in Multi::IRC
diff --git a/lib/VNDB/Handler/ULists.pm b/lib/VNDB/Handler/ULists.pm
index 363c2262..1c82d982 100644
--- a/lib/VNDB/Handler/ULists.pm
+++ b/lib/VNDB/Handler/ULists.pm
@@ -239,6 +239,7 @@ sub wishlist {
       { post => 'sel', required => 0, default => 0, multi => 1, template => 'int' },
       { post => 'batchedit', required => 1, enum => [ -1, @{$self->{wishlist_status}} ] },
     );
+    $frm->{sel} = [ grep $_, @{$frm->{sel}} ]; # weed out "select all" checkbox
     if(!$frm->{_err} && @{$frm->{sel}} && $frm->{sel}[0]) {
       $self->dbWishListDel($uid, $frm->{sel}) if $frm->{batchedit} == -1;
       $self->dbWishListAdd($frm->{sel}, $uid, $frm->{batchedit}) if $frm->{batchedit} >= 0;
@@ -303,6 +304,8 @@ sub wishlist {
     $own ? (footer => sub {
       Tr;
        td colspan => 3;
+        input type => 'checkbox', class => 'checkall', name => 'sel', value => 0;
+        txt ' ';
         Select name => 'batchedit', id => 'batchedit';
          option mt '_wishlist_select';
          optgroup label => mt '_wishlist_changeprio';
diff --git a/lib/VNDB/Handler/Users.pm b/lib/VNDB/Handler/Users.pm
index 75ddb547..6cedc734 100644
--- a/lib/VNDB/Handler/Users.pm
+++ b/lib/VNDB/Handler/Users.pm
@@ -611,7 +611,7 @@ sub notifies {
       class    => 'notifies',
       pageurl  => "/u$uid/notifies?r=$f->{r}",
       header   => [
-        [ '<input type="checkbox" class="checkall" name="notifysel" value="0" />' ],
+        [ '' ],
         [ mt '_usern_col_type' ],
         [ mt '_usern_col_age' ],
         [ mt '_usern_col_id' ],
@@ -639,6 +639,8 @@ sub notifies {
       footer => sub {
         Tr;
          td colspan => 5;
+          input type => 'checkbox', class => 'checkall', name => 'notifysel', value => 0;
+          txt ' ';
           input type => 'submit', name => 'markread', value => mt '_usern_but_markread';
           input type => 'submit', name => 'remove', value => mt '_usern_but_remove';
           b class => 'grayedout', ' '.mt '_usern_autodel';