-
Notifications
You must be signed in to change notification settings - Fork 2k
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
Scheduled Updates: Add Plugins menu item with drilldown #90577
Comments
Support References This comment is automatically generated. Please do not edit it.
|
Nice to hear that 🚀 I have a question. Are we using the same data table (A4A - JetPack) as the one on the site list? Let's make sure that we won't add one more data table UI. Please ping me whenever you need me, I'm happy to help :) |
Just wanted to double check that the submenu item doesn't live on the global nav itself, but shows only after you click into "Plugins". |
Exactly, that's the idea.
Are you talking about the sites table, which lives on the In other efforts, we could create a reusable data table component in a unified, maintainable place and apply it to the existing features (sites, scheduled updates, etc.). |
How far are we from this? In my mind, I thought we had the data table component that could be reused =/ |
@cleacos just tagging you on this convo. Figured you might be able to share some context on the current state of things with regards to reusing the data table for things outside of sites. |
If I understood correctly, you want to create a new Plugins section and reuse the data table component with the Preview Pane option for any expanded option. It's possible with the current implementation; just use the
|
This will add a "Plugins" menu item to the new multisite UI with a drilldown function to expose submenu items.
We don't want to expose the multisite plugins page at this point, so the only submenu item should be "Scheduled Updates".
Let's make sure we stay closely aligned with @davemart-in, @lucasmendes-design, and @JanaMW27 on this one.
See video mockup in p9Jlb4-bDg-p2#comment-11238
The text was updated successfully, but these errors were encountered: