-
Notifications
You must be signed in to change notification settings - Fork 40
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
Remove shortcut module from core #76
Comments
I actually like this module, and use it to list frequently used pages for people using my sites. If we really do want to take it out, then it should go into contrib. I guess I can take over it in that case, but we should probably figure out how we are going to handle contrib for backdrop. |
+1 that I do like it, but I would also accept it as contrib. |
All tests are OK |
Thanks guys. Yeah support for shortcut in contrib would be great demonstrations of extending the toolbar. We're not sure where we're going to head with the toolbar integration (e.g. the mobile implementation, adding search or dropdowns like admin_menu... there's a lot unknowns). But for Backdrop core, I feel like shortcuts are an often confusing and rarely understood feature. So I'm in favor of removing too. How we handle contrib is still being worked out. For those interested I'd suggest following this issue in the BackdropCMS.org queue: backdrop-ops/backdropcms.org#8 |
Merged backdrop/backdrop#70. Thanks so much @luiselizondo! You're a module-removing machine. :) |
No description provided.
The text was updated successfully, but these errors were encountered: