Changeset 1889

Show
Ignore:
Timestamp:
04/14/08 18:28:41 (3 months ago)
Author:
bsmith
Message:

bugzid:74303 - Review Design of Threaded Comments

Files:

Legend:

Unmodified
Added
Removed
Modified
Copied
Moved

  • branches/release-34/default_templates/comment_preview.mtml

    r1851 r1889  
    3232                                        <div class="asset-meta"> 
    3333                                            <span class="byline"> 
    34                                                 <__trans phrase="By"> <span class="vcard author"><$MTCommentAuthorLink default_name="Anonymous" show_email="0"$></span><MTIfNonEmpty tag="CommentAuthorIdentity"><$MTCommentAuthorIdentity$></MTIfNonEmpty> 
    35                                                 <MTCommentParent> 
    36                                                 <__trans phrase="replied to"> <a href="#comment-<$MTCommentID$>"><$MTCommentAuthor$></a> 
    37                                                 </MTCommentParent> 
    38                                                 <__trans phrase="on"> <a href="#comment-<$MTCommentID$>"><abbr class="published" title="<$MTCommentDate format_name="iso8601"$>"><$MTCommentDate$></abbr></a> 
     34                                                <mt:IfCommentParent> 
     35                                                    <__trans phrase="[_1] replied to <a href="[_2]">comment from [_3]</a>" params="<MTIfNonEmpty tag="CommentAuthorIdentity"><$MTCommentAuthorIdentity$></MTIfNonEmpty> <span class="vcard author"><$MTCommentAuthorLink default_name="Anonymous" show_email="0"$></span>%%<mt:CommentParent>#comment-<$MTCommentID$></mt:CommentParent>%%<mt:CommentParent><$MTCommentAuthor$></mt:CommentParent>"> 
     36                                                <mt:else> 
     37                                                    <MTIfNonEmpty tag="CommentAuthorIdentity"><$MTCommentAuthorIdentity$></MTIfNonEmpty> <span class="vcard author"><$MTCommentAuthorLink default_name="Anonymous" show_email="0"$></span> 
     38                                                </mt:IfCommentParent> 
     39                                                | <a href="#comment-<$MTCommentID$>"><abbr class="published" title="<$MTCommentDate format_name="iso8601"$>"><$MTCommentDate$></abbr></a> 
    3940                                                <MTIfCommentsAccepted> | <$MTCommentReplyLink$></MTIfCommentsAccepted> 
    4041                                            </span> 

  • branches/release-34/default_templates/comments.mtml

    r1851 r1889  
    1515                    <div class="asset-meta"> 
    1616                        <span class="byline"> 
    17                             <__trans phrase="By"> <span class="vcard author"><$MTCommentAuthorLink default_name="Anonymous" show_email="0"$></span><MTIfNonEmpty tag="CommentAuthorIdentity"><$MTCommentAuthorIdentity$></MTIfNonEmpty> 
    18                             <MTCommentParent> 
    19                             <__trans phrase="replied to"> <a href="#comment-<$MTCommentID$>"><$MTCommentAuthor$></a> 
    20                             </MTCommentParent> 
    21                             <__trans phrase="on"> <a href="#comment-<$MTCommentID$>"><abbr class="published" title="<$MTCommentDate format_name="iso8601"$>"><$MTCommentDate$></abbr></a> 
     17                            <mt:IfCommentParent> 
     18                                <__trans phrase="[_1] replied to <a href="[_2]">comment from [_3]</a>" params="<MTIfNonEmpty tag="CommentAuthorIdentity"><$MTCommentAuthorIdentity$></MTIfNonEmpty> <span class="vcard author"><$MTCommentAuthorLink default_name="Anonymous" show_email="0"$></span>%%<mt:CommentParent>#comment-<$MTCommentID$></mt:CommentParent>%%<mt:CommentParent><$MTCommentAuthor$></mt:CommentParent>"> 
     19                            <mt:else> 
     20                                <MTIfNonEmpty tag="CommentAuthorIdentity"><$MTCommentAuthorIdentity$></MTIfNonEmpty> <span class="vcard author"><$MTCommentAuthorLink default_name="Anonymous" show_email="0"$></span> 
     21                            </mt:IfCommentParent> 
     22                            | <a href="#comment-<$MTCommentID$>"><abbr class="published" title="<$MTCommentDate format_name="iso8601"$>"><$MTCommentDate$></abbr></a> 
    2223                            <MTIfCommentsAccepted> | <$MTCommentReplyLink$></MTIfCommentsAccepted> 
    2324                        </span> 
    …… 
    7374                    </div> 
    7475                    <div id="comment-form-remember-me"> 
    75                         <label for="comment-bake-cookie"><input type="checkbox" id="comment-bake-cookie" name="bakecookie" onclick="if (!this.checked) forgetMe(document.comments_form)" value="1" /> 
    76                             <__trans phrase="Remember personal info?"></label> 
     76                        <input type="checkbox" id="comment-bake-cookie" name="bakecookie" onclick="if (!this.checked) forgetMe(document.comments_form)" value="1" /> 
     77                        <label for="comment-bake-cookie"><__trans phrase="Remember personal info?"></label> 
    7778                    </div> 
    7879                    <div id="comment-form-reply"> 
    79                         <label for="comment-reply" id="comment-reply-label"><input type="checkbox" id="comment-reply" name="comment_reply" value="" onclick="setCommentParentID();" /> 
    80                             <__trans phrase="Replying to"></label> 
     80                        <input type="checkbox" id="comment-reply" name="comment_reply" value="" onclick="setCommentParentID();" /> 
     81                        <label for="comment-reply" id="comment-reply-label"></label> 
    8182                    </div> 
    8283                </div> 

  • branches/release-34/default_templates/javascript.mtml

    r1887 r1889  
    175175     
    176176    var checkbox = document.getElementById('comment-reply'); 
    177  
    178     // Clear the current label but NOT our checkbox! 
    179177    var label = document.getElementById('comment-reply-label'); 
    180     for(var i = 1; i < label.childNodes.length; i++) { 
    181         label.removeChild(label.childNodes[i]); 
    182     } 
    183      
     178    var text = document.getElementById('comment-text'); 
     179 
    184180    // Populate label with new values 
    185     var reply_text = document.createTextNode(' <__trans phrase="Replying to"> '); 
    186     label.appendChild(reply_text); 
    187  
    188     var anchor = document.createElement('a'); 
    189     anchor.setAttribute('href', '#comment-' + parent_id); 
    190     anchor.innerHTML = author; 
    191     label.appendChild(anchor); 
    192      
     181    var reply_text = '<__trans phrase="Replying to <a href="#comment-[_1]">comment from [_2]</a>" params="'+ parent_id +'%%'+ author +'">'; 
     182    label.innerHTML = reply_text; 
     183 
    193184    checkbox.value = parent_id;  
    194     checkbox.setAttribute('checked', true); 
    195     checkbox.focus(); 
    196      
     185    checkbox.checked = true; 
     186    text.focus(); 
     187 
    197188    setCommentParentID(); 
    198189} 

  • branches/release-34/mt-static/themes-base/blog.css

    r1879 r1889  
    300300#comment-form-url, 
    301301#comment-form-remember-me, 
     302#comment-form-reply, 
    302303#comments-open-text, 
    303304.trackbacks-header, 
    …… 
    309310.search-results-header { 
    310311    margin-bottom: .75em; 
     312} 
     313 
     314#comment-form-remember-me label, 
     315#comment-form-reply label { 
     316    display: inline; 
    311317} 
    312318