-
Notifications
You must be signed in to change notification settings - Fork 2k
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
Theme showcase: selecting feature:block-themes only displays one theme #96209
Comments
Flaggin this for @Automattic/t-rex @Automattic/lego as well. |
The teams above. ^^ Looks like Dean has grabbed it. |
I've added to the Lego board as well in case we have capacity to pick this up in the upcoming weeks if T-Rex hasn't already addressed this. |
FWIW, it returns one theme because only one theme is tagged This is not a development issue, as much as a categorization one. I don't know what the most efficient way to do this would be, but it's really a matter of going into the source site and updating the Theme Feature taxonomy on all the relevant themes. |
Digging into this further, it looks like we need to do a few things:
If we remove the @richtabor curious what you think? Also curious to hear from theme designers if there's something I'm missing here since you all handle the launches @iamtakashi @henriqueiamarino @beafialho ❤ |
+1. And ensure that the full-site-editing tag is removed from theme's wp-admin. Worth noting that the |
The
The only thing that could be a blocker is that our HEs are used to giving |
Would a Calypso re-routing work? Another approach is to have |
That would be my only (minor) concern too. I'd say it's not a problem as long as we communicate the new URL to HEs. 🤔 |
How many customers already have the current URL and occasionally use it? How long will it take all HEs to update their predefs and get used to the change? If we can keep the old one working then lets do that. Cool URIs don't change. |
@dsas We could debate whether it's a cool URL. 😄 I'm not opposed to create a routing exception, it should be fairly trivial. I'm mostly questioning the usefulness of the URL altogether. |
One thought is because updating HE predefs equates to many, many hours of work combined. Another thought is because breaking URLs is bad for everyone. |
Generally speaking, for things that are trivial, and are unlikely to have a huge bite then I'd rather prioritise the HE + Customer experience over the developer experience. <40 daily uniques in the last month is a gnats bite I guess, it might lead to a trivial number of tickets. Communicating to hundreds of HEs to check and update their pre-defs sounds time consuming and error prone. We'd also have to check and update our support docs etc too. |
Right, since @tanjoymor has a strong opinion on this, it totally beats my lukewarm contrarianism. I also reserve my right to, one day, say to everybody that "I told you so". 😄 @dsas, @taipeicoder: Whoever grabs this issue, we have several alternative approaches that we can take here:
|
I’m no expert on the technical side of this, but to me option 3 of redirecting feels like the better long-term option. (And may prevent @Copons from getting an I told you so moment 😂) Curious though if search terms can also create a redirect environment? Like if someone searches for “full site editing” can it show the “block-theme” results? It is a worthy exercise to update predefs and docs over time, it just doesn’t feel like the highest and best use of immediate time in terms of priority. Redirects allow more time for those updates to occur, and prevents broken URLs. Also noting that ~40 daily uniques potentially equals 14K+ users per year. With discussions around every customer matters, this doesn’t feel insignificant in terms of user experience imo. 🤷♀️ |
I think for that it would be enough to not delete the FSE tag. Maybe there are checks to prevent showing empty tags, but it should solvable via code. 🤔 |
@annezazu I'd like to make these tags as consistent to the Core theme library as possible, so I have some questions! When we talk about "block themes", are we specifically talking about themes with a For backward compatibility, Dotorg uses the |
Here's what Core did for context: https://meta.trac.wordpress.org/ticket/7524 While the full site editing tag is still there, everything in the interface shows "block theme"! That's what I want.
Core shows the "Block Theme" feature filter and I think we should too. Since it seems like my screenshot didn't help, here's a quick video to help explain how Core has it set up: Core.approach.movWe should align with Core here to make it easier for launching themes but we also can and should offer a better experience! Unlike in the Theme Directory, we have full control here to make changes. Part of this includes searching for block themes. When that happens, I think we should automatically just set the feature to "block theme" and display all. |
No work has been done on this unfortunately despite being on two team's list. As I stated above, we should be showing all block themes when searching for that feature.
I agree! @fditrapani has it on his list to loop back on our overall features/tags and perhaps this is something the broader designers coming from our division could help with too? |
Adding this to the design board but I believe the larger task is outlined in this issue: #98287 @crisbusquets sorry to potentially create more noise but I've added them both to the design board for you all to sort through! Do what you think is best in terms of keeping just one or both. |
i'm flagging this as In Progress on the One Board for now since it's actively being worked on. Feel free to ping us if you need PAs/PEAs input. |
I am going to manually fix this. We have 280 themes with the Full Site Editing feature tag and only 4 with the Block theme feature tag. I'm going to have any theme marked as Full Site Editing also marked as Block theme. The result will be that themes will show up in both places and (we could) in theory deprecate the Full Site Editing feature tag. Either way, it greatly lowers the urgency. I'll update when I'm done. |
Tag teamed with @taipeicoder who kindly jumped in! I'm going to update the block theme feature description and close this out. For future reference, if I can ever fix something manually, let me know. I will always happily dive in. |
Nice work Anne! |
I'm setting up a new site to build out a project for someone and tried to filter by
feature:block-themes
only to find one theme displayed (and a partner one at that! I'm on a premium plan!):@ianstewart which team is best suited to address this asap?
The text was updated successfully, but these errors were encountered: