Redacting the removal of a widget does not restore it

[turt2live]

Description

Unlike #4915, navigating away and back again does not restore the widget.

Steps to reproduce

• Add a widget
• Remove the widget (with the X)
• Redact the "X widget removed by Z" event

The widget should be restored (immediately refreshing the app bar as well).

Log: not sent

Version information

• Platform: web (in-browser)
• Browser: Chrome 60
• OS: Windows 10
• URL: riot.im/develop

[t3chguy]

not sure how this would work,

when I redact the removal and look at the room state
working widget config is in prev_content, and I can see that the state is redacted so I could apply prev_content, but then if I redact the addition of the widget which occured right before that, nothing changes and the widget would still be alive :L

[turt2live]

@lampholder this is going to sound odd, but I'd argue this is lower priority. It's unlikely (hopefully) that someone is going to redact the removal of a widget. I only did because I was curious as to what would happen

[lampholder]

Heh, I agree the specific problem probably isn't P1 - my P1 was just a reflection that the interplay between widgets and events really needs to be thought about properly before we dig too deep a hole.

You're right though P1 probably is a bit strong :)