I think we do, as a structure. The key thing for me is that we create the situation where, by default, a post being deleted is not permanent. It *can* be made to be default that a post being deleted is permanent, but that the default is a soft delete.
Sure.
However, if soft deletes are default, then I think it should be associated with enabling post moderation -- so it should be enabled by default. IIRC if post moderation (core features) is disabled, SMF/Wedge don't check for the is_approved bit. We'd need to check it everytime because we'll probably use is_approved to store the state (unapproved/approved/deleted).
Okay I should probably check the actual code before I say things like that ahah...
Replacement for the delete button when the post is deleted, or if the admin so desires, replacing it normally. But the default should be to replace it in deleted mode.
We're in agreement here.
Not yet, I felt like hell and went back to bed, only just got back up, just in time to do chores.
Still not?
;)In the meantime I've converted all of the stylings to the simple css syntax. Phew! It was a bit nightmarish... It's not perfect and it's untested, so I'll probably wait until tomorrow to commit, but at least it's done. Means I can really move Warm to be the default...
Also, how do we display a deleted topic?
Oh, that'll probably be an interesting thing to implement, ahah...
Either we don't show anything, and do it only in the admin area, or we show the deleted topic link only for admins but in the message index itself, or we show the 'deleted by...' mention for everyone. Red background probably ain't too good when you just say "deleted" without linking, btw (same for showing a "message deleted" mention inside topics for non-admins. Should be a bit grayish or something I guess...)
.approvebg, things like that... There are different background colors available for the message index. We just need to add a new one for deleted topics.
Topics also have a is_approved flag IIRC. (Or they use the one for the very first post. Either way, it shouldn't be a performance issue to have this.)