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.

Sign up for GitHub

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

Jump to bottom

theme:enable should be theme:install #5229

Closed
gitressa opened this issue Sep 11, 2022 · 1 comment · Fixed by #5230
Closed

theme:enable should be theme:install #5229

gitressa opened this issue Sep 11, 2022 · 1 comment · Fixed by #5230

Comments

@gitressa
Copy link
Contributor

Describe the bug
In Drupal 8.x and above, there is no longer a way to "enable/disable" themes. Instead we "install/uninstall" them.

However, Drush still has a theme:enable command, while theme:uninstall is fine.

This is confusing (enable is the opposite of uninstall?) and inconsistent with the Drupal UI.

To Reproduce
List commands in Drush and you'll see that theme:enable turns a module on, and theme:uninstall turns it off (which also involves adding/removing configuration, database tables, and/or content).

Expected behavior
The theme:enable command should be named theme:install for consistency with the Drupal UI and with the theme:uninstall command.

Additional information
#4588.
@gitressa gitressa mentioned this issue Sep 11, 2022
Merged
@weitzman
Copy link
Member

Lets discuss in the PR

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Update theme:enable to theme:install
2 participants
@weitzman @gitressa