The spoiler feels very 1998. It reminds me of the <marquee>
tag, but I like it.
I think the current spoiler-guard
If you saw this text before you knew this was supposed to be hidden I haven't done my job.
isn't clear what it is. At first it just looks like the poster made a mistake and left a blank box.
The spoiler shouldn't reveal what is under it on flyover. It should require a click.
It should be clearer that it hides some information from the UI at a first glance. Gray can would work well against black text. Stripes help indicate it is a type of "scratchable screen" -
Don't look now, but css3CSS 3 supports candy stripes
<style>
.spoiler {
background:repeating-linear-gradient(
135deg,
#fff,
#fff 15px,
#eee 15px,
#eee 20px
);
}
</style>
<p class="spoiler">
some spoiler text
</p>