Posts: 535
Threads: 31
Joined: Jul 2010
Works for me. Ignore that for now.
I'm giving you three guesses...
Posts: 2,894
Threads: 59
Joined: Jun 2010
The feature seems to work fine but I don't see why you need the ID text field? I think it would be better to only have the message saying who the reply is to and the link to undo the reply. It's always best to make things as simple as possible.
Also the reply comment box forces a horizontal scroll bar so you have to scroll to the right to see the rating and reply link.
It would be nice to have some height spacing between the comment and the reply box. This should be in the stylesheet. The CSS for this should be independent from the normal comment box height spacing.
Apart from those things, it actually looks really good!
Have
you completed
the interview?
Posts: 535
Threads: 31
Joined: Jul 2010
I told you that I need to change that field to "hidden". You didn't read the whole post did you?
The styling is also temporary.
Thank you for the complement!
I need to check out what will happen if you disable the form. I don't think it gets rid of the [reply] link.
Preview fix:
in "includes/functions/comments.php"
edit line 157[?] from
Code:
if($enabled_reply && ){
to
Code:
if($enabled_reply && !$is_preview){
I'm giving you three guesses...
Posts: 2,894
Threads: 59
Joined: Jun 2010
I'd put the $default_reply_name in a language file instead of the database.
I'm not sure yet what $default_reply does?
Have
you completed
the interview?
Posts: 2,894
Threads: 59
Joined: Jun 2010
I did read your whole post but it didn't make a lot of sense at the time as I hadn't seen the feature in action.
Have
you completed
the interview?
Posts: 535
Threads: 31
Joined: Jul 2010
Those two are for the "repopulate() " function. I don't know of any better way to re-enter the value into the fields. I can make them all fit into one db row if you want.
I'm giving you three guesses...
Posts: 2,894
Threads: 59
Joined: Jun 2010
It doesn't make sense for the undo reply link to show when it's not a reply.
Have
you completed
the interview?
Posts: 2,894
Threads: 59
Joined: Jun 2010
Do you think the reply link should be removed from the reply boxes so it's really clear that you can't reply to a reply?
Have
you completed
the interview?
Posts: 535
Threads: 31
Joined: Jul 2010
Undo reply link fix:
"includes/functions/comments.php" line 156
PHP Code:
<?php
//Reply link
if($enabled_reply && !$is_preview){
global $comments, $anchor_text;
$box .= "<span class='reply_link'>[<a href='#".$anchor_text."' onclick='document.getElementById(\"reply_to_box\").value=";
$box .= $comments['id'];
$box .= "; document.getElementById(\"currently_replying_to\").innerHTML=\"".$comments['name']."\";";
$box .= " document.getElementById(\"reset_reply\").style.visibility=\"visible\"'>Reply</a>]</span>";
}
And
reset reply link line in "includes/template/form.php" line ~270
Code:
<a id="reset_reply" href="#<?php echo $anchor_text; ?>" onclick='document.getElementById("reply_to_box").value="0"; document.getElementById("currently_replying_to").innerHTML="no one"; this.style.visibility="hidden";'>Click here to not reply.</a>
Reply Link fix:
stylesheetX.css
Code:
.reply_to .reply_link, .reply_to:hover .reply_link {
visibility: hidden;
}
I'm giving you three guesses...