Spoiler Tags (1.5.1)

[triplew64]

Feb 4, 2013 14:10:46 GMT -8 Todge said:

Feb 3, 2013 15:50:58 GMT -8 triplew64 said:

I've done some basic testing of v1.2.8. I'm not a developer so forgive me if I ask simple questions.

One
thing that I do notice is that the link does not become "active". In
other words, previously my cursor would turn into the finger to click on
the link. It does not do that now.

Also, how can I (if I can) place an image as the clickable link?

The cursor should still change to the pointer to show you the spoiler is a link, what browser are you using, and could you post a link to a page containing a spoiler please?

As for using an image as a spoiler title, I'm still working on that.

I am using IE9 and Firefox 18.0.1

You can see what I  am talking about on this page.
warofthedamned.freemessageboards.com/thread/108/test-thread

[Todge]

You have added an image tag to the CSS of the spoiler title, remove that and all should be good.

[triplew64]

That was it thanks.

[Tim Camara]

Todge - I discovered today that this plugin gives a script error in Opera (12.12). Not sure if it's important enough to fix, but thought I'd let you know anyway.

[Todge]

Could you elaborate please? I just checked using Opera and all seems to be ok.

[Tim Camara]

Sorry for the incomplete bug report. Here's the full report from the error console in Opera:

[2/5/2013 3:02:56 PM] JavaScript - tim.proboards.com/post/new/422684
Inline script thread
Uncaught exception: ReferenceError: Undefined variable: addSpoilerButton
Error thrown at line 657, column 9 in tim.proboards.com/post/new/422684:
window.addEventListener('load', addSpoilerButton, false);

Browser Details:
About Opera
Version information
Version
12.12
Build
1707
Platform
Win32
System
Windows 7

[Todge]

So you are not seeing the spoiler button on the reply page? Strange, as that variable is a function name.

Could you try deleting the plugin, then downloading and importing is again please. It sounds as if part of the code is missing.

[Tim Camara]

Hm, works after reimport. Bug report retracted...

[Todge]

Thank you.

[ermentrude]

I admit I'm clueless. We have a forum that is only 800px wide, purposely. If I make the forum 850px wide, I can see the button. Is there any way I can control where it appears so it is visible?

[Todge]

Feb 5, 2013 18:54:23 GMT -8 ermentrude said:

I admit I'm clueless. We have a forum that is only 800px wide, purposely. If I make the forum 850px wide, I can see the button. Is there any way I can control where it appears so it is visible?

Can you try v1.3.2 please.

[ermentrude]

I get the following in Firebug and cannot see the button at all, even if the screen is wider:

ReferenceError: getBy is not defined
[Break On This Error]

var iconCell = getBy('li','class','button button-insertSmiley');

[Todge]

Strange.. can you try it now please.

[ermentrude]

Worked like a charm, thank you!

[merleh]

Todge, is it possible for an admin to change the default text into Spoiler: Click here to show/click here to hide? Like the result you get if you use the (spoiler)(/spoiler) tags without the plugins?