[tw5] uniquified slider (reveal) macro not working

[Dave]

I thought that I was clever to make a slider macro/reveal widget that
stored its state in just the tiddler it was in:

\define slider(label,text)
<$set name=this value=$(currentTiddler)$ >
<$button popup="$:/state/<<this>>$label$" class="btn-invisible
tw-slider">$label$</$button>
<$reveal type="nomatch" text="" default="" state="$:/state/<<this>>$label$"
animate="yes">

$text$

</$reveal></$set>
\end

<<slider "*" "hello there">>


but when I put "<<slider "*" "hello there">>" in two separate tiddlers,
they both open when I click on either of them. If I change the "*" to
something different in the second one, like "**" they stay separate, but I
thought the use of <<this>> (whose value=$(currentTiddler)$) would put the
state in different places because the tiddlers are named differently.

What am I doing wrong here?


thanks,
- Dave

[Mohammad]

Hi Dave
Check this
http://tw5custom.tiddlyspot.com/#Add%20a%20slider%20macro

/Mohammad

[Jed Carty]

For reasons that have to do with how the macro parser works that I don't
think I can explain in an understandable way right now your set widget is
the problem.

try using this macro instead:

\define slider(label,text)
<$button popup="$:/state/$(currentTiddler)$$label$" class="btn-invisible
tw-slider">$label$</$button>
<$reveal type="nomatch" text="" default="" state="$:/state/$(currentTiddler)
$$label$" animate="yes">


$text$


</$reveal>
\end


<<slider "*" "hello there">>

the quick thing is that $(variable)$ is a direct text substitution and
<<variable>> doesn't work the same way. if you look at the state tiddler
created using yours it is literally called $:/state/<<this>>* without the
<<this>> part being substituted.

[Dave]

Thank you Jed (and Mohammed)

I'm glad the reason mine didn't work was convoluted... That way I don't feel so bad for getting it wrong, ha ha