Delete study flair

[kraktus]

fix #15653

[kraktus]

[kraktus]

Thanks for the help. Is $set("field" -> none) not unsetting field actually used in the codebase? Since it seems like a footgun when editing field, unlike something like optionA.map: x => $set("field" -> x)

[ornicar]

$set("field" -> none) does not set field. It also doesn't unset it, because $set and $unset are distinct mongodb commands.

[ornicar]

I think the DELETE button should have a top margin instead of a left margin.

[kraktus]

$set("field" -> none) does not set field. It also doesn't unset it, because $set and $unset are distinct mongodb commands.

Yeah, but why should we accept option when using $set then? As shown in this PR, it eat the error when trying to unset down the line.

[ornicar]

We accept options so that we can set fields depending on whether their values are Some or None. I think it's used in many places of the codebase.

[kraktus]

I think the DELETE button should have a top margin instead of a left margin.

Agreed, better? The button label is still to the left of the emoji picker, due to the padding inside the button. When hovering over it we see they're actually aligned