WordPress / twentytwentythree

Twenty Twenty-Three, the default WordPress theme sporting many styles that will launch with WordPress 6.1.
123 stars 64 forks source link

Increase the minimum required WordPress version #24

Closed carolinan closed 1 year ago

carolinan commented 2 years ago

It is common for the default themes to be compatible with WordPress versions below the version the theme is first included in.

This theme's minimum required version is set to 5.9, but the theme can not be activated on 5.9.3:

The package could not be installed. The theme is missing the index.php file.
Theme installation failed.

The theme could include an index.php file without causing conflicts for 6.1. But are there other 6.1 features that are breaking on 5.9?

Updating templates to include new features like the "no results" block for the queries and the new comments blocks also depends on increasing the version.

MaggieCabrera commented 2 years ago

I don't think we followed that rule for TT2, did we @jffng ?

jffng commented 2 years ago

We did not, since TT2 was the first block theme and would not work without FSE which shipped with that release.

It is common for the default themes to be compatible with WordPress versions below the version the theme is first included in.

TT3 is heavily focused on style variations, which shipped with 6.0 I think. Could we bump the minimum required version to 6.0? Since it's quite a plain theme without style variations, I don't see a lot of value to lowering it to 5.9.

carolinan commented 2 years ago

6.0 would still mean that the theme can not use the spacing presets or the fluid typography settings.

mikachan commented 2 years ago

6.0 would still mean that the theme can not use the spacing presets or the fluid typography settings.

This is a great point, I liked the idea of this theme showing examples of using new features like fluid typography and spacing presets.

Plus if we use anything lower than 6.1, we're going to need to add more CSS.

jffng commented 2 years ago

I don't like the idea of adding CSS specifically for 6.0. I think it's okay certain aspects of the theme don't work on 6.0, as long as the lack of those features don't present accessibility issues.

This brings up a question — can TT3 get the a11y ready tag if its minimum required version is lower than 6.1? I'm thinking about this TT2 ticket.

In general, I understand wanting to be more inclusive about the share of folks that can use the theme by lowering the required version, but to me it's kind of an edge case — someone wanting to stay on 5.9 or 6.0 and use TT3. Maybe I'm missing some perspective here though.

MaggieCabrera commented 2 years ago

I think block themes are a new paradigm that shouldn't adhere to the rules default classic themes were following just because it's how it's been done. In this case, I don't think it makes sense to support the lower WP versions. It may make sense in a couple of years, when features aren't being added so frequently, but not so much today, specially when the whole point of this theme release is to showcase some of those features.

mikachan commented 2 years ago

Based on the above, should we change the minimum version to 6.1?

carolinan commented 2 years ago

I think so, but it can't be done yet because then we can't activate it =D

mikachan commented 2 years ago

Ohh ha right! 😅