-
-
Notifications
You must be signed in to change notification settings - Fork 83
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
Deleting Post Prominence Doesn't Work #1862
Comments
Hi! To help us figure out how best to resolve this request, we have some questions:
|
(possibly related: #1799) |
I am on version 0.6.4 and am using a child theme. I'm trying to delete homepage featured and homepage top story. Thanks for the help! I'll peep issue 1799 to see if that has anything that might help me too. |
Are you using a custom homepage layout, or one of the ones in https://github.com/INN/largo/tree/v0.6.4/homepages/layouts ? |
I am using the "Top Stories" layout |
The "Homepage Featured" and "Homepage Top Story" prominence terms are being automatically registered by your homepage layout because they drive how that homepage layout works. https://github.com/INN/largo/blob/v0.6.4/homepages/layouts/TopStories.php#L17-L28 Those prominence terms affect how posts are displayed on the homepage; the specifics are explained at https://largo.inn.org/guides/administrators/theme-options/layout/homepage-layout-top-stories/ If you don't mind my curiosity: Why don't you use those prominence terms? |
Good deal! That information is exactly what I needed. I will probably just shift away from those and work at moving towards a custom homepage. As far as why I don't use them, it's a naming thing. I did a big no-no when I couldn't get custom my custom homepage to work and just made some small tweaks to TopStories in the parent theme. Basically all I did was remove the three featured stores from the box that contains the top story and the top story is taking up the full width of that box. I want to remove that option because the top story happens to be labeled "This Week's Feature" on the front in (per client request) and the existence of that other option is causing confusion. Anyhow, this was very helpful and I will consider my self created issue resolved. Thanks so much for your help! |
Title says most of it. When I try to delete some of the default post prominences that I am not using, they just come back! A fix would be cool, but moreover is someone could point me in the right direction of where I could:
Beyond that, if anyone has thoughts on where the bug is originating, I would be happy to take a crack at fixing it and submitting a PR.
The text was updated successfully, but these errors were encountered: