Chat: after deleting a message it disappears only after refresh #2354

[pvm-code]

#2354

What was changed?

• Added handler action for deleting messages.

[netlify]

✅ Deploy Preview for preview-common ready!

Name	Link
🔨 Latest commit	0665b34
🔍 Latest deploy log	https://app.netlify.com/sites/preview-common/deploys/658435aad7cf6f0008202ee7
😎 Deploy Preview	https://deploy-preview-2398--preview-common.netlify.app/
📱 Preview on mobile	Toggle QR Code... Use your smartphone camera to open QR code link.

---

To edit notification comments on pull requests, go to your Netlify site configuration.

[elatif2020]

@MeyerPV
See the screen recording:
After message deletion the focus is been changed to another item in the inbox.
Also the loader location in the delete message button should be centered.

Untitled.video.-.Made.with.Clipchamp.4.mp4

[andreymikhadyuk]

@MeyerPV If PR is approved by Latif, then merge please.

[roienatan]

@MeyerPV can you please fix the location of the loader in this PR?

See ticket also

[roienatan]

Or actually to be consistent with deleting a discussion/proposal instead of a loader we should show "deleting...". What do you think? @elatif2020

[elatif2020]

Or actually to be consistent with deleting a discussion/proposal instead of a loader we should show "deleting...". What do you think? @elatif2020

Thanks @roienatan this is a good point. For now both are good. I hope we will make soon a broader work to use the same standard behaviors across the app (especially same pop-ups design).
So if it's easy let's switch to "deleting..." like we have for deletion of discussions, and otherwise we can stay with the spinner, just move it to the center.