Codeinwp / neve

A fast, lightweight, AMP ready WordPress theme built with speed and usability in mind.
https://themeisle.com/themes/neve/
GNU General Public License v2.0
262 stars 84 forks source link

single customization options as gutenberg plugin sidebar #1572

Closed selul closed 4 years ago

selul commented 4 years ago

Description:

Right now we have the single page options as old editor meta box inside the admin. We need to look into moving this setting to Gutenberg plugin sidebar

Some things that we need to take care of:

selul commented 4 years ago

adding here the new design that John made

Some things regarding the implementation:

selul commented 4 years ago

Adding some things here based on the progress we have right now, just to make sure we don't miss them. If those are present, we can ignore them.:

rodica-andronache commented 4 years ago

@cristian-ungureanu here is what I saw while testing

https://www.loom.com/share/ad1ef4afec3643c1bee10d7c044bb042

The message could also be updated to “Note: It seems that the shop page has an individual sidebar layout already set. To be able to control the layout from here, edit your page and set the sidebar to “Inherit”

cristian-ungureanu commented 4 years ago

@rodica-andronache All the issues you've mentioned are now fixed. Let me know if there is anything else.

rodica-andronache commented 4 years ago

@cristian-ungureanu

after that, deactivate Neve PRO. -> The Author Biography, Related Posts and Sharing Icons still remain in the list, in this form 12

cristian-ungureanu commented 4 years ago

@rodica-andronache fixed this as well

rodica-andronache commented 4 years ago

couldn't find other issues 👍 I just had a suggestion if you think it makes sense. To not display all the options from the Neve sidebar, when a post or page has the Elementor Full Width or Canvas templates, as they don't actually do anything. I find this confusing after I import a starter site, which has these types of templates, and I go over to edit the pages/posts.

rodica-andronache commented 4 years ago

@cristian-ungureanu I'm still testing this. So far, saw these two issues

rodica-andronache commented 4 years ago

For this one, I'm not sure if it's correct or not https://www.loom.com/share/32b467bb30ea466cb78ab33cf7b214bf By default, when creating a page, the Container is set to Default and Custom Content Width is 100%. But the way the content looks in the editor is different than what it looks like on the frontend. Only if you set Container to Full Width, the editor and frontend looks the same

cristian-ungureanu commented 4 years ago

For this one, I'm not sure if it's correct or not

https://www.loom.com/share/32b467bb30ea466cb78ab33cf7b214bf By default, when creating a page, the Container is set to Default and Custom Content Width is 100%. But the way the content looks in the editor is different than what it looks like on the frontend. Only if you set Container to Full Width, the editor and frontend looks the same

About this, it's the same logic that we apply on the frontend. The issue is that the editor is not wide enough when contained and set to 100%. Here's a video that explains it https://www.loom.com/share/25238939c8404b9f8fbfa3dcf45caf98

rodica-andronache commented 4 years ago

@cristian-ungureanu for Pages, by default the Sidebar is set to None. After resetting the options, the Sidebar is set to Inherit ( which has a sidebar by default ) https://www.loom.com/share/0dc7f1565b654eb78ce86e756b32304c

rodica-andronache commented 4 years ago

@cristian-ungureanu the default order for Elements in Posts is not the same as on frontend. In the editor, the order is : Post Title, Featured Image, Tags, Post meta, Content, Comments

1

while on frontend, the order is Post Title, Post meta, Featured Image, Content, Tags, Comments

2
cristian-ungureanu commented 4 years ago

@cristian-ungureanu for Pages, by default the Sidebar is set to None. After resetting the options, the Sidebar is set to Inherit ( which has a sidebar by default )

https://www.loom.com/share/0dc7f1565b654eb78ce86e756b32304c

@rodica-andronache for new pages we set the meta to sidebar none and content width 100% when the page is published. Those are not the defaults, the defaults are the settings from the customizer. The reset button should "erase" the meta. It shouldn't get back to "none", it should go back to customizer settings which are applying if there is no meta.

pirate-bot commented 4 years ago

:tada: This issue has been resolved in version 2.8.0 :tada:

The release is available on GitHub release

Your semantic-release bot :package::rocket: