Web-stories-wp: Alert UI

[BrittanyIRL]

We've been using the default browser alert to confirm deleting a story. We should switch this out for an actual alert of some sort. We could follow the one in material ui to cut down on UX time.

This story will be comprised of

• Alert UI
• Tests
• Storybook
• Replacing default browser pop up in use on removing stories

https://material-ui.com/components/dialogs/#alerts

[BrittanyIRL]

(going to move forward mimicking material ui to give users a better experience. We can always circle back with new designs.)

[swissspidy]

There's already a Dialog component in the editor app, if we can re-use that.

With that being said, we should be really starting to properly extract commonly used components into their own folder/package

[BrittanyIRL]

Thanks, @swissspidy I'll see about copying or mimicking that with the intentions of pulling it out into a shared space.

[o-fernandez]

This makes sense - go ahead with your plan to mimic material UI and we can always do a UX exercise if necessary. Don't think we should block the initial effort on UX bandwidth.

[BrittanyIRL]

Now delete story confirmation looks like this:

instead of this:

[o-fernandez]

That looks good to me as a start, we can make any UX tweaks as necessary in future tickets/revisions.

[csossi]

Verified in QA

[samitron7]

@swissspidy are story deletion permanent? If so, @BrittanyIRL if it's easy to change the message to read:

Delete permanently
[Story title] will be permanently deleted. This action cannot be undone.

If not, we can close out and I'll file a ticket for update when we get the new visuals in.

[swissspidy]

@samitron7 It's up to us. Right now & by default, deleting means putting into the trash. Items can be recovered from trash for around 30 days (depends on website configuration). IMO we should keep it that way.

Trash recovery can currently be done via the classic WP list table.

[BrittanyIRL]

text updates like the above are super easy! Just let me know if you want it changed per Pascal's comment above.

[samitron7]

Based on @swissspidy's comment, let's just keep it that way.

Verify in UAT