JOTAKI, Taisuke

Ah! I checked a child theme and the parent theme has a screenshot. Is this correct behavior?

Checked it out and found one issue.

When you don’t have screenshot.png in a theme, it is supposed to display the error message “No screenshot detected!”, but it doesn’t display the message.

As for the rest, I think it works fine and here is the updated Japanese translation:
https://wp.tekapo.com/tawp/wp-content/uploads/2012/02/themecheck-ja.zip

Hi,

This is the last patch and includes the first one.

https://pastie.org/3364542

I’ve made some lines gettexted, and also revised some current gettexted lines because the gettext command can’t parse the current one.

The current one example:

$this->error[] = __( "<span class='tc-lead tc-warning'>WARNING</span>: could not find the file <strong>{$file}</strong> in the theme.", "themecheck" );

revised like this:

$this->error[] = sprintf(__('<span class="tc-lead tc-warning">WARNING</span>: could not find the file <strong>%1$s</strong> in the theme.', 'themecheck'), $file);

Thank you, Simon!

Here is the first patch:
https://pastie.org/3335127

I’m still looking into it and I think we need a few more patches , so I will post here again.

I was just thinking exactly the same thing.:)
Why don’t we make a ticket on Trac?