Changeset 605ef7 in indico


Ignore:
Timestamp:
10/04/10 17:17:51 (3 years ago)
Author:
Jose Benito <jose.benito.gonzalez@…>
Branches:
master, burotel, hello-world-walkthrough, ipv6, v0.98-series, v0.98.2, v0.98.3, v0.98b1, v0.98b2, v0.99, 051b2622c51afb171a1dedb46a0df4fbb0cbd02e, d9941f8582b36b24821a11ea5ba16fda6a457fb1
Children:
1d3b3c
Parents:
a4c859
git-author:
csvetelina.angelova@…> (01/08/10 16:16:40)
git-committer:
Jose Benito <jose.benito.gonzalez@…> (10/04/10 17:17:51)
Message:

[FIXES] - task #108 + small layout fixes

  • Task #108 - Deleting reviewer - the loops within the services now is working properly.
  • If the final judgement is given by the referee or if the author did not submit any materials yet - in the List of Contributions now the link for each contribution is disabled and there is a small popup saying the reason why the editor or the reviewer can't access the contribution.
  • If the user is a referee or reviewer or editor, now he can't access the Contributions list for the whole conference. Now he can see only the contributions assigned to him.
  • small fix in the Competences page, in the Assign Contributions page, in the Judge Layout page, in the Judge Content page
Location:
indico/MaKaC/webinterface/tpls
Files:
2 edited

Legend:

Unmodified
Added
Removed

  • indico/MaKaC/webinterface/tpls/ConfReviewingAssignContributions.tpl

    r9b3d51 r605ef7  
    77<% if not ConfReview.hasReviewing(): %> 
    88<table align="center"><tr><td><%= _("Type of reviewing has not been chosen yet")%></td></tr></table> 
    9 <% end %> 
    10 <% elif not ConfReview.getJudgedContributions(user):%> 
    11 <p style="padding-left: 25px;"><font color="gray"><%= _("There are no contributions to assign as a Referee.")%></font></p> 
    129<% end %> 
    1310<% else: %> 
    … …  
    1613<% end %> 
    1714<%else:%> 
    18 <div style="padding-top:10px; padding-bottom: 10px;"><em><%= _("Please, select one or more contributions to assign Reviewers")%></em></div> 
    19  
    20 <div id="showHideFilteringHelp" style="padding-top: 10px;"><div id="showHideFiltering" style="display:inline"></div></div> 
     15<div style="padding-top:10px; padding-bottom: 10px;padding-left: 10px"><em><%= _("Please, select one or more contributions to assign Reviewers")%></em></div> 
     16 
     17<div id="showHideFilteringHelp" style="padding-top: 10px;padding-left: 40px"><div id="showHideFiltering" style="display:inline"></div></div> 
    2118<br/> 
    2219<table id="filteringTable" class="Revtab" width="90%%" align="center"> 
    … …  
    109106</table> 
    110107 
    111 <table> 
     108<table style="padding-left: 40px"> 
    112109    <% if not IsOnlyReferee and not (ConfReview.getChoice() == 3 or ConfReview.getChoice() == 1): %> 
    113110    <tr> 
    … …  
    146143</div> 
    147144 
    148 <table class="Revtab" width="95%%" cellspacing="0" align="center" border="0" style="padding-left:2px; margin-bottom:1em"> 
     145<table style="padding-left:40px; margin-bottom:1em;"> 
     146        <tr> 
     147            <td style="padding-bottom: 20px; padding-top: 20px"> 
     148                <%= _("Select:") %> 
     149            </td> 
     150            <td nowrap class="titleCellFormat" style="padding-bottom: 20px;  padding-top: 20px; border-right:5px solid #FFFFFF;border-left:5px solid #FFFFFF"> 
     151                 <span onclick="selectAll('selectedContributions')" align="left" style="cursor:pointer;padding-bottom:5px;color:#0B63A5;list-style-type:none;" onmouseover="this.style.color='#E25300'" onmouseout="this.style.color='#0B63A5'"> 
     152                        <%= _("All")%> 
     153                 </span>, 
     154                 <span onclick="deselectAll('selectedContributions')" align="left" style="cursor:pointer;padding-bottom:5px;color:#0B63A5;list-style-type:none;" onmouseover="this.style.color='#E25300'" onmouseout="this.style.color='#0B63A5'"> 
     155                        <%= _("None")%> 
     156                 </span> 
     157            </td> 
     158        </tr> 
     159</table> 
     160 
     161   <table class="Revtab" width="95%%" cellspacing="0" align="center" border="0" style="padding-left:20px; margin-bottom:1em"> 
    149162<!-- 
    150163    <tr> 
    … …  
    154167    <thead> 
    155168        <tr> 
    156             <td nowrap class="titleCellFormat" style="border-right:5px solid #FFFFFF;border-left:5px solid #FFFFFF"> 
    157                 <ul style="padding-left: 0px; padding-right: 20px;"> 
    158                     <li onclick="selectAll('selectedContributions')" align="left" style="padding-bottom:5px;color:#0B63A5;list-style-type:none;" onmouseover="this.style.color='#E25300'" onmouseout="this.style.color='#0B63A5'"> 
    159                         <span style="cursor:pointer"><%= _("Select All")%></span> 
    160                     </li> 
    161                     <li onclick="deselectAll('selectedContributions')" style="color:#0B63A5;list-style-type:none;" onmouseover="this.style.color='#E25300'" onmouseout="this.style.color='#0B63A5'"> 
    162                         <span style="cursor:pointer"><%= _("Deselect All")%></span> 
    163                     </li> 
    164                 </ul> 
    165             </td> 
     169             
    166170            <td nowrap class="titleCellFormat" style="border-right:5px solid #FFFFFF;border-left:5px solid #FFFFFF;"> 
    167171                <%= _("Id")%> 
    … …  
    241245            </td> 
    242246            --> 
    243              
    244247            <td> 
    245248                <ul> 
    … …  
    278281</table> 
    279282 
    280 <table> 
     283<table style="padding-left: 40px"> 
    281284    <% if not IsOnlyReferee and not (ConfReview.getChoice() == 3 or ConfReview.getChoice() == 1): %> 
    282285    <tr> 
    … …  
    716719 * @param {Object} role 
    717720 */ 
    718 var removeReviewersAlerts = function(contributions, order, role) { 
     721var removeReviewersAlerts = function(contributions, role) { 
    719722    contributionsWithoutReviewers = [] 
    720723    for (i in contributions) { 
    … …  
    738741        popup.draw = function(){ 
    739742         
    740             var span1 = Html.span({}, $T("Some of the contributions you checked do not have a Reviewer.")); 
    741             var span2 = Html.span({}, $T("Do you want to remove a reviewer only from the contributions that have one?")); 
    742             var yesButton = Html.button('popUpButton', $T("Yes")); 
    743             yesButton.observeClick(function(){ 
     743            var span1 = Html.span({}, $T("The Content Reviewers will be removed only from the contributions that have ones.")); 
     744            var okButton = Html.button('popUpButton', $T("OK")); 
     745            okButton.observeClick(function(){ 
    744746                deselectWithoutReviewer(contributions); 
    745                 fetchUsers(order, role); 
     747                removeUser('allReviewers'); 
    746748                popup.close(); 
    747749            }); 
    748                          
    749              var noButton = Html.button('popUpButton', $T("No")); 
    750              noButton.observeClick(function(){ 
    751                 popup.close(); 
    752             });  
    753               var buttons = Widget.inline([yesButton, noButton]) 
    754               var all = Widget.lines([span1, span2, buttons]) 
     750             
     751              var all = Widget.lines([span1, okButton]) 
    755752              return this.ExclusivePopup.prototype.draw.call(this, Html.div({style: {height: '130px', width: '420px'}},[all]));   
    756753                }; 
    … …  
    10051002    
    10061003   if (order == 'remove' && role == 'reviewer')  { 
    1007         if (!removeReviewersAlerts(checkedContributions, order, role)) { 
     1004        if (!removeReviewersAlerts(checkedContributions, role)) { 
    10081005            return; 
    10091006        }  
    … …  
    10311028                 
    10321029                popup.draw = function(){ 
    1033                             var users = $L();  
    1034                             var userTemplate = function(user) { 
    1035                                 var li = Html.li(); 
    1036                                 var userName = Widget.link(command(function(){ 
    1037                                     userSelected(user); 
    1038                                     var killProgress = IndicoUI.Dialogs.Util.progress() 
    1039                                     popup.close(); 
    1040                                     killProgress(); 
    1041                                 }, user.name)); 
    1042                              
    1043                              
    1044                             var userCompetences = Html.span({style:{marginLeft:'5px'}}, 
     1030                        var users = $L();  
     1031                        var userTemplate = function(user) { 
     1032                            var li = Html.li(); 
     1033                            var userName = Widget.link(command(function(){ 
     1034                                userSelected(user); 
     1035                                var killProgress = IndicoUI.Dialogs.Util.progress() 
     1036                                popup.close(); 
     1037                                killProgress(); 
     1038                            }, user.name)); 
     1039                         
     1040                         
     1041                        var userCompetences = Html.span({style:{marginLeft:'5px'}}, 
    10451042                            user.competences.length == 0 ? $T('(no competences defined)') : $T('(competences: ') + user.competences.join(', ') + ')' 
    10461043                        ); 
    … …  
    10491046                        return li; 
    10501047                    }     
    1051                          
     1048                     
    10521049                        var userList = Html.ul(); 
    10531050                        bind.element(userList, users, userTemplate); 
    … …  
    11511148            function(result,error) { 
    11521149                if (!error) { 
     1150                    if (!removeReviewersAlerts(checkedContributions, role)) { 
     1151                                return; 
     1152                        } 
    11531153                    for (i in checkedContributions) { 
    11541154                        contributionId = checkedContributions[i]; 
    11551155                        contribution = getContribution(contributionId); 
    1156                         if (!removeReviewersAlerts(checkedContributions, role)) { 
    1157                                 return; 
    1158                         }  
    11591156                        contribution.reviewManager.reviewersList = []; 
    11601157                        updateContribution(contributionId); 

  • indico/MaKaC/webinterface/tpls/ContributionReviewingHistory.tpl

    ra4c859 r605ef7  
    1212<<<<<<< HEAD:indico/MaKaC/webinterface/tpls/ContributionReviewingHistory.tpl 
    1313<<<<<<< HEAD:indico/MaKaC/webinterface/tpls/ContributionReviewingHistory.tpl 
     14<<<<<<< HEAD:indico/MaKaC/webinterface/tpls/ContributionReviewingHistory.tpl 
    1415                        <td colspan="3" class="groupTitle" style="width: 60%;padding-top:20px; border-bottom: none"> 
     16======= 
     17                        <td colspan="3" class="groupTitle" style="padding-top:20px; border-bottom: none; font-size: 20px;"> 
     18>>>>>>> 4608831... [FIXES] - task #108 + small layout fixes:indico/MaKaC/webinterface/tpls/ContributionReviewingHistory.tpl 
    1519======= 
    1620                        <td colspan="3" class="groupTitle" style="padding-top:20px; border-bottom: none; font-size: 20px;"> 
Note: See TracChangeset for help on using the changeset viewer.