Fix editing in threaded mode adding additional local-only emoji

[Plastikmensch]

The case blocks are not scoped, so redeclaring variables doesn't work, so declare them outside and setting them as needed in case blocks.

[ClearlyClaire]

Ok I'm not sure about merging this, it's a bit of a mess, the eye is part of the post, and can be removed through editing (but that probably shouldn't be the case?)

[Plastikmensch]

Fixed Markdown being stripped when editing a Markdown formatted toot by using action.text instead of action.raw_text, which was undefined.

Ok I'm not sure about merging this, it's a bit of a mess, the eye is part of the post, and can be removed through editing (but that probably shouldn't be the case?)

It is indeed a bit of a mess and you are right, when sending a local-only toot the local-only emoji should be unconditionally added if the local-only option is set. That's why it needs to be removed. I have no idea why that doesn't happen when it's manually removed, but I though that was on purpose?

I should probably also mention that this PR builds on top of #2430

[Plastikmensch]

This is no longer needed as it's included in #2430