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

Footer credits changed via Customizer "Site Identity" options will not update #35968

Closed
mrfoxtalbot opened this issue Sep 4, 2019 · 13 comments
Closed
Assignees
Labels
Customer Report Issues or PRs that were reported via Happiness. Previously known as "Happiness Request". [Feature] Customizer The site customizer for traditional, non-block themes. footer credits

Comments

@mrfoxtalbot
Copy link

Steps to replicate

  1. Go to My Site > Design > Customize https://wordpress.com/customize
  2. Select "Site Identity" and change the Footer credit options:

I tested this with Modern Business and Twentynineteen

Result

The footer will not update

Expected

Footer credits should change accordingly

  • User report: #2322311-hc
    Reproduced here: mrfoxtestingsite.wordpress.com
@geekinthegirl
Copy link

Another instance of this here: #15531608-hc

User changed their Footer Credit option to 'Hide', with no effect.

@chiape
Copy link

chiape commented Nov 4, 2019

Hey there, we've just found a similar issue here and it only happens when changing the language to EN-gb.

Affected themes:

  • Barnsbury
  • Dalston
  • Maywood
  • Mayland

Also, instead of changing accordingly the link redirect to WordPress.org

@desnum
Copy link

desnum commented Nov 27, 2019

Hi there,

I have another case in the Redhill theme, where "Hide" doesn't work to remove the footer credit only when the language is set to EN-gb.

zen-2523410

@zdenys
Copy link
Contributor

zdenys commented Dec 19, 2019

Another case here 2572085-zen with Sophisticated Business Theme. But it seems that it affects many (if not all) themes and depends specifically on en-gb language being selected in https://wordpress.com/me/account https://wordpress.com/settings/general/

Possibly it affects any other non-English languages too as in Automattic/themes#1222

@zdenys zdenys added the [Feature] Customizer The site customizer for traditional, non-block themes. label Dec 19, 2019
@kwight
Copy link
Contributor

kwight commented Dec 19, 2019

I was hoping this might be a quick fix, but I've been unable to replicate. I tried in WP Admin and WP.com customizers, with en-gb and fr-ca languages, on both free and Premium plans. (I did notice that translations of the footer are not working in fr-ca, but I assume that's unrelated.)

Any other tips or indications for reproducing?

@zdenys
Copy link
Contributor

zdenys commented Dec 19, 2019

I realized that in my previous comment I incorrectly indicated where en-gb should be set. It's in https://wordpress.com/settings/general/ (so for the site itself) and not in https://wordpress.com/me/account.

I can reproduce with Barnsbury, Dalston, Maywood, Mayland, Shawburn, Alves, Exford, Stratford, Coutoire, Morden, Stow, Hever, Leven, Brompton, Redhill.

Also reported here Automattic/themes#1424 and here Automattic/themes#1 (additional details here pNEWy-cyw-p2)

@kwight
Copy link
Contributor

kwight commented Dec 19, 2019

The cause is a regex problem, being unable to match on languages with country codes (eg. en-GB, fr-CA). A patch is ready for review in D36982-code.

@kwight kwight self-assigned this Dec 20, 2019
@kwight
Copy link
Contributor

kwight commented Dec 20, 2019

Fixed in D36982-code.

@gwensmithx
Copy link

I'm reopening this issue because I ran into this today. Site is on the Pro Plan. Site Language is in German and even though it's set to hide the footer credit in the customizer, it still shows.

I can replicate this on my test site. I set my site language to German and Spanish and it disregarded the hidden setting in the customizer. Setting the language back to English resolves the issue and the credit is hidden.

Screen.Capture.on.2022-05-25.at.17-58-20.mp4

@gwensmithx gwensmithx reopened this May 25, 2022
@upwardmomentum84
Copy link

Ran into this here: 12976382-hc

Theme is Payton. I was able to replicate the issue on my end on a test site.

@github-actions
Copy link

github-actions bot commented Aug 9, 2022

Support References

This comment is automatically generated. Please do not edit it.

  • 2572085-zen

@mrfoxtalbot
Copy link
Author

Cross-referencing #58798, #59363, & #81611

@mmtr
Copy link
Member

mmtr commented Jul 24, 2024

This should have been fixed now by D156044-code (Simple sites) and Automattic/jetpack#38458 (Atomic sites).

@mmtr mmtr closed this as completed Jul 24, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Customer Report Issues or PRs that were reported via Happiness. Previously known as "Happiness Request". [Feature] Customizer The site customizer for traditional, non-block themes. footer credits
Projects
None yet
Development

No branches or pull requests

9 participants