Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Tag Management: Delete flow #4633

Closed
ErisDS opened this issue Dec 12, 2014 · 1 comment · Fixed by #4656
Closed

Tag Management: Delete flow #4633

ErisDS opened this issue Dec 12, 2014 · 1 comment · Fixed by #4656
Assignees
Labels
affects:admin Anything relating to Ghost Admin
Milestone

Comments

@ErisDS
Copy link
Member

ErisDS commented Dec 12, 2014

Currently, deleting a tag is a simple matter of hitting this huge button:

It's very easy to do!

The delete tag mechanism should be a small plain text link as per the original screenshot / design:

Also clicking the link should cause a modal, similar to the ones we show for posts and users:

It might be a good idea to add the number of posts the tag is linked to on this screen, as has just been added for users, but note that deleting the tag won't delete the posts, this is just an indicator of how many posts the tag will be detached from.

@ErisDS ErisDS added the affects:admin Anything relating to Ghost Admin label Dec 12, 2014
@ErisDS ErisDS added this to the Next Backlog milestone Dec 12, 2014
@novaugust novaugust mentioned this issue Dec 12, 2014
20 tasks
@felixrieseberg
Copy link
Member

Dibs.

felixrieseberg added a commit to felixrieseberg/Ghost that referenced this issue Dec 17, 2014
Closes TryGhost#4633
- The ‘delete’ button is now a smaller plain text link, opening a
confirmation modal analog to the delete user/post flow
- Adding a post count is dependent on TryGhost#4654, but the modal is already a
neat step up from the immediate, warning-less deletion.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
affects:admin Anything relating to Ghost Admin
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants