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

add "delete post" button to the PSM #649

Merged
merged 1 commit into from Apr 19, 2017

Conversation

kevinansfield
Copy link
Collaborator

closes TryGhost/Ghost#8332

  • moves toggleDeletePostModal action from the edit controller to the editor-base-controller mixin
  • adds delete button to the bottom of the PSM unless it's a new post
  • adds deletePost attribute to gh-post-settings-menu to allow a delete post action to be passed in

Note: this PR adds the necessary functionality but the styling for settings menu delete buttons is currently broken (TryGhost/Ghost#8345) and will be fixed in a separate PR

closes TryGhost/Ghost#8332
- moves `toggleDeletePostModal` action from the `edit` controller to the `editor-base-controller` mixin
- adds delete button to the bottom of the PSM unless it's a new post
- adds `deletePost` attribute to `gh-post-settings-menu` to allow a delete post action to be passed in
@coveralls
Copy link

Coverage Status

Coverage decreased (-0.05%) to 73.585% when pulling 2d0c772 on kevinansfield:psm-delete-button into 85e3bab on TryGhost:master.

@kirrg001 kirrg001 self-assigned this Apr 19, 2017
Copy link
Contributor

@kirrg001 kirrg001 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

functionality works 👍

@kirrg001 kirrg001 merged commit 6e573b7 into TryGhost:master Apr 19, 2017
@kevinansfield kevinansfield deleted the psm-delete-button branch June 4, 2018 10:02
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
3 participants