Fixes #36404
Matching Jetpack PR:
Automattic/jetpack#14101
- Internal references:
- p1HpG7-84F-p2
- p6TEKc-3iM-p2
- p6TEKc-3jj-p2
- p6TEKc-3l7-p2
Changes proposed in this Pull Request
The Mobile theme was developed 8 years ago, back when responsive themes weren’t a thing. It isn’t as useful today, and can even be harmful when one enables it on their site without realizing the consequences, as outlined in Automattic/jetpack#7602.
This PR comes as a first step to deprecate the Mobile Theme feature. We’ll start by greying out Mobile Theme Settings from the Writing Settings UI when the feature is not active. Folks that still use the feature can access its settings and disable it. Folks who do not use it cannot turn it on via the UI.
In addition to those changes, we’ll display a notice on top of the settings:
- If the Mobile Theme feature is active on your site and if you run Jetpack 8.1 Alpha or newer, we’ll show you a notice inviting you to check the look of the site on mobile devices if you did not run the Mobile Theme feature on your site. We can do so via a Google Tool.
- If you run an older version of the Jetpack plugin, or if the Mobile Theme feature is already off on your site, you’ll see a generic notice announcing the deprecation.
Module active
Module inactive
Testing instructions
- Start from a site running Jetpack 8.1 Alpha.
- Run
yarn docker:wp jetpack module activate minileven
to enable the module on your Jetpack site.
- Go to Manage > Settings > Writing.
- Notice that the Mobile Theme settings are there.
- Notice the notice.
- Click the 2 links to check the support docs as well as the Google checking tool.
- Disable the feature.
- The settings should become grey.
Unfortunately, no screenshots were provided by the developer.