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

Remove the metaboxes (or update to match the new Gutenberg style) #1747

Open
stevejburge opened this issue Aug 27, 2024 · 4 comments
Open

Remove the metaboxes (or update to match the new Gutenberg style) #1747

stevejburge opened this issue Aug 27, 2024 · 4 comments

Comments

@stevejburge
Copy link
Member

stevejburge commented Aug 27, 2024

Compare the Planner metaboxes to the Future metabox. Maybe we don't need the Notifications metabox any more?
Screenshot 2024-08-27 at 3 18 40 PM

@stevejburge stevejburge changed the title Update the metaboxes to match the new Gutenberg style Remove the metaboxes (or update to match the new Gutenberg style) Sep 24, 2024
@olatechpro
Copy link
Collaborator

@stevejburge Future plugin is using Gutenberg Blocks while Planner is still using Metabox from classic editor. So, unless we decided to transfer these metabox to blocks, the icons will always be different but wondering if difference in icons really worth the time we need to create blocks of these metaboxes?

If we're removing notifications metabox cos of difference in icons only, what happen to Editorial Field metabox, we remove it too?

@stevejburge
Copy link
Member Author

@olatechpro Can we move those a legacy status, so they don't show for new installs?

@olatechpro
Copy link
Collaborator

@stevejburge We currently don't have anything legacy settings in planner.

So, to confirm what you mean:

  1. We should add new settings in both Editorial Fields settings and Notification fields settings that reads "Enable Metabox" which is disabled by default for new installation?
  2. OR We should simply remove both metabox for new installation? I'm not sure this is a good idea especially for Editorial Fields

@stevejburge
Copy link
Member Author

@olatechpro Closer to option 1, please.

The Editorial Fields box will show only if they are enabled.

The Notifications box could be a setting.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants