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

See All People links to wrong category #430

Open
eidolonnight opened this issue Oct 16, 2024 · 4 comments
Open

See All People links to wrong category #430

eidolonnight opened this issue Oct 16, 2024 · 4 comments
Labels
[Component] Content Issue in the site content, posts, pages, etc [Component] Theme Front-end help needed

Comments

@eidolonnight
Copy link

The "See All People" link in WordPress/news currently links to the Community category, which is a broad category for community content. This link should instead link to the Interviews category that houses all People of WordPress interviews. The Interviews category should also use the layout of the Community category.

Image

Image

@eidolonnight eidolonnight added [Component] Content Issue in the site content, posts, pages, etc [Component] Theme Front-end help needed labels Oct 16, 2024
@ryelle ryelle removed this from WordPress.org Oct 16, 2024
@ryelle
Copy link
Contributor

ryelle commented Oct 16, 2024

@eidolonnight Just for clarity — this is the expected behavior: "See all people" should link to Community (#266) and the "Community" archive (including non-people posts) is meant to be styled like this (#148).

@eidolonnight
Copy link
Author

Sorry for the hasty issue @ryelle . Here is the expected behavior:

As an aside, I assume the Interviews category is what is used to show the featured images in the "People of WordPress" section on the main news page? The extraneous community posts don't appear there, only faces.

@ryelle
Copy link
Contributor

ryelle commented Oct 17, 2024

Okay, I just wanted to clarify that you want to change the existing behavior to your new proposed outcome, not that something broke and there's a bug.

Interviews category should inherit the same style as the Community category.

Can "Community" keep the current style too, or do they need to swap styles?

I assume the Interviews category is what is used to show the featured images in the "People of WordPress" section on the main news page?

No, this uses the "People of WordPress" tag (tag archives are not used on the site, so it's not styled).

For prioritization, do you need this change by a certain date?

@eidolonnight
Copy link
Author

I just wanted to clarify that you want to change the existing behavior to your new proposed outcome, not that something broke and there's a bug.

Correct. This is not a technical bug, though clicking a "See All People" link and not seeing people breaks the user experience.

Can "Community" keep the current style too, or do they need to swap styles?

Yes, "Community" can keep the current style too. I see no reason to change it.

No, this uses the "People of WordPress" tag (tag archives are not used on the site, so it's not styled).

I see. Thanks for the clarification.

For prioritization, do you need this change by a certain date?

We do sometimes share these stories as we approach State of the Word, so before then would be great. That said, aside from traffic sent here, these stories are not the most popular. I'm unsure of the technical lift, so the prioritization is ultimately up to you.

@ryelle ryelle moved this to 📋 To do in WordPress.org Oct 23, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
[Component] Content Issue in the site content, posts, pages, etc [Component] Theme Front-end help needed
Projects
Status: 📋 To do
Development

No branches or pull requests

2 participants