Spoiler Alert

Cheun

These are the type of spoilers that I love <3 Blurr to make a blind person having a hard time, have an even harder time.

wignu

Thanks for updating this sijad , one request - wouldit be possible to have an admin option to have the spoiled text/image either blurred or, as with your previous version, removed?

Also is it possible to have a title for the spoiled code?

sijad

wignu do you consider details?

JoshyPHP

@sijad For future reference, if you add a title for the spoiler I recommend naming the bbcode parameter explicitely in the definition, e.g. [SPOILER title={TEXT2}]{TEXT}[/SPOILER]. If you don't name it, it'll be called spoiler.

The value will be available in the template either as @title wherever XPath is expected or as {TEXT2} anywhere else.

wignu

@sijad, thanks for that, would you be able to do that but with spoiler BBCode?

sijad

wignu you mean title?

wignu

sijad wignu you mean title?

Yes with the title, the ability to hide whatever's spoilt completely and just have a default spoiled label if no title is set ?. Be great if it was possible thanks.

sijad

wignu I'll put title in todo but I'm not sure about hiding as long as details do the same

JoshyPHP

@sijad The spoiler markup currently uses a span element. It prevents it from hosting a block element like a quote or code block, or an embedded video with a responsive div wrapper. Have you thought about using a div element, perhaps with display: inline-block?

This is related to that post.

sijad

JoshyPHP thanks for informing me.
do you think it's ok?
sijad/flarum-ext-spoiler-alertd238f43

I'm not sure about display: inline-block, because other elements (like <p> tags) are display: block;, and it prevents displaying spoiler inline with other elements any way, what do you think?

JoshyPHP

You're right about p and div. If you change it to a div I think it'll break the current paragraph. The best solution might be to have different BBCodes for inline spoilers and block spoilers.

jordanjay29

JoshyPHP Maybe add in the original click-to-show implementation for block-style spoilers? Makes sense you'd want to hide large items like quotes and codes, but keep inline content "hidden" just behind a blur.

sijad

JoshyPHP The best solution might be to have different BBCodes for inline spoilers and block spoilers.

don't you think there might be a better solution? this may confuse users

JoshyPHP

If there is one, I can't see it at the moment. I think that having two distinct BBCodes for inline text and blocky things is better than having only one that does both.

wignu

For simplicity for users it would be preferable to have one spoiler code to rule them all ?

DAMNimST0NED

wignu This exactly.

wignu For simplicity for users it would be preferable to have one spoiler code to rule them all

sijad

wignu I just followed discourse naming, can you please explain how users can handle both spoiler and detail with only one extension?

f0r3v3r

It is just weird because Details does what the original BBCode spoiler does, and this hijacks spoiler and makes it do something entirely different. I feel like details should still be spoiler (I know this ship has sailed) and this extension should be named and called with something else.

Just my two-cents, albeit too late...

DAMNimST0NED

f0r3v3r I understand what your getting at. This seems more like a pay to unlock or like to unlock thing then a true spoiler tag. But luckily we still have the other option.

HalilEsen

sijad mobile web not work

hanleylau22

Kulga Ok, that's pretty cool

i like that

« Previous Page Next Page »