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

Single post layout cover not working for CPT #4062

Open
rodica-andronache opened this issue Aug 4, 2023 · 12 comments · May be fixed by #4295
Open

Single post layout cover not working for CPT #4062

rodica-andronache opened this issue Aug 4, 2023 · 12 comments · May be fixed by #4295
Labels
bug This label could be used to identify issues that are caused by a defect in the product. on-hold Not being actively worked by being blocked or not having the capacity to work on it.

Comments

@rodica-andronache
Copy link
Contributor

Description

The Cover option that we have in Customize > Layout > Single Post > Header Layout is also applying for any CPTs created, but is not working well in that case ( the title and meta are not visible )

Step-by-step reproduction instructions

  1. Create a CPT using Custom Post Type UI and check it on frontend https://vertis.d.pr/i/2PaKtt
  2. Go to Customize > Layout > Single Post > Header Layout and change it to Cover
  3. Check the CPT again -> the title is not visible anymore https://vertis.d.pr/i/ikjtME

In the video you can see the difference between how the Cover option is working for Posts vs CPT https://vertis.d.pr/v/yrpnu1

Screenshots, screen recording, code snippet or Help Scout ticket

No response

Environment info

No response

Is the issue you are reporting a regression

No

@rodica-andronache rodica-andronache added the bug This label could be used to identify issues that are caused by a defect in the product. label Aug 4, 2023
@irinelenache
Copy link
Contributor

@girishpanchal30
Copy link
Contributor

@rodica-andronache Customizer Layout works for CPT if you have installed the neve-pro-addon plugin.

Without Pro: https://tinyurl.com/25vo97vp
With Pro: https://tinyurl.com/2cr92msw

In free it will work with posts and pages only.

@vytisbulkevicius Could you please ask the docs team to add a note to the Neve document?

Thanks

@vytisbulkevicius
Copy link
Contributor

@girishpanchal30, to me it still looks more like a bug as we change how custom post type looks like when Cover is selected and there is no way to keep the default option for them to keep the title visible.

So my understanding if we want to keep this as a PRO feature would be to keep the default option for the CPT so title can be visible and it's not affected by the changes OR to make it work for custom post types as well based on the available selection in customizer.

@selul, what do you think?

@selul
Copy link
Contributor

selul commented Aug 28, 2024

I think the cpts should be not affected by this since the setting and use the default is for post post_type

@vytisbulkevicius
Copy link
Contributor

Yes, so @girishpanchal30 the goal here is to make sure that those changes in customizer don't make affect to CPTs but when working on this we need to be careful that we don't break the functionality of Neve PRO.

@girishpanchal30 girishpanchal30 linked a pull request Aug 29, 2024 that will close this issue
6 tasks
@girishpanchal30 girishpanchal30 linked a pull request Aug 29, 2024 that will close this issue
6 tasks
@girishpanchal30
Copy link
Contributor

@vytisbulkevicius I've enabled the use of the cover layout for CPT without affecting the features of Neve PRO.
Please review this PR and let me know if you find any issues.

Thanks

@rodica-andronache
Copy link
Contributor Author

@girishpanchal30 now it works the same with or without PRO. The Normal/Cover layouts apply to CPTs in the same way as it does for posts. @vytisbulkevicius is this the expected behaviour? From your latest message I understood that this should work just in PRO.

@girishpanchal30
Copy link
Contributor

@rodica-andronache, After reviewing the features and reconsidering, the cover layouts feature should work only when the pro addon is activated.

I've made some modifications with the latest commit, please recheck with the build zip and let me know if you find any issues.

Thanks

@rodica-andronache
Copy link
Contributor Author

rodica-andronache commented Sep 4, 2024

@girishpanchal30

@girishpanchal30
Copy link
Contributor

@rodica-andronache I have resolved both issues in the latest commit. Please review it and let me know if you encounter any problems.

@rodica-andronache
Copy link
Contributor Author

@girishpanchal30 now those two issues are fixed, but it's something else.
When using Neve and Neve Pro, following the steps in the video (possibly in other cases as well), at some point, the title of the norma post disappears:

https://vertis.d.pr/v/GYKpob

  1. create a post
  2. change the Header Layout from Customizer to Cover -> everything is ok
  3. change the Title alignment from the single post options -> everything is ok
  4. change the Header Layout from Customizer to Normal -> the post title is gone

@selul
Copy link
Contributor

selul commented Sep 5, 2024

@vytisbulkevicius I suggest putting this on hold, is getting too complicated and too much effort is spent for something that is not that critical right now.

@vytisbulkevicius vytisbulkevicius added the on-hold Not being actively worked by being blocked or not having the capacity to work on it. label Sep 5, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug This label could be used to identify issues that are caused by a defect in the product. on-hold Not being actively worked by being blocked or not having the capacity to work on it.
Projects
None yet
Development

Successfully merging a pull request may close this issue.

5 participants