Freaky spoiler tags?

[D'nial]

It's not just you. I've noticed several spoiler tags in other members' posts that don't work now.

[kaelisebonrai]

Still broken as of 25/03/2010, using the default forum skin. Or at least the older content is...
Testing...

test spoiler Show Spoiler

test

[diafero]

The main issue is that this makes many old posts relying on spoiler tags unreadable

[Tsar Hoikas]

The main issue is that this makes many old posts relying on spoiler tags unreadable

Thanks for the poke. I'll be doing a bit of maintenance on the server and forum later this afternoon. I theoretically understand why emails from the server are being blocked by Gmail, etc., so I'm going to try to fix that, update phpBB to the latest version, then I'll fix the spoiler tags. I wish that the spoiler tag wasn't a code mod. Those are annoying.

[diafero]

Thanks a lot for all your hard work, Hoikas

In the phpBB3 forum I set up, I also added a spoiler tag, but using the "custom tag" feature - no code mod required! I could not get custom titles (for the collapsed spoiler), but the rest works. Here's the configuration:
BBCode usage:

Code: Select all

[spoiler]{TEXT}[/spoiler]

HTML code:

Code: Select all

<dl class="codebox"><dt>Spoiler: <a href="javascript:void(0);" onClick="var spoiler = this.parentNode.parentNode.getElementsByTagName('dd')[0]; if ( spoiler.style.display == 'none' ) { spoiler.style.display = 'block'; this.innerHTML = 'Hide text'; } else { spoiler.style.display = 'none'; this.innerHTML = 'Show hidden text'};">Show hidden text</a></dt><dd style="display:none">{TEXT}</dd></dl>

It shows "Show hidden text" when collapsed and "Hide text" when expanded. However I don't know how it would deal with all the existing spoiler tags on these forums. Maybe if you manage to also get the custom title in there, it should work because it has the same syntax?

[Tsar Hoikas]

Thanks!

I made a few changes, so I guess I'll share them with you since it is your bbcode.

Code: Select all

[spoiler={TEXT1}]{TEXT2}[/spoiler]

Code: Select all

<dl class="codebox"><dt>{TEXT1} <a href="javascript:void(0);" onClick="var spoiler = this.parentNode.parentNode.getElementsByTagName('dd')[0]; if ( spoiler.style.display == 'none' ) { spoiler.style.display = 'block'; this.innerHTML = 'Hide Spoiler'; } else { spoiler.style.display = 'none'; this.innerHTML = 'Show Spoiler'};">Show Spoiler</a></dt><dd style="display:none" class="code"><code>{TEXT2}</code></dd></dl>

[Deledrius]

It's working! Thanks!

[Aloys]

testing Show Spoiler

Yay, it works! Nice work. Let's raise a collective toast to Hoikas and Diafero!

[ZURI]

Cheers!

[Egon]

:) Show Spoiler

Cheers!