Skip to content
This repository has been archived by the owner on Dec 16, 2022. It is now read-only.

Delete a post whilst in the Customizer #137

Closed
danielbachhuber opened this issue May 12, 2016 · 7 comments · Fixed by #152
Closed

Delete a post whilst in the Customizer #137

danielbachhuber opened this issue May 12, 2016 · 7 comments · Fixed by #152
Assignees
Labels
Milestone

Comments

@danielbachhuber
Copy link
Contributor

It would be great if I could trash or delete a post whilst in the Customizer.

I recognize that this task isn't small, and will need some UX exploration, etc. before it can go to development.

@westonruter
Copy link
Contributor

Definitely. This is something we'll work on after #134 is merged for adding post insertion.

@westonruter westonruter added this to the 0.7 milestone May 12, 2016
@westonruter
Copy link
Contributor

We'll also have the control for managing the post status (#40) which could potentially include “trash” as one of the options until a dedicated button/UI is provided for deletion.

@danielbachhuber
Copy link
Contributor Author

We'll also have the control for managing the post status (#40) which could potentially include “trash” as one of the options until a dedicated button/UI is provided for deletion.

Oh, huh. That kinda sucks, but could work as an interim solution.

@westonruter
Copy link
Contributor

It's just a matter of the UI. Implementing trash would be simple, as it would be just setting the post_status for the post setting object to trash. So yeah, I don't see why a delete button and the post status dropdown couldn't be done at the same time.

@westonruter
Copy link
Contributor

As for the behavior of clicking “Trash” I think it should:

  1. Set the post_status for the post setting to trash.
  2. Trigger a refresh of the previous (should happen automatically).
  3. Collapse the post section for the trashed post.
  4. Add a deleted style to the post section in the list of posts in the panel, perhaps with 50% opacity.
  5. Upon clicking Save, the post section should be removed from the panel entirely.

@danielbachhuber
Copy link
Contributor Author

@westonruter Did we end up with a button here, or just the status dropdown?

@westonruter
Copy link
Contributor

@danielbachhuber this is being discovered in #172.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
Projects
None yet
Development

Successfully merging a pull request may close this issue.

4 participants