-
-
Notifications
You must be signed in to change notification settings - Fork 3.2k
feat: Remove delete page from toolbar #7748
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
base: main
Are you sure you want to change the base?
Conversation
discard specific verbose_name from perms_needed
As discussed with @fsbraun on Slack, pages shall exclusively be deletable through the page tree but not through the toolbar.
I would second the notion. Let me add my rationale:
The alternative would be:
|
In the course of this, I would also remove the item "Duplicate this page". Reason is, that users have no way to specify the target and hence that duplicate would end up as a sibling of the target with a slug named "copy-of-…". By allowing the create duplicates from the page tree, this problem can be avoided because normally those duplicated pages are added to a different parent anyway. |
There hasn't been any activity on this pull request recently. This pull request has been automatically marked as stale because of that and will be closed if no further activity occurs within 7 days. Thank you for your contributions. |
Description
Currently the toolbar offers a menu item to delete the current page.
This is problematic, because the user first has to change into edit mode, before allowing him to delete that page.
Moreover, after deletion which page shall the browser render?
@fsbraun and I therefore discussed on Slack and concluded, that deleting a page only shall be possible through the page tree.
develop