Changeset 13865


Ignore:
Timestamp:
Apr 2, 2012, 10:49:46 PM (9 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.