Re^4: Node parents shown while viewing a node

[oha]

in reply to Re^3: Node parents shown while viewing a node
in thread Node parents shown while viewing a node

I agree with you and that's what i did.

It was just a "cosmetic" suggestion. I just wonder if someone considered it a nice addition or a bad one (the parents chain can be misleading i know).

Then probably it's up to pmdev to decide if it's worth to implement or not; probably based on the replies here and the time required to do it.

Oha

[jdporter]

OK, just for you: Put the following code in your Free Nodelet:

<!-- the HTML: -->
Hide old replies: <input name="hide_old_replies" id="hide_old_replies"
+ type="checkbox" onClick="toggle_old_replies()" />

<!-- the JS: -->
<script language="javascript"><!--

    function toggle_old_replies(){
      var chk = document.getElementById('hide_old_replies');
      var state = chk.checked ? 'none' : 'table-cell';
      var allelems = document.all ? document.all : document.getElement
+sByTagName("*");
      var inc = 0;
      for ( i = 0; i < allelems.length; i++ ) {
        if ( allelems[i].className == "reply-body" ) {
          allelems[i].style.display = state;
        }
      }
    }

//--></script>

This only toggles the visibility of the bodies of "old" replies; the header boxes etc. for old replies remain visible. Hopefully that's sufficient for your desires.

Thanks to tye for applying the patch I wrote to make this possible.

[oha]

ty both very much!

i tried it and found the Free Nodelet Setting eat up the square brackets on js, so i patched it to work, as in Re^3: Free Nodelet freed.

here it is

<!-- the HTML: -->
Hide old replies: <input name="hide_old_replies"
id="hide_old_replies" type="checkbox" onClick="toggle_old_replies()" /
+>

<!-- the JS: -->
<script language="javascript"><!--

    function toggle_old_replies(){
      var chk = document.getElementById('hide_old_replies');
      var state = chk.checked ? 'none' : 'table-cell';
      var allelems = document.all ? document.all : document.getElement
+sByTagName("*");
      var inc = 0;
      for ( i = 0; i < allelems.length; i++ ) {
        var elem = eval("allelems\x5Bi]");
        if ( elem.className == "reply-body" ) {
          elem.style.display = state;
        }
      }
    }

//--></script>

Oha

[ikegami]

Or you can use the much more readable Free Nodelet escape character (`).

var elem = allelems`[i];

[tye]

The Free Nodelet is no longer supposed to mess with square brackets that occur inside of HTML comments. I'm also pretty sure that jdporter tested that code so I'm doubly surprised that you had this problem.