New plugin: Spoilers

Support for XEP-0382: Spoiler messages

Work in progress! Open for discussion around its development

changed target branch from master to gajim_1.1

Hi Seve, works as intended for me on Windows, do you use Wayland by any chance?

Hm i see now, sometimes it works sometimes not

The popup seems to close on the first typed char, and if i delete the last char in the input, on all other cases it stays open, i investigate why this happens

This happens because your stanza outgoing code gets executed when gajim sends chatstates which happens on the first key press (typing..)

    def _stanza_message_outgoing(self, event):
        if not event.message:
            return

This should fix it

Hey @lovetox ! I'm very sorry, I didn't get any notifications about you adding comments to this MR. I'm reading your comments just by chance now, after working on some changes.

Regarding the chatstates, yes, I found that problem too, but it seems I solved in a different way and yours is much nicer, because on the current status, there's a bit of delay when closing the popover after sending the message. If I can do it as you mention in _stanza_message_outgoing would be much better, indeed.

Thank you for your comments, I'll make sure I check them more often

I have to add more things, like using it in MUCs and others. I'll ping you on updates

added 1 commit

• 1697584b - Basic functionality working with spoiler mode being disabled after sending the message.

Compare with previous version

added 1 commit

• d9b93916 - Logic for message_sent can be done in _stanza_message_outgoing, based on Phillipp's review

Compare with previous version

added 1 commit

• a45107a3 - Add MUC support

Compare with previous version

added 1 commit

• 08e368e5 - Fix logic for multiple spoiler elements not getting marked as spoiler when language is unknown

Compare with previous version

added 1 commit

• cfe951e3 - Use a better string for displaying the message

Compare with previous version

Hey @Seve can we merge that?

Service discovery is the last thing missing, haven't been able to work on it yet, but I will ping you back when it's done

Hi @Seve any news? :)

changed title from Spoilers plugin to New plugin: Spoilers

changed the description