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

Enable all wpcom themes to opt-in to appearanceTools #6411

Closed
pbking opened this issue Aug 17, 2022 · 11 comments
Closed

Enable all wpcom themes to opt-in to appearanceTools #6411

pbking opened this issue Aug 17, 2022 · 11 comments

Comments

@pbking
Copy link
Contributor

pbking commented Aug 17, 2022

Does a new add_theme_support need to be created to support this?

Can this be accomplished with a mu-plugin?

Can we limit this to Varia/Seedlet?

It’s not great that themes as recent as TT1 lag behind features

@pbking
Copy link
Contributor Author

pbking commented Aug 17, 2022

I started working on this by allowing appearanceTools to be enabled via a theme support. With that in place we could then probably simply add_theme_support( 'appearance-tools' ); in a mu-plugin or perhaps more carefully per-theme (adding it to Varia would cover a lot of ground).

@mikachan
Copy link
Member

Thank you for working on this!

With that in place we could then probably simply add_theme_support( 'appearance-tools' ); in a mu-plugin

This is also what I was thinking (but doesn't mean it's the best way..) Looking at adding it to specific themes sounds like a good plan too, especially for submitting to dotorg.

@jeffikus
Copy link
Contributor

Thanks for taking the initiative on this.

Could we not do a blanket mu-plugin approach for dotcom and then opt out on themes that report issues? We could still add the support for themes we ship to dotorg as well.

@pbking
Copy link
Contributor Author

pbking commented Aug 18, 2022

Yea. My 'druthers would be to start with Varia themes and if there are no side effects ship the mu-plugin to update all the rest. This way at least those Varia classic themes would be a public example of how to take advantage of the feature and give us a little time between "some known smallish number of themes" and "all the themes".

@simison
Copy link
Member

simison commented Sep 1, 2022

Just asked the same about Lettre #6492 (comment)

And same could be done for Lynx (link-in-bio) theme, cc @alaczek

@pbking
Copy link
Contributor Author

pbking commented Sep 1, 2022

I suggest holding off a couple of days @simison. I just deployed the change to provide this feature to all of the Varia themes and don't expect any issues. I'm working on a separate mu-plugins solution that we should probably deploy early next week which would cover any remaining themes (both classic and block).

@pbking
Copy link
Contributor Author

pbking commented Sep 1, 2022

The mu-plugins implementation is here: D87048-code

@pbking
Copy link
Contributor Author

pbking commented Sep 7, 2022

The above diff has been abandoned. A better solution is to enable this either in wpcomsh or ETK would be better than the previous change as it only encompasses simple sites.

However, determining if it should be enabled is still ongoing.

@pbking pbking closed this as completed Nov 10, 2022
@simison
Copy link
Member

simison commented Nov 10, 2022

However, determining if it should be enabled is still ongoing.

Was it enabled and what was the technique at the end?

@pbking
Copy link
Contributor Author

pbking commented Nov 14, 2022

There were issues with some sites when this was enabled at a smaller scale (only Varia themes). We weren't able to fully resolve those and the best scenario seems to be enabling "appearanceTools" on a per-theme basis for now. Which is a different (and large) beast itself.

@apeatling
Copy link
Member

@pbking Do you have any links to some of the issues you ran into when enabling this on Varia themes? Interested what you saw as we're going to see what we can do to help.

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

No branches or pull requests

5 participants