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

User-generated templates should be deletable #28195

Closed
jameskoster opened this issue Jan 14, 2021 · 4 comments
Closed

User-generated templates should be deletable #28195

jameskoster opened this issue Jan 14, 2021 · 4 comments
Labels
Needs Design Feedback Needs general design feedback. Needs Design Needs design efforts.

Comments

@jameskoster
Copy link
Contributor

Templates supplied by the theme, that have been customised can be reverted back to their original state.

Templates that have been user generated via the [+] button in the site editing W panel should be deletable.

This flow needs some design exploration to determine where folks should be sent post deletion.

@jameskoster
Copy link
Contributor Author

First pass at this:

Screenshot 2021-01-14 at 12 08 34

The language in the modal can probably use some optimisation :)

As for where users get sent after deleting the current template, in the future I think they should be sent to the mosaic view – this would be consistent with deleting a post which sends you back to the post list.

Until we have the mosaic view implemented, returning them to the last-edited template seems most sensible to me.

@youknowriad
Copy link
Contributor

I'm about to make things more complex :)

What If I actually want to delete a theme's template? Say theme provides "single.html" but I just want to rely on its ancestor "index.html" ? Maybe not a common-use case and potentially something that can come later though.

@jameskoster
Copy link
Contributor Author

I'm about to make things more complex :)

Thanks :D

It probably shouldn't be possible to literally delete a theme .html template in the UI, so this might be better framed more as a "Disable" than "Delete". I can certainly see a use case for that though... you might want to temporarily disable a template while you work on it, or as you said, permanently exclude one specific theme template.

Probably worth opening a new issue for this, since the affordance could theoretically be applied to both theme-supplied and user-generated templates.

@jameskoster
Copy link
Contributor Author

Closing this as it's essentially a duplicate of #31076

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Needs Design Feedback Needs general design feedback. Needs Design Needs design efforts.
Projects
None yet
Development

No branches or pull requests

2 participants