Changeset 13865


Ignore:
Timestamp:
04/02/12 22:49:46 (8 years ago)
Author:
rvelices
Message:
  • comment edit form looks ok now with the new comment layout
  • removed unused css / simplify it
  • simplified jquery drop boxes ...
Location:
trunk
Files:
8 edited

Legend:

Unmodified
Added
Removed
  • trunk/comments.php

    r12930 r13865  
    207207if (isset($action)) 
    208208{ 
    209   check_pwg_token(); 
    210  
    211209  $comment_author_id = get_comment_author_id($comment_id); 
    212210 
     
    217215    if ('delete' == $action) 
    218216    { 
     217      check_pwg_token(); 
    219218      delete_user_comment($comment_id); 
    220219      $perform_redirect = true; 
     
    223222    if ('validate' == $action) 
    224223    { 
     224      check_pwg_token(); 
    225225      validate_user_comment($comment_id); 
    226226      $perform_redirect = true; 
     
    231231      if (!empty($_POST['content'])) 
    232232      { 
     233        check_pwg_token(); 
    233234        update_user_comment( 
    234235          array( 
     
    503504        $url, 
    504505        array( 
    505           'edit' => $comment['comment_id'], 
    506           'pwg_token' => get_pwg_token(), 
     506          'edit' => $comment['comment_id'] 
    507507          ) 
    508508        ); 
     
    515515        $tpl_comment['IMAGE_ID'] = $comment['image_id']; 
    516516        $tpl_comment['CONTENT'] = $comment['content']; 
     517        $tpl_comment['PWG_TOKEN'] = get_pwg_token(); 
    517518      } 
    518519    } 
  • trunk/include/picture_comment.inc.php

    r13773 r13865  
    199199            'action'=>'edit_comment', 
    200200            'comment_to_edit'=>$row['id'], 
    201             'pwg_token' => get_pwg_token(), 
    202201            ) 
    203202          ); 
     
    208207            $tpl_comment['KEY'] = $key; 
    209208            $tpl_comment['CONTENT'] = $row['content']; 
     209            $tpl_comment['PWG_TOKEN'] = get_pwg_token(); 
    210210          } 
    211211      } 
  • trunk/picture.php

    r13736 r13865  
    333333      redirect($url_self); 
    334334    } 
    335     case 'edit_comment' : 
    336     { 
    337       check_pwg_token(); 
     335    case 'edit_comment': 
     336    { 
    338337      include_once(PHPWG_ROOT_PATH.'include/functions_comment.inc.php'); 
    339338      check_input_parameter('comment_to_edit', $_GET, false, PATTERN_ID); 
     
    344343        if (!empty($_POST['content'])) 
    345344        { 
     345          check_pwg_token(); 
    346346          $comment_action = update_user_comment( 
    347347            array( 
     
    375375          } 
    376376          unset($_POST['content']); 
    377           break; 
    378377        } 
    379378        else 
    380379        { 
    381380          $edit_comment = $_GET['comment_to_edit']; 
    382           break; 
    383381        } 
    384382      } 
     383      break; 
    385384    } 
    386385    case 'delete_comment' : 
  • trunk/themes/default/fix-ie5-ie6.css

    r13774 r13865  
    5151 
    5252.filter LABEL INPUT, 
    53 .filter LABEL SELECT, 
    54 .filter LABEL SPAN, 
    55 .filter LABEL TEXTAREA { 
     53.filter LABEL SELECT { 
    5654  display: block; 
    5755  margin: 0.5em 0; 
    5856} 
    5957.filter FIELDSET * LABEL INPUT, 
    60 .filter FIELDSET * LABEL SELECT, 
    61 .filter FIELDSET * LABEL TEXTAREA, 
    62 .filter LABEL SPAN INPUT, 
    63 .filter LABEL SPAN SELECT, 
    64 .filter LABEL SPAN TEXTAREA { 
     58.filter FIELDSET * LABEL SELECT { 
    6559        display: inline; 
    6660        vertical-align: top; 
  • trunk/themes/default/template/comment_list.tpl

    r13801 r13865  
    4444                {if isset($comment.IN_EDIT)} 
    4545                <a name="edit_comment"></a> 
    46                 <form  method="post" action="{$comment.U_EDIT}" class="filter" id="editComment"> 
    47                         <fieldset> 
    48                                 <legend>{'Edit a comment'|@translate}</legend> 
    49                                 <label>{'Comment'|@translate}<textarea name="content" id="contenteditid" rows="5" cols="80">{$comment.CONTENT|@escape}</textarea></label> 
    50                                 <input type="hidden" name="key" value="{$comment.KEY}"> 
     46                <form method="post" action="{$comment.U_EDIT}" id="editComment"> 
     47                        <p><label>{'Edit a comment'|@translate} :</label></p> 
     48                        <p><textarea name="content" id="contenteditid" rows="5" cols="80">{$comment.CONTENT|@escape}</textarea></p> 
     49                        <p><input type="hidden" name="key" value="{$comment.KEY}"> 
     50                                <input type="hidden" name="pwg_token" value="{$comment.PWG_TOKEN}"> 
    5151                                <input type="hidden" name="image_id" value="{$comment.IMAGE_ID|@default:$current.id}"> 
    5252                                <input type="submit" value="{'Submit'|@translate}"> 
    53                         </fieldset> 
     53                        </p> 
    5454                </form> 
    5555                {else} 
  • trunk/themes/default/template/index.tpl

    r13844 r13865  
    88                        <span class="pwg-icon pwg-icon-sort">&nbsp;</span><span class="pwg-button-text">{'Sort order'|@translate}</span> 
    99                </a> 
    10                 <div id="sortOrderBox" class="switchBox" style="display:none"> 
     10                <div id="sortOrderBox" class="switchBox"> 
    1111                        <div class="switchBoxTitle">{'Sort order'|@translate}</div> 
    1212                        {foreach from=$image_orders item=image_order name=loop}{if !$smarty.foreach.loop.first}<br>{/if} 
     
    1919                </div> 
    2020                {footer_script}{literal} 
    21 $(document).ready(function() { 
    22   $("#sortOrderBox").css({'top':0,'left':0}); 
    23   var sortOrderBox_width = $("#sortOrderBox").outerWidth(true); 
    24   var sortOrderBox_height = $("#sortOrderBox").outerHeight(true); 
    25    
    26   $("#sortOrderLink").click(function() { 
    27     $("#sortOrderBox").toggle(); 
    28      
    29     if ($(this).offset().left + sortOrderBox_width > $(window).width()) { 
    30       $("#sortOrderBox").css("left", $(window).width() - sortOrderBox_width - 5); 
    31     } else { 
    32       $("#sortOrderBox").css("left", $(this).offset().left); 
    33     } 
    34     $("#sortOrderBox").css("top", $(this).offset().top + $(this).outerHeight(true)); 
    35   }); 
    36    
    37   $("#sortOrderBox").bind("mouseleave", function() { 
    38     $(this).hide(); 
    39   }); 
     21$("#sortOrderLink").click(function() { 
     22        var elt = $("#sortOrderBox"); 
     23        elt.css("left", Math.min( $(this).offset().left, $(window).width() - elt.outerWidth(true) - 5)) 
     24                .css("top", $(this).offset().top + $(this).outerHeight(true)) 
     25                .toggle(); 
     26}); 
     27$("#sortOrderBox").on("mouseleave", function() { 
     28        $(this).hide(); 
    4029}); 
    4130                {/literal}{/footer_script} 
     
    4736                        <span class="pwg-icon pwg-icon-sizes">&nbsp;</span><span class="pwg-button-text">{'Photo sizes'|@translate}</span> 
    4837                </a> 
    49                 <div id="derivativeSwitchBox" class="switchBox" style="display:none"> 
     38                <div id="derivativeSwitchBox" class="switchBox"> 
    5039                        <div class="switchBoxTitle">{'Photo sizes'|@translate}</div> 
    5140                        {foreach from=$image_derivatives item=image_derivative name=loop}{if !$smarty.foreach.loop.first}<br>{/if} 
     
    5847                </div> 
    5948                {footer_script}{literal} 
    60 $(document).ready(function() { 
    61   $("#derivativeSwitchBox").css({'top':0,'left':0}); 
    62   var derivativeSwitchBox_width = $("#derivativeSwitchBox").outerWidth(true); 
    63   var derivativeSwitchBox_height = $("#derivativeSwitchBox").outerHeight(true); 
    64    
    65   $("#derivativeSwitchLink").click(function() { 
    66     $("#derivativeSwitchBox").toggle(); 
    67      
    68     if ($(this).offset().left + derivativeSwitchBox_width > $(window).width()) { 
    69       $("#derivativeSwitchBox").css("left", $(window).width() - derivativeSwitchBox_width - 5); 
    70     } else { 
    71       $("#derivativeSwitchBox").css("left", $(this).offset().left); 
    72     } 
    73     $("#derivativeSwitchBox").css("top", $(this).offset().top + $(this).outerHeight(true)); 
    74   }); 
    75    
    76   $("#derivativeSwitchBox").bind("mouseleave", function() { 
    77     $(this).hide(); 
    78   }); 
     49$("#derivativeSwitchLink").click(function() { 
     50        var elt = $("#derivativeSwitchBox"); 
     51        elt.css("left", Math.min( $(this).offset().left, $(window).width() - elt.outerWidth(true) - 5)) 
     52                .css("top", $(this).offset().top + $(this).outerHeight(true)) 
     53                .toggle(); 
     54}); 
     55$("#derivativeSwitchBox").on("mouseleave", function() { 
     56        $(this).hide(); 
    7957}); 
    8058                {/literal}{/footer_script} 
  • trunk/themes/default/template/picture.tpl

    r13844 r13865  
    3636} 
    3737 
    38 $(document).ready(function() { 
    39   $("#derivativeSwitchBox").css({'top':0,'left':0}); 
    40   var derivativeSwitchBox_width = $("#derivativeSwitchBox").outerWidth(true); 
    41   var derivativeSwitchBox_height = $("#derivativeSwitchBox").outerHeight(true); 
    42    
    43   $("#derivativeSwitchLink").click(function() { 
    44     $("#derivativeSwitchBox").toggle(); 
    45      
    46     if ($(this).offset().left + derivativeSwitchBox_width > $(window).width()) { 
    47       $("#derivativeSwitchBox").css("left", $(window).width() - derivativeSwitchBox_width - 5); 
    48     } else { 
    49       $("#derivativeSwitchBox").css("left", $(this).offset().left); 
    50     } 
    51     $("#derivativeSwitchBox").css("top", $(this).offset().top + $(this).outerHeight(true)); 
    52   }); 
    53    
    54   $("#derivativeSwitchBox").bind("mouseleave click", function() { 
    55     $(this).hide(); 
    56   }); 
     38$("#derivativeSwitchLink").click(function() { 
     39        var elt = $("#derivativeSwitchBox"); 
     40        elt.css("left", Math.min( $(this).offset().left, $(window).width() - elt.outerWidth(true) - 5)) 
     41                .css("top", $(this).offset().top + $(this).outerHeight(true)) 
     42                .toggle(); 
     43}); 
     44$("#derivativeSwitchBox").on("mouseleave click", function() { 
     45        $(this).hide(); 
    5746}); 
    5847{/literal}{/footer_script} 
     
    6049  <span class="pwg-icon pwg-icon-sizes">&nbsp;</span><span class="pwg-button-text">{'Photo sizes'|@translate}</span> 
    6150</a> 
    62 <div id="derivativeSwitchBox" class="switchBox" style="display:none"> 
     51<div id="derivativeSwitchBox" class="switchBox"> 
    6352  <div class="switchBoxTitle">{'Photo sizes'|@translate}</div> 
    6453  {foreach from=$current.unique_derivatives item=derivative key=derivative_type} 
     
    210199        {/if} 
    211200        {if $display_info.categories and isset($related_categories)} 
    212         <div id="Categories"  class="imageInfo"> 
     201        <div id="Categories" class="imageInfo"> 
    213202                <dt>{'Albums'|@translate}</dt> 
    214203                <dd> 
     
    306295                elt.style.left = (ePos.offsetLeft)+"px"; 
    307296                elt.style.top = (ePos.offsetTop+ePos.offsetHeight)+"px"; 
    308                 elt.style.display=""; 
     297                elt.style.display="block"; 
    309298        } 
    310299        else 
     
    351340                        <div id="commentAdd"> 
    352341                                <h4>{'Add a comment'|@translate}</h4> 
    353                                 <form  method="post" action="{$comment_add.F_ACTION}" id="addComment" > 
     342                                <form method="post" action="{$comment_add.F_ACTION}" id="addComment" > 
    354343                                        {if $comment_add.SHOW_AUTHOR} 
    355                                                 <p><label>{'Author'|@translate}&nbsp;:</label></p> 
     344                                                <p><label>{'Author'|@translate} :</label></p> 
    356345                                                <p><input type="text" name="author" /></p> 
    357                                                 <p><label>{'Comment'|@translate}&nbsp;:</label></p> 
     346                                                <p><label>{'Comment'|@translate} :</label></p> 
    358347                                        {/if} 
    359348                                        <p><textarea name="content" id="contentid" rows="5" cols="50">{$comment_add.CONTENT}</textarea></p> 
    360349                                        <p><input type="hidden" name="key" value="{$comment_add.KEY}" /> 
    361                                                 <input class="submit" type="submit" value="{'Submit'|@translate}"></p> 
     350                                                <input type="submit" value="{'Submit'|@translate}"></p> 
    362351                                </form> 
    363352                        </div> 
  • trunk/themes/default/theme.css

    r13801 r13865  
    271271 
    272272.commentElement { 
     273        border-radius: 5px; 
    273274        margin: 5px; 
    274275        padding: 2px 0 0 2px; 
     
    367368        border-radius: 4px; 
    368369        z-index: 100; 
    369     text-align:left; 
    370         position: absolute; /*left, right set through js*/ 
     370        text-align:left; 
     371        display: none; 
     372        position: absolute; 
     373        left: 0; top: 0; /*left, right set through js*/ 
    371374} 
    372375 
     
    529532} 
    530533 
    531 .filter UL { 
    532   margin: 0; 
    533 } 
    534534.filter UL, 
    535535.filter LABEL { 
     
    558558/*IE6 needs override because of > selector*/ 
    559559.filter LABEL>INPUT, 
    560 .filter LABEL>SELECT, 
    561 .filter LABEL>SPAN, 
    562 .filter LABEL>TEXTAREA { 
     560.filter LABEL>SELECT { 
    563561        display: block; 
    564562        margin: 0.5em 0; 
     
    566564 
    567565.filter FIELDSET * LABEL>INPUT, 
    568 .filter FIELDSET * LABEL>SELECT, 
    569 .filter FIELDSET * LABEL>TEXTAREA, 
    570 .filter SPAN>INPUT, 
    571 .filter SPAN>SELECT, 
    572 .filter SPAN>TEXTAREA { 
     566.filter FIELDSET * LABEL>SELECT { 
    573567        display: inline; 
    574568        vertical-align: top; 
     
    576570} 
    577571 
    578  
    579  
    580  
    581 .filter INPUT[type="submit"] { 
    582         margin-top: 1em; 
    583 } 
    584572 
    585573.properties UL { 
     
    805793 
    806794#commentAdd input[type="text"], 
    807 #commentAdd textarea { 
    808         max-width: 100%; 
    809         width: 100%; 
    810 } 
    811  
    812 #pictureCommentList .commentElement { 
    813         border-radius: 5px; 
     795#comments TEXTAREA { 
     796        max-width: 99%; 
     797        width: 99%; 
    814798} 
    815799 
     
    825809#pictureCommentList .navigationBar      { float: right; margin-bottom: 5px;} 
    826810#pictureComments h4                                     { margin: 0;} 
    827 #pictureComments form p                         { margin: 5px 0; text-align: left;} 
    828 #pictureComments form p textarea                { margin: 10px 0;} 
     811#comments FORM P         { 
     812        margin: 5px 0; text-align: left; 
     813} 
     814 
    829815#pictureComments INPUT[type=submit]     { margin: 0;} 
    830816 
Note: See TracChangeset for help on using the changeset viewer.