|
|
|||||||||||
|
|
|||
|
|||
|
Thanks for pointing this out.
I think I'll be able to select the appropriate priority next time. :) |
|
This petition for a change to Unconfirmed was rejected
|
|||
|
|||
|
I can see you made a fix here, you increased the spacer after the first button - that's not what I was expecting.
I was expecting a decrease of spacer before second reply button, which will be perfect for consistency with new thread buttons. I belive there is some conditionals to check if it's the last post before going to another page. You can use this conditionals, and bypass postbit_wrapper, and use standard spacer, and it should work perfectly. |
|
This petition for a change to Confirmed was rejected
|
|||
|
|||
|
I also think that a decrease of spacer before second reply button is a best solution.
|
|
|
|||
|
|||
|
I have made a further change for 3.7. They all have a 3px margin now.
|
|
|
|||
|
|||
|
Just for my information, will 3.6.9 have the current fix, or will you leave them as is?
|
|
|
|||
|
|||
|
By the way:
Status Closed (Fixed) Fixed Version (none) |
|
This petition for a change to Confirmed was rejected
|
|||
|
|||
|
I belive this bug haven't been fixed in this issue nor in this one: http://www.vbulletin.com/forum/proje...?issueid=23570 (nothing changed then before)
I think the fix to this will be by applying this change in postbit_wrapper template.
<!-- post #$post[postid] -->
<if condition="$show['spacer']"> $spacer_open <div id="edit$post[postid]" style="padding:0px 0px <if condition="!$post['islastshown']">$stylevar[cellpadding]px<else />3px</if> 0px"> </if> $postbit <if condition="$show['spacer']"> </div> $spacer_close </if> <!-- / post #$post[postid] --> |
|
|
|||
|
|||
|
Can someone post which one's are the ones i have to change so that i can fix it manually?
|
|
|
|||
|
|||
|
I think mine is correct though you should wait a developer confirming that.
|
|
|
|||
|
|||
|
As noted, this bug is fixed for 3.7. 3.6.9 only has a partial fix.
|
|
|
|||
|
|||
|
I don't see a fix for it here on these forums.
|
|
|
|||
|
|||
|
For the spaces between 'Reply' button and postbit, this is fixed. Look at the code:
Code:
<table cellpadding="0" cellspacing="0" border="0" width="100%" style="margin-bottom:3px">
<tr valign="top">
<td class="smallfont"><a href="newreply.php?do=newreply&noquote=1&p=xxx" rel="nofollow"><img src="images/buttons/reply.gif" alt="Reply" border="0" /></a></td>
</tr>
</table>
// postbits....
<table cellpadding="0" cellspacing="0" border="0" width="100%" style="margin-top:-3px">
<tr valign="top">
<td class="smallfont"><a href="newreply.php?do=newreply&noquote=1&p=xxx" rel="nofollow"><img src="images/buttons/reply.gif" alt="Reply" border="0" /></a></td>
</tr>
</table>
For Project Tools, it have a margin of 6px instead of -3px, and this is not the place to discuss about PT. |
|
This petition for a change to Confirmed was rejected
|
|||
|
|||
|
I belive the fix was implemented here for a day or two then went back to the old look.
|
|
|
||
|
||
|
Mazinger, I cannot confirm that this bug is in existance on vBulletin. The only place where the spacing for the "Post Reply" button is inconsistent is here, in Project tools. (and as such, a bug should be reported against project tools)
Other than that, It seems that everything is in order. If you are still having issues, can you post screenshots etc to help us determine what the exact issue is? |
|
|
|||
|
|||
|
Simply, the spacer before the second reply button (showthread) was reduced by the release candidate 1 or two, then the fix was cancelled (as it went back to classic one)
|
|