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

Update page.php for a better search results list #1451

Closed
wants to merge 2 commits into from
Closed

Update page.php for a better search results list #1451

wants to merge 2 commits into from

Conversation

leopuleo
Copy link
Contributor

@leopuleo leopuleo commented May 3, 2015

Thanks to
<?php get_template_part('templates/content', get_post_type() != 'post' ? get_post_type() : get_post_format()); ?>
it's amazing to manage different post type archive / taxonomy term page without crazy files duplication.

Unfortunately, this breaks search results list because content-page.php is in templates folder.

screenshot

I propose to remove content-page.php and move the code in page.php.
I've noticed this after PR #1431

@leopuleo leopuleo changed the title Update page.php for a search results list Update page.php for a better search results list May 3, 2015
@retlehs
Copy link
Member

retlehs commented May 7, 2015

i'm not sure what the best solution is for this, but i don't really like the idea of removing content-page.php

i also don't like the idea of creating search.php + content-search.php files, or adding is_search conditionals anywhere

😩

@leopuleo
Copy link
Contributor Author

leopuleo commented May 7, 2015

I agree with you @retlehs, I chose the most drastic solution...

What about a simple file re-naming:
content-page.php -> page-content.php

It's similar to the actual situation, but the name can confuse.
By the other hand, it's coherent with page-header.php

It's hard...any solution can be right or wrong 😃

@retlehs retlehs mentioned this pull request May 7, 2015
@retlehs
Copy link
Member

retlehs commented May 7, 2015

after sleeping on this and chatting with @Foxaii, this is what we're gonna roll with: #1459

thanks a lot for bringing this up!

@retlehs retlehs closed this May 7, 2015
@leopuleo
Copy link
Contributor Author

leopuleo commented May 7, 2015

Very well, sleeping always brings a solution 😉

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

Successfully merging this pull request may close these issues.

2 participants