Asylum Feedback/Support

RE: Mark as Read not working?

207.244.166.107

The Killer Piglet

In Reply to: RE: Mark as Read not working? posted by Rod M on February 26, 2025 at 19:28:18

shoot, I just realized we need to set every chil li to the aa-d class
I'll take a peek tomorrow.
I'd also be much easier to play with teh DOM using JQuery, never really done much with vanilla JS.
KP

Edits: 02/26/25

Follow Ups:

OK, these seems happier - The Killer Piglet 10:17:01 02/27/25 (4)
//you can paste into Chrome console and try it, have to be in Thread View
//it's meant as a drop-in replacement for current function, no need to re-do anything else, within aa.js file
//once I save here it will lose all the formatting, you can paste into https://beautifier.io/ linked below

function confirmRead(ReadForum, ReadThread)
{
if (userID)
{
if (userID != null)
{
if (confirm("Are you sure you want to mark toggle - test " + ReadThread + " as read/unread?"))
{
var anchorElement = document.querySelector('a[href*="' + ReadThread + '.html"]');

var nearestLiParent = anchorElement.parentNode.closest('li');
toggleElement = nearestLiParent;

toggleElement.classList.add('aa-d');

var nextSib = nearestLiParent.nextElementSibling;
if (anchorElement)
{
var count = 0;
var ulChildren;
var children = nearestLiParent.nextElementSibling.querySelectorAll('*');
children = nextSib.querySelectorAll('li');

for (let i = 0; i < children.length; i++)
{

var toggleElement = children[i];
toggleElement.classList.add('aa-d');
}
}
}
}
else
{
alert("You must log in to Mark Threads as read.")
}
}
else
{
alert("You must log in to Mark Threads as read.")
}
}
- https://beautifier.io/ (Open in New Window)
- RE: OK, these seems happier - Rod M 12:55:25 02/27/25 (3)
  Thanks, I get the function, but nothing happens. I've guessing that I need to modify the code to add the root number in each child.
  
  Take a look at the test forum:
  
  https://www.audioasylum.com/cgi/tv.mpl?forum=test
  
  What I need is to grab some of the source for the first topic and the changes to make it work immediately. I tried the console and it didn't do anything, but that's because the link does a perl function and isn't using the JS.
  
  -Rod
  - to clarify, pasting the code into console won't do anything directly. - The Killer Piglet 06:10:26 02/28/25 (0)
    It will just add the new function to the browser session, you can then click the existing 'Mark as Read' Link and it will run the new code.
    
    The current link is JS, that JS then calls a server side function.
    The new function does all the work on the client side.
  - I see the old function is till aa.js - The Killer Piglet 13:48:41 02/27/25 (0)
    It also does nothing new for me when I run click, still have to refresh page
    best approach is to rename that
    confirmRead_oold
    and paste the new code into aa.js
    
    Edits: 02/27/25
  - RE: OK, these seems happier - The Killer Piglet 13:39:37 02/27/25 (0)
    Hum, not sure what is running in Test, I will see if I can snag the JS
    
    It is diff that my code though
    
    it shows
    Are you sure you want to mark thread 1251 as read?
    vs
    Are you sure you want to mark toggle - test 1234 as read/unread?
    
    (I stuk 'toggle - test' in there)
    
    I dropped the script int consoles and run as expected

Post a Followup:

FAQ

Post a Message!

Forgot Password?

Moniker (Username):
Password (Optional):
	Remember my Moniker & Password (What's this?) Eat Me
E-Mail (Optional):
Subject:

Message:	(Posts are subject to Content Rules)

Optional Link URL:
Optional Link Title:
Optional Image URL:
Upload Image:
E-mail Replies:	Automagically notify you when someone responds.

To view your new posting or follow-up, click on the RELOAD or REFRESH button on your browser.

This post is made possible by the generous support of people like you and our sponsors:

Atma-Sphere Music Systems, Inc.