Sidekick icon indicating copy to clipboard operation
Sidekick copied to clipboard

[Feature Request] Saved rolls also remember comments

Open Anduh opened this issue 5 years ago • 6 comments

The current feature to save macros is great, but it would also be nice if it could save any comment alongside the actual roll. That way you can include a comment that explains what the roll is meant for, if other people don't know what your shorthand means.

E.g: /r $i = 1d20+6 #Initiative /r $wolf = repeat(2, 1d20+5, brief) #Bobs-pet-wolf-attack

Of course that would lead to macros potentially taking more space, so it might be smart to limit how long comments can be saved with the macro.

Anduh avatar Jul 07 '19 17:07 Anduh

This becomes problematic with second-level memorizations, e.g. $foo = 1d10, $bar = $foo - 1d2... Maybe a multiline verbose mode. Or in the future a WASM plugin to render the roll as a picture.

ArtemGr avatar Jul 07 '19 18:07 ArtemGr

This becomes problematic with second-level memorizations, e.g. $foo = 1d10, $bar = $foo - 1d2...

Oh I didn't realize this was possible, I just tried this out and it works great! Like how yyou don't even need to place a space between the variable name and any modifier, e.g /r $c+1.

Yeah, that sure sounds like it would lead to trouble. Maybe it could be separated to not be something that gets inherited when making nested roll calls or when saved?

I guess you could include this in the documentation? Doubt I'm the only one who missed out on this feature because this flexibility wasn't mentioned in there.

Anduh avatar Jul 10 '19 08:07 Anduh

I would also appreciate this feature. Possibly, for the second-level memorization issue, limit it to only use the comment for the root memorization?

e.g.

/r $roll = d20 # It's a d20 roll /r $attack = $roll + 4 # Attack roll

/r $attack : Attack roll = (17) + 4 = 21

AndrewHows avatar Apr 15 '20 12:04 AndrewHows

Thanks for the example, Andrew. Yeah, I think that would work!

ArtemGr avatar Apr 15 '20 12:04 ArtemGr

I was just about to make an issue for this, glad to see it's on the docket!

RSchneyer avatar Apr 16 '20 18:04 RSchneyer

Nice to see this might get included.

Anduh avatar Apr 16 '20 18:04 Anduh