<![CDATA[ Quip Validation Fix - MODX Community Forums]]> https://forums.modx.com/thread/?thread=25306 <![CDATA[Re: Quip Validation Fix]]> https://forums.modx.com/thread/25306/quip-validation-fix#dis-post-129891 Quote from: splittingred at Aug 03, 2010, 07:52 PM

Quote from: AMDbuilder at Aug 03, 2010, 07:40 PM

I guess I should fill out the CLI to contribute directly?

Yep. laugh

Resolved, I quit being lazy smiley]]>
AMDbuilder Aug 03, 2010, 03:38 PM https://forums.modx.com/thread/25306/quip-validation-fix#dis-post-129891
<![CDATA[Re: Quip Validation Fix]]> https://forums.modx.com/thread/25306/quip-validation-fix#dis-post-129890 Quote from: AMDbuilder at Aug 03, 2010, 07:40 PM

I guess I should fill out the CLI to contribute directly?

Yep. laugh]]>
splittingred Aug 03, 2010, 02:52 PM https://forums.modx.com/thread/25306/quip-validation-fix#dis-post-129890
<![CDATA[Quip Validation Fix]]> https://forums.modx.com/thread/25306/quip-validation-fix#dis-post-129889 http://svn.modxcms.com/docs/display/ADDON/Quip.QuipReply.tplAddComment

The issue is with the labels not lining up. The corrected code is as follows:


[[+preview]]
<span class="quip-success" id="quip-success-[[+idprefix]]">[[+successMsg]]</span>

<form id="quip-add-comment-[[+idprefix]]" action="[[+url]]#quip-comment-preview-box-[[+idprefix]]" method="post">


<div class="quip-comment quip-add-comment">
<input type="hidden" name="nospam" value="" />
<input type="hidden" name="thread" value="[[+thread]]" />
<input type="hidden" name="parent" value="[[+parent]]" />
    <span class="quip-error">[[+error]]</span>
    
     <div class="quip-fld">
        <label for="quip-comment-name-[[+idprefix]]">[[%quip.name? &namespace=`quip` &topic=`default`]]:</label>
        <input type="text" name="name" id="quip-comment-name-[[+idprefix]]" value="[[+name]]" />
        <br />
    </div>
    
    <div class="quip-fld">
        <label for="quip-comment-email-[[+idprefix]]">[[%quip.email]]: </label>
        <input type="text" name="email" id="quip-comment-email-[[+idprefix]]" value="[[+email]]" />
        <br />
    </div>
    
    <div class="quip-fld">
        <label for="quip-comment-website-[[+idprefix]]">[[%quip.website]]: </label>
        <input type="text" name="website" id="quip-comment-website-[[+idprefix]]" value="[[+website]]" />
        <br />
    </div>

    <div class="quip-fld">
        <label for="quip-comment-notify-[[+idprefix]]">[[%quip.notify_me]]: </label>
        <input type="checkbox" value="1" name="notify" id="quip-comment-notify-[[+idprefix]]" [[+notify:if=`[[+notify]]`:eq=`1`:then=`checked="checked"`]] />
        <br />
    </div>

    <div class="quip-fld recaptcha">
    [[+quip.recaptcha_html]]
    </div>
    
    
    <p><span class="quip-allowed-tags">[[%quip.allowed_tags? &tags=`[[++quip.allowed_tags:htmlent]]`]]</span>[[%quip.comment_add_new]] </p>
    <textarea name="comment" id="quip-comment-box-[[+idprefix]]" rows="5" cols="40">[[+comment]]</textarea>
    
    <button type="submit" name="quip-preview" value="1">[[%quip.preview]]</button>
    
    <br class="clear" />
</div>
</form>




The short version is adding this to the end of the current labels: "-[[+idprefix]]" (without quotes)

I guess I should fill out the CLI to contribute directly?]]>
AMDbuilder Aug 03, 2010, 02:40 PM https://forums.modx.com/thread/25306/quip-validation-fix#dis-post-129889