|
Welcome,
Guest
|
|
TOPIC: Hiding Comments
Hiding Comments 15 years 4 months ago #4934
|
Hiding Comments 15 years 4 months ago #4936
|
I think this is not as good idea as it might first sound for one simple reason - in JoomlaComment, currently there's no "checking" for multiple votes. This means that a single user can continuously give negative votes to one comment - there is no way to check for multiple from the source IP I think and it would be pretty intense to implement....?
In regards to hiding via #class, I think we don't even need JavaScript to do it - it is do-able via PHP and CSS i'm sure, but to have the "This comment was hidden due to bad ratings. Click here to unhide" text will probably have to be AJAX yeah  I will remember this when working on the new template(s). |
|
Hiding Comments 15 years 4 months ago #4949
|
Ok, here is my kludge...
In a template, just before {post}, put this Javascript: <script type="text/javascript">
function HideBadComment(id) {
var contentChild = document.getElementById("content"+id);
var moreElement = document.getElementById("more" + id);
var moreHTML = "<font color='#d0d0d0'><i>(more...)</i></font>";
var lessHTML = "<font color='#404040'><i>(less)</i></font>";
if (contentChild && moreElement) {
if (contentChild.firstChild.className == "hide") {
moreElement.innerHTML = moreHTML;
moreElement.isHidden = true;
moreElement.onclick = function() {
if (this.isHidden) {
this.isHidden = false;
this.innerHTML = lessHTML;
contentChild.firstChild.className = "";
} else {
this.isHidden = true;
this.innerHTML = moreHTML;
contentChild.firstChild.className = "hide";
}
}
}
}
}
</script><script type="text/javascript">HideBadComment("{id}");</script><td style='overflow: hidden;'>{content}</td><td id="content{id}" style='overflow: hidden;'>{content}<div id="more{id}" style="cursor:pointer; cursor:hand;" /></td> p.s. On a different topic, after taking time to contribute this, I hope you will be able to (soon) add code to store the logged-in user ID for each vote, in addition to the IP, and go by that instead (for security), thanks. |
|
Hiding Comments 15 years 4 months ago #4956
|
James,
Thanks for that sweet action of Java script there! Since I don't deal with Java pretty much ever, is that sort of thing XHTML valid? Or does it have to be in a seperate .js file? Anywho, I will definately save this thread for when we crack at the new templates in new releases the IP+username checking is a definate necessity. I've referrenced this thread in the dev site and submitted it as a future feature Thanks a lot James! |
|
|
Time to create page: 0.117 seconds
