Twenty Twenty: Remove Non-printable characters from starter-content.php file

[fahimmurshed]

I have check this theme with "Theme Check" plugin and it's show this:

INFO: Non-printable characters were found in the starter-content.php file. You may want to check this file for errors.

/inc/starter-content.php:129

'<h2 class="has-accent-color has-text-align-center">' . __( '”Cyborgs, as the philosopher Donna Haraway established, are not reverent. They do not remember the cosmos.”', 'twentytwenty' ) . '</h2>',

Removed the ” and ”

[fahimmurshed]

Patch

[ocean90]

Instead of removing the quotes they should be replaced with HTML entities.

[fahimmurshed]

Thanks, Dominik

[SergeyBiryukov]

In 47388:

Twenty Twenty: Replace quotation marks in starter content with HTML entities.

Props fahimmurshed.
Fixes #49534.

[ocean90]

@SergeyBiryukov I'd prefer if you could give others a chance to create a patch first. There was nothing important here that requires an immediate commit for the 5.4 branch.

[SergeyBiryukov]

Replying to ocean90:

I'd prefer if you could give others a chance to create a patch first. There was nothing important here that requires an immediate commit for the 5.4 branch.

This is one of the author's 12 tickets, most of which were closed for various reasons without a commit.

Getting people to contribute to core is hard as is, I'm too familiar with contributors (new and old) giving up if their efforts are not recognized.

This seemed like a good opportunity to provide some encouragement.

[fahimmurshed]

Love your appreciations @SergeyBiryukov. Thanks so much.
To be honest, It's encouraging me to contribute here more and more.
I just love this community :)