Menu

#1 Add color hints for paper status in AveragePaperScore.php

open
nobody
None
5
2005-03-23
2005-03-23
Anonymous
No

In Chair/AveragePaperScore.php, it would be a nice
visual clue to users that a paper has been accepted or
rejected by coloring the HTML row of that paper if the
'judgement' column has been selected.
(better yet, perhaps the row can be colored without the
need for a judgement column.)

The coloring effect can be achieved by replacing the
code around line 479:

print "<tr> <td $redText> $rowNum<BR>($paperId)</td> ";

with:

$colorRowOutcome = $row['outcome'];
if($colorRowOutcome == 'accepted')
{
print "<tr bgcolor=#009900>";
}else if($colorRowOutcome=='rejected'){
print "<tr bgcolor=#CC0000>";
}else{
print "<tr>";
}
print " <td $redText> $rowNum<BR>($paperId)</td> ";

Cheers,
Michael
mlocasto AT acm.org

Discussion

  • Nobody/Anonymous

    Logged In: NO

    After a bit more poking about, I realized the above solution
    only works for the PC Chair role.

    In order to let all PC members see the decision, then line
    149 needs to be commented out and moved outside the
    enclosing if() test for the PC Chair role:
    if( $_SESSION['Me']->isChair ){
    ....
    ....
    if( IsSet($_REQUEST['GradeCol']) ){
    $judgeCol = 1;
    //$judgeData = ", Paper.outcome "; // move to below
    print " CHECKED";
    }
    print "> Judgement Column<BR>\n";
    }

    // put here so that PC members can see a colored outcome
    // this also works if you are the PC Chair only.
    $judgeData = ", Paper.outcome ";

    if( IsSet($_REQUEST['COLS']) ){
    ....

    This makes sure that judgeData is included in $query
    variable, so that $result resultSet includes the outcome and
    thus $row['outcome'] holds the string outcome result for the
    test suggested by my previous result.

     
  • Nobody/Anonymous

    Logged In: NO

    The above fix works only for PC Chairs. For PC Members to
    see a colored outcome, the code around line 149 needs to be
    adjusted.

    if( IsSet($_REQUEST['GradeCol']) ){
    $judgeCol = 1;
    //$judgeData = ", Paper.outcome "; // move to below
    print " CHECKED";
    }
    print "> Judgement Column<BR>\n";
    }

    // put here so that PC members can see a colored outcome
    $judgeData = ", Paper.outcome ";

     

Log in to post a comment.