WordPress.org

Make WordPress Themes

Opened 3 months ago

Closed 4 weeks ago

Last modified 4 weeks ago

#46717 closed theme (live)

THEME: TIJAJI – 1.24

Reported by: tijaji Owned by: Mirucon
Priority: previously reviewed Keywords: theme-tijaji
Cc: yossy22@…

Description

TIJAJI - 1.03

日本人が作ったスマホ対応のシンプルなレスポンシブテンプレート。Simple responsive template made by Japanese. Suitable for smartphone.

Theme URL - http://tijaji.jp/
Author URL - http://www.tjerji.com

Trac Browser - https://themes.trac.wordpress.org/browser/tijaji/1.03

SVN - https://themes.svn.wordpress.org/tijaji/1.03
ZIP - https://wordpress.org/themes/download/tijaji.1.03.zip?nostats=1

Diff with previous version: https://themes.trac.wordpress.org/changeset?old_path=tijaji/1.02&new_path=tijaji/1.03

History:

Ticket Summary Status Resolution Owner
#46669 THEME: TIJAJI – 1.0 closed not-approved acosmin
#46672 THEME: TIJAJI – 1.01 closed not-approved acosmin
#46682 THEME: TIJAJI – 1.02 closed not-approved rabmalin
#46717 THEME: TIJAJI – 1.24 closed live Mirucon

(this ticket)

#49344 THEME: TIJAJI – 1.25 closed live themetracbot
#49350 THEME: TIJAJI – 1.26 closed live themetracbot


https://themes.svn.wordpress.org/tijaji/1.03/screenshot.png

Change History (56)

#1 @themetracbot
3 months ago

  • Summary changed from THEME: TIJAJI – 1.03 to THEME: TIJAJI – 1.04

TIJAJI - 1.04

日本人が作ったスマホ対応のシンプルなレスポンシブテンプレート。テーマカスタマイザーにてカラム数や文字サイズ、フォントなどカスタムが可能。

Theme URL - http://tijaji.jp/
Author URL - http://www.tjerji.com

Trac Browser - https://themes.trac.wordpress.org/browser/tijaji/1.04

SVN - https://themes.svn.wordpress.org/tijaji/1.04
ZIP - https://wordpress.org/themes/download/tijaji.1.04.zip?nostats=1

Diff with previous version: https://themes.trac.wordpress.org/changeset?old_path=tijaji/1.03&new_path=tijaji/1.04

History:

Ticket Summary Status Resolution Owner
#46669 THEME: TIJAJI – 1.0 closed not-approved acosmin
#46672 THEME: TIJAJI – 1.01 closed not-approved acosmin
#46682 THEME: TIJAJI – 1.02 closed not-approved rabmalin
#46717 THEME: TIJAJI – 1.24 closed live Mirucon

(this ticket)

#49344 THEME: TIJAJI – 1.25 closed live themetracbot
#49350 THEME: TIJAJI – 1.26 closed live themetracbot


https://themes.svn.wordpress.org/tijaji/1.04/screenshot.png

#2 @poena
3 months ago

  • Owner set to poena
  • Status changed from new to reviewing

This ticket was mentioned in Slack in #themereview by poena. View the logs.


3 months ago

#4 @poena
3 months ago

  • Owner changed from poena to mirucon

#5 @poena
3 months ago

  • Owner changed from mirucon to Mirucon

#6 @Mirucon
3 months ago

@tijaji さん、こんにちは。

普段テーマレビューは英語で行われるものですが、今回は特例として日本語にて私がこちらのレビューを担当させていただきます。返信や質問等はすべて日本語で大丈夫です。

レビューを始める前に一つだけお聞きしたいのですが、こちらにあるレビューガイドラインについてご存知でしょうか?https://make.wordpress.org/themes/handbook/review/required/

テーマを公式レポジトリに登録する場合には、こちらのレビューガイドラインに沿ったテーマであることが必須条件になっています。このテーマのチケットはこれまでに三回ほどガイドラインに沿っていないことを理由にクローズされています。必ずこちらのガイドラインをお読み頂き、それに沿ったテーマを作ってくださるようお願いします。

英語力に自信がないようであれば、私と同じく日本人のテーマレビュアーである西川さんの書かれたこちらの記事が役に立ちます。少しだけ情報が古いので、原文であるガイドラインも併せてお読みいただきたいのですが、理解する際には助けになるのではないかと思います。

https://nskw-style.com/videos/theme-repository

質問等はこちらのチケット内にて回答させていただきたく思いますので、何かありましたらお気軽に質問してください。

#7 @tijaji
3 months ago

ありがとうございます。
日本語の返信を見てすごく安心しました。
https://nskw-style.com/videos/theme-repository
の動画を少し見てみましたが日本語でとても分かりやすそうです。
教えてくれてありがとうございます。
少しですが動画を見た感じだとまず変数にユニークな接頭語がつけられていないのでここを修正します。
他にも動画を見ながら修正していきます。
修正が終わったらまたアップロードしますのでよろしくお願いいたします。

#8 @ShinichiN
3 months ago

@tijaji 動画の撮影時期と同じ頃に翻訳された必須条件項目がここにあります。古いんですけど参考になれば。
レポジトリ毎ダウンロードするといいかもです。

https://github.com/WPTRT/contribution-handbook/blob/master/version-contributionday-ja/stepseven.html

#9 @tijaji
3 months ago

動画で説明されていたものですね。
ありがとうございます。
英語あまりわからないので助かります。

#10 @themetracbot
3 months ago

  • Summary changed from THEME: TIJAJI – 1.04 to THEME: TIJAJI – 1.10

TIJAJI - 1.10

日本人が作ったスマホ対応のシンプルなレスポンシブテンプレート。テーマカスタマイザーにてカラム数や文字サイズ、フォントなどカスタムが可能。Simple responsive template made by Japanese. Suitable for smartphone. With theme customizer, you can customize the number of columns, font size, font and so on.

Theme URL - http://tijaji.jp/
Author URL - http://www.tjerji.com

Trac Browser - https://themes.trac.wordpress.org/browser/tijaji/1.10

SVN - https://themes.svn.wordpress.org/tijaji/1.10
ZIP - https://wordpress.org/themes/download/tijaji.1.10.zip?nostats=1

Diff with previous version: https://themes.trac.wordpress.org/changeset?old_path=tijaji/1.04&new_path=tijaji/1.10

History:

Ticket Summary Status Resolution Owner
#46669 THEME: TIJAJI – 1.0 closed not-approved acosmin
#46672 THEME: TIJAJI – 1.01 closed not-approved acosmin
#46682 THEME: TIJAJI – 1.02 closed not-approved rabmalin
#46717 THEME: TIJAJI – 1.24 closed live Mirucon

(this ticket)

#49344 THEME: TIJAJI – 1.25 closed live themetracbot
#49350 THEME: TIJAJI – 1.26 closed live themetracbot


https://themes.svn.wordpress.org/tijaji/1.10/screenshot.png

#11 @tijaji
3 months ago

ガイドライン通り沿って修正しました。
まだワードプレス勉強中なので至らないところがあるかと思いますがよろしくお願いいたします。

#12 @themetracbot
3 months ago

  • Summary changed from THEME: TIJAJI – 1.10 to THEME: TIJAJI – 1.11

TIJAJI - 1.11

日本人が作ったスマホ対応のシンプルなレスポンシブテンプレート。テーマカスタマイザーにてカラム数や文字サイズ、フォントなどカスタムが可能。Simple responsive template made by Japanese. Suitable for smartphone. With theme customizer, you can customize the number of columns, font size, font and so on.

Theme URL - http://tijaji.jp/
Author URL - http://www.tjerji.com

Trac Browser - https://themes.trac.wordpress.org/browser/tijaji/1.11

SVN - https://themes.svn.wordpress.org/tijaji/1.11
ZIP - https://wordpress.org/themes/download/tijaji.1.11.zip?nostats=1

Diff with previous version: https://themes.trac.wordpress.org/changeset?old_path=tijaji/1.10&new_path=tijaji/1.11

History:

Ticket Summary Status Resolution Owner
#46669 THEME: TIJAJI – 1.0 closed not-approved acosmin
#46672 THEME: TIJAJI – 1.01 closed not-approved acosmin
#46682 THEME: TIJAJI – 1.02 closed not-approved rabmalin
#46717 THEME: TIJAJI – 1.24 closed live Mirucon

(this ticket)

#49344 THEME: TIJAJI – 1.25 closed live themetracbot
#49350 THEME: TIJAJI – 1.26 closed live themetracbot


https://themes.svn.wordpress.org/tijaji/1.11/screenshot.png

#13 @tijaji
3 months ago

function.phpで

load_theme_textdomain( 'tijaji', get_template_directory() . '/languages' );

のようにそのまま書いている個所があったので

function tijaji_text_domain(){

load_theme_textdomain( 'tijaji', get_template_directory() . '/languages' );

}
add_action('after_setup_theme', 'tijaji_text_domain');

のように修正したのが5か所くらいあります。

それとリストデザインの修正でCSSを少し変更しました。

#14 follow-ups: @Mirucon
3 months ago

@tijaji さん、こんにちは。

本来であればこれほど要求事項を満たしていないテーマの場合チケットはクローズされまた別の審査を待つことになります。今回は特別にレビューを続けますが、もう一度ガイドラインを読み、(また日本語訳も作りました) きちんとテーマの作りがそれに沿っているかを確認してください。また7日以内にアップデートを提供してくださるようお願いします。

日本語訳リンク: https://mirucon.github.io/required-ja/

Required:

  • get_userdatabylogin() has been deprecated. get_userdatabylogin() 関数は廃止されています。代わりに get_user_by() を使用してください。File: author.php, line: 4.
  • the_title() should not have post id as parameter in the loop. ループ内では the_title() 関数の引数に投稿 ID を指定しないでください - esc_html( get_the_title($comment->comment_post_ID) ) のようなコードは、 the_title() に置き換えられます。また get_comment_link() 関数も同様です。
  • Use the_excerpt to get post excerpt instead of your own function. 投稿の抜粋を取ってくるのには the_excerpt() 関数を使ってください。また excerpt_length フックを使うと文字数も変更できます。
  • Do not get data from database directly. 直接データベースからデータを取ってくるのは避けてください。例えば author.php の66行目の実装部分は get_comments() 関数を使えます。
  • All of the theme strings should be translatable. すべてのテーマ内の文字列を翻訳可能な状態にしてください - footer.php, line: 18.
  • Provide a unique prefix for everything the Theme defines. テーマ内のすべての定義にプレフィックスを提供してください。function.php, line: 776 - 782 のテーマのスタイル / スクリプトのハンドル名にプレフィックスがありません。
  • Use esc_attr() to escape HTML attributes, not esc_html(). HTML の属性内でエスケープする際は、esc_html ではなく esc_attr を使用してください。エスケープに関する記事は日本語版 Codex にある他、私も以前記事にしました。

https://wpdocs.osdn.jp/%E3%83%87%E3%83%BC%E3%82%BF%E6%A4%9C%E8%A8%BC

https://www.mirucon.com/2017/07/11/the-complex-wordpress-escaping/

Recommended:

  • author.php line: 19, esc_html( __('Recent Posts by %s','tijaji') ) の 代わりに esc_html__() 関数を使用することも出来ます。
  • Consider to use the_posts_pagination() instead of paginate_links(). paginate_links() 関数の代わりに the_posts_pagination() 関数を使うことを考慮してください。

#15 @tijaji
3 months ago

ご連絡ありがとうございます。
レビューしてくださりありがとうございます。
また、日本語訳まで用意していただきなんとお礼を言ったらよいか分かりません。
本当にありがとうございます。
修正して、またご連絡させていただきます。
お手数をおかけして恐縮ですがよろしくお願いいたします。

#16 @themetracbot
3 months ago

  • Summary changed from THEME: TIJAJI – 1.11 to THEME: TIJAJI – 1.12

TIJAJI - 1.12

日本人が作ったスマホ対応のシンプルなレスポンシブテンプレート。テーマカスタマイザーにてカラム数や文字サイズ、フォントなどカスタムが可能。Simple responsive template made by Japanese. Suitable for smartphone. With theme customizer, you can customize the number of columns, font size, font and so on.

Theme URL - http://tijaji.jp/
Author URL - http://www.tjerji.com

Trac Browser - https://themes.trac.wordpress.org/browser/tijaji/1.12

SVN - https://themes.svn.wordpress.org/tijaji/1.12
ZIP - https://wordpress.org/themes/download/tijaji.1.12.zip?nostats=1

Diff with previous version: https://themes.trac.wordpress.org/changeset?old_path=tijaji/1.11&new_path=tijaji/1.12

History:

Ticket Summary Status Resolution Owner
#46669 THEME: TIJAJI – 1.0 closed not-approved acosmin
#46672 THEME: TIJAJI – 1.01 closed not-approved acosmin
#46682 THEME: TIJAJI – 1.02 closed not-approved rabmalin
#46717 THEME: TIJAJI – 1.24 closed live Mirucon

(this ticket)

#49344 THEME: TIJAJI – 1.25 closed live themetracbot
#49350 THEME: TIJAJI – 1.26 closed live themetracbot


https://themes.svn.wordpress.org/tijaji/1.12/screenshot.png

#17 in reply to: ↑ 14 @tijaji
3 months ago

  • get_userdatabylogin() has been deprecated. get_userdatabylogin() 関数は廃止されています。代わりに get_user_by() を使用してください。File: author.php, line: 4.
  • the_title() should not have post id as parameter in the loop. ループ内では the_title() 関数の引数に投稿 ID を指定しないでください - esc_html( get_the_title($comment->comment_post_ID) ) のようなコードは、 the_title() に置き換えられます。また get_comment_link() 関数も同様です。
  • Use the_excerpt to get post excerpt instead of your own function. 投稿の抜粋を取ってくるのには the_excerpt() 関数を使ってください。また excerpt_length フックを使うと文字数も変更できます。
  • Do not get data from database directly. 直接データベースからデータを取ってくるのは避けてください。例えば author.php の66行目の実装部分は get_comments() 関数を使えます。
  • All of the theme strings should be translatable. すべてのテーマ内の文字列を翻訳可能な状態にしてください - footer.php, line: 18.
  • Provide a unique prefix for everything the Theme defines. テーマ内のすべての定義にプレフィックスを提供してください。function.php, line: 776 - 782 のテーマのスタイル / スクリプトのハンドル名にプレフィックスがありません。

TIJAJI - 1.12 にて修正しました。

#18 @themetracbot
3 months ago

  • Summary changed from THEME: TIJAJI – 1.12 to THEME: TIJAJI – 1.13

TIJAJI - 1.13

日本人が作ったスマホ対応のシンプルなレスポンシブテンプレート。テーマカスタマイザーにてカラム数や文字サイズ、フォントなどカスタムが可能。Simple responsive template made by Japanese. Suitable for smartphone. With theme customizer, you can customize the number of columns, font size, font and so on.

Theme URL - http://tijaji.jp/
Author URL - http://www.tjerji.com

Trac Browser - https://themes.trac.wordpress.org/browser/tijaji/1.13

SVN - https://themes.svn.wordpress.org/tijaji/1.13
ZIP - https://wordpress.org/themes/download/tijaji.1.13.zip?nostats=1

Diff with previous version: https://themes.trac.wordpress.org/changeset?old_path=tijaji/1.12&new_path=tijaji/1.13

History:

Ticket Summary Status Resolution Owner
#46669 THEME: TIJAJI – 1.0 closed not-approved acosmin
#46672 THEME: TIJAJI – 1.01 closed not-approved acosmin
#46682 THEME: TIJAJI – 1.02 closed not-approved rabmalin
#46717 THEME: TIJAJI – 1.24 closed live Mirucon

(this ticket)

#49344 THEME: TIJAJI – 1.25 closed live themetracbot
#49350 THEME: TIJAJI – 1.26 closed live themetracbot


https://themes.svn.wordpress.org/tijaji/1.13/screenshot.png

#19 in reply to: ↑ 14 @tijaji
3 months ago

  • Use esc_attr() to escape HTML attributes, not esc_html(). HTML の属性内でエスケープする際は、esc_html ではなく esc_attr を使用してください。エスケープに関する記事は日本語版 Codex にある他、私も以前記事にしました。
  • author.php line: 19, esc_html( __('Recent Posts by %s','tijaji') ) の 代わりに esc_html__() 関数を使用することも出来ます。
  • Consider to use the_posts_pagination() instead of paginate_links(). paginate_links() 関数の代わりに the_posts_pagination() 関数を使うことを考慮してください。

TIJAJI - 1.13 にて修正しました。
よろしくお願いいたします。

#20 @themetracbot
2 months ago

  • Summary changed from THEME: TIJAJI – 1.13 to THEME: TIJAJI – 1.14

TIJAJI - 1.14

日本人が作ったスマホ対応のシンプルなレスポンシブテンプレート。テーマカスタマイザーにてカラム数や文字サイズ、フォントなどカスタムが可能。Simple responsive template made by Japanese. Suitable for smartphone. With theme customizer, you can customize the number of columns, font size, font and so on.

Theme URL - http://tijaji.jp/
Author URL - http://www.tjerji.com

Trac Browser - https://themes.trac.wordpress.org/browser/tijaji/1.14

SVN - https://themes.svn.wordpress.org/tijaji/1.14
ZIP - https://wordpress.org/themes/download/tijaji.1.14.zip?nostats=1

Diff with previous version: https://themes.trac.wordpress.org/changeset?old_path=tijaji/1.13&new_path=tijaji/1.14

History:

Ticket Summary Status Resolution Owner
#46669 THEME: TIJAJI – 1.0 closed not-approved acosmin
#46672 THEME: TIJAJI – 1.01 closed not-approved acosmin
#46682 THEME: TIJAJI – 1.02 closed not-approved rabmalin
#46717 THEME: TIJAJI – 1.24 closed live Mirucon

(this ticket)

#49344 THEME: TIJAJI – 1.25 closed live themetracbot
#49350 THEME: TIJAJI – 1.26 closed live themetracbot


https://themes.svn.wordpress.org/tijaji/1.14/screenshot.png

#21 @themetracbot
2 months ago

  • Summary changed from THEME: TIJAJI – 1.14 to THEME: TIJAJI – 1.15

TIJAJI - 1.15

日本人が作ったスマホ対応のシンプルなレスポンシブテンプレート。テーマカスタマイザーにてカラム数や文字サイズ、フォントなどカスタムが可能。Simple responsive template made by Japanese. Suitable for smartphone. With theme customizer, you can customize the number of columns, font size, font and so on.

Theme URL - http://tijaji.jp/
Author URL - http://www.tjerji.com

Trac Browser - https://themes.trac.wordpress.org/browser/tijaji/1.15

SVN - https://themes.svn.wordpress.org/tijaji/1.15
ZIP - https://wordpress.org/themes/download/tijaji.1.15.zip?nostats=1

Diff with previous version: https://themes.trac.wordpress.org/changeset?old_path=tijaji/1.14&new_path=tijaji/1.15

History:

Ticket Summary Status Resolution Owner
#46669 THEME: TIJAJI – 1.0 closed not-approved acosmin
#46672 THEME: TIJAJI – 1.01 closed not-approved acosmin
#46682 THEME: TIJAJI – 1.02 closed not-approved rabmalin
#46717 THEME: TIJAJI – 1.24 closed live Mirucon

(this ticket)

#49344 THEME: TIJAJI – 1.25 closed live themetracbot
#49350 THEME: TIJAJI – 1.26 closed live themetracbot


https://themes.svn.wordpress.org/tijaji/1.15/screenshot.png

#22 @themetracbot
2 months ago

  • Summary changed from THEME: TIJAJI – 1.15 to THEME: TIJAJI – 1.16

TIJAJI - 1.16

日本人が作ったスマホ対応のシンプルなレスポンシブテンプレート。テーマカスタマイザーにてカラム数や文字サイズ、フォントなどカスタムが可能。Simple responsive template made by Japanese. Suitable for smartphone. With theme customizer, you can customize the number of columns, font size, font and so on.

Theme URL - http://tijaji.jp/
Author URL - http://www.tjerji.com

Trac Browser - https://themes.trac.wordpress.org/browser/tijaji/1.16

SVN - https://themes.svn.wordpress.org/tijaji/1.16
ZIP - https://wordpress.org/themes/download/tijaji.1.16.zip?nostats=1

Diff with previous version: https://themes.trac.wordpress.org/changeset?old_path=tijaji/1.15&new_path=tijaji/1.16

History:

Ticket Summary Status Resolution Owner
#46669 THEME: TIJAJI – 1.0 closed not-approved acosmin
#46672 THEME: TIJAJI – 1.01 closed not-approved acosmin
#46682 THEME: TIJAJI – 1.02 closed not-approved rabmalin
#46717 THEME: TIJAJI – 1.24 closed live Mirucon

(this ticket)

#49344 THEME: TIJAJI – 1.25 closed live themetracbot
#49350 THEME: TIJAJI – 1.26 closed live themetracbot


https://themes.svn.wordpress.org/tijaji/1.16/screenshot.png

#23 @tijaji
2 months ago

テーママスタマイザーの項目と簡単な説明を管理画面に追加しました。
テーマカスタマイザーに一部機能を追加しました。
1.14と1.15はテーマをアップロードした際に文字化けの警告が出たので修正して1.16をアップロードしました。
よろしくお願いいたしますm(_ _)m

#24 @themetracbot
2 months ago

  • Summary changed from THEME: TIJAJI – 1.16 to THEME: TIJAJI – 1.17

TIJAJI - 1.17

日本人が作ったスマホ対応のシンプルなレスポンシブテンプレート。テーマカスタマイザーにてカラム数や文字サイズ、フォントなどカスタムが可能。Simple responsive template made by Japanese. Suitable for smartphone. With theme customizer, you can customize the number of columns, font size, font and so on.

Theme URL - http://tijaji.jp/
Author URL - http://www.tjerji.com

Trac Browser - https://themes.trac.wordpress.org/browser/tijaji/1.17

SVN - https://themes.svn.wordpress.org/tijaji/1.17
ZIP - https://wordpress.org/themes/download/tijaji.1.17.zip?nostats=1

Diff with previous version: https://themes.trac.wordpress.org/changeset?old_path=tijaji/1.16&new_path=tijaji/1.17

History:

Ticket Summary Status Resolution Owner
#46669 THEME: TIJAJI – 1.0 closed not-approved acosmin
#46672 THEME: TIJAJI – 1.01 closed not-approved acosmin
#46682 THEME: TIJAJI – 1.02 closed not-approved rabmalin
#46717 THEME: TIJAJI – 1.24 closed live Mirucon

(this ticket)

#49344 THEME: TIJAJI – 1.25 closed live themetracbot
#49350 THEME: TIJAJI – 1.26 closed live themetracbot


https://themes.svn.wordpress.org/tijaji/1.17/screenshot.png

#25 @tijaji
2 months ago

見直していると属性の中で esc_html() を使用している個所がまだあったので esc_attr() に修正しました。

#26 @Mirucon
2 months ago

すみません、ちょっと多忙につきまだアップデートを見れていません。今週末をめどにレビューを投稿する予定ですので、しばらくお待ち下さい。

#27 @tijaji
2 months ago

私の方は後回しで全く問題ないです。
勉強不足でみなさんにご迷惑をかけているのが申し訳ないです。
週末と言わずにMiruconさんには休息を取っていただき、レビューは来週でも再来週でも本当にお手すきの時にで大丈夫です。
教えていただいた英語訳や、英語のサイトの方もGoogle翻訳を使いながら足りない知恵を絞って学んでいき、
できるだけ手間を取らせないよう努力していきます。
ご多忙の中、わざわざご連絡くださりありがとうございます。

#28 @Mirucon
2 months ago

遅くなりましたが、こちらがバージョン1.17のレビューです。確認してすべて修正してください。

Required:

  • プレフィックスのついてない関数 - functions.php にある以下の関数にプレフィックス (tijaji_) がついていません。
line 161: rss_comment_footer();
line 172: category_id_class();
line 186: has_thumb_class();
  • 翻訳可能になっていない文字列 - functions.php, line: 164 にある文字列が翻訳可能な状態になっていません。
  • エスケープが必要 - functions.php, line: 164 にある get_permalink() 関数は esc_url() 関数を使用してエスケープされている必要があります。
  • jQuery のようなコアに同梱されているスクリプトはテーマ側で別のバージョンを用意し読み込ませることは出来ません。それを読み込んでいる部分 (functions.php line:288) を削除してください。
  • HTML 内のインラインスタイル (style="" の HTML 属性) の使用は基本的に不可となっています。(特に外部に表示されるフロント部分 (つまりテンプレートファイル内) での使用 / background-image: url(<?php echo esc_url( header_background_image() ); ?> のような PHP が必要な場合は例外。) インラインスタイルではなく、CSS 内で定義してください。header.php line: 41
  • タイトルを HTML の title 属性内に表示する際には the_title() 関数でなく the_title_attribute() 関数を使用してください。file: content.php L5, 7, 29, 31, author.php L40, 88.
  • HTML 属性内では esc_attr() 関数を使用してください - searchform.php L4. (alt 内と value の2つです)
  • 空のフォルダーを削除してください - inc/add.

Recommended:

  • __() のような国際化に使用される関数内では、極力 HTML を使わないのが望ましいです。例えば inc/customizer-sidebar.php L:22 では、
__( '<div class="deco_title decotop">Font setting</div>', 'tijaji' )

ではなく、

'<div class="deco_title decotop">' . __( 'Font setting', 'tijaji' ) . '</div>'

になっている方がより翻訳者に優しいです。

  • the_excerpt() 関数から HTML を取り除く際には、esc_html() 関数よりも wp_strip_all_tags() を使用するのが好ましいです。
  • single_cat_title(), single_tag_title() 関数ではなく、the_archive_title() の使用が推奨されています。たとえばcategory.php ファイル内では、
<h1><?php /* translators: %s: category term */  printf( esc_html__( 'Category Archives: %s','tijaji' ), single_cat_title( '', false ) ); ?></h1>

は、

<h1><?php the_archive_title(); ?></h1>

のようになります。表示されるラベル (例えば Category: といった表示) はget_the_archive_title フックを使用して編集できます。

https://developer.wordpress.org/reference/functions/the_archive_title/

#29 @tijaji
2 months ago

了解しました。
修正してまだアップロードさせていただきます。
分かりやすくレビューしてくださりありがとうございます。

#30 @themetracbot
2 months ago

  • Summary changed from THEME: TIJAJI – 1.17 to THEME: TIJAJI – 1.18

TIJAJI - 1.18

日本人が作ったスマホ対応のシンプルなレスポンシブテンプレート。テーマカスタマイザーにてカラム数や文字サイズ、フォントなどカスタムが可能。Simple responsive template made by Japanese. Suitable for smartphone. With theme customizer, you can customize the number of columns, font size, font and so on.

Theme URL - http://tijaji.jp/
Author URL - http://www.tjerji.com

Trac Browser - https://themes.trac.wordpress.org/browser/tijaji/1.18

SVN - https://themes.svn.wordpress.org/tijaji/1.18
ZIP - https://wordpress.org/themes/download/tijaji.1.18.zip?nostats=1

Diff with previous version: https://themes.trac.wordpress.org/changeset?old_path=tijaji/1.17&new_path=tijaji/1.18

History:

Ticket Summary Status Resolution Owner
#46669 THEME: TIJAJI – 1.0 closed not-approved acosmin
#46672 THEME: TIJAJI – 1.01 closed not-approved acosmin
#46682 THEME: TIJAJI – 1.02 closed not-approved rabmalin
#46717 THEME: TIJAJI – 1.24 closed live Mirucon

(this ticket)

#49344 THEME: TIJAJI – 1.25 closed live themetracbot
#49350 THEME: TIJAJI – 1.26 closed live themetracbot


https://themes.svn.wordpress.org/tijaji/1.18/screenshot.png

#31 @tijaji
2 months ago

ご指摘いただいた箇所の修正が終わりました。
よろしくお願いいたします。

#32 @tijaji
2 months ago

追記です。
ご指摘いただいた箇所を修正した以外に、ユーザーページにSNSのURL入力フォームを設置し、投稿者ページで表示させるような修正も加えていました。

#33 @Mirucon
2 months ago

だいぶ改善されてきていますが、まだいくつか修正点が残っています。また現在このテーマにはライセンス上の問題があります。画像を含むすべてのファイルは GPL-Compatible ライセンスであることを確認してください。

Required:

  • automatic-feed-links がサポートされている場合 WordPress は自動でフィードリンクの meta タグを吐き出します。header.php line: 28, 29 にある該当部分を削除してください。
  • @import url の CSS 内での使用は認められません。wp_enqueue_style() を使った適切なエンキューのアプローチを使用してください。
  • 誤字を修正してください - functions.php line: 207, pinback ではなく pingback です。
  • readme.txt の文字エンコードは UTF-8 になっているべきです。Shift JIS ですと文字化けして表示される可能性があります。
  • meta タグの content 内では、区切りに ; ではなく , を使用してください。header.php line: 5.
  • 以下の文字列を翻訳可能にしてください - functions.php line: 266 - 269.
  • ライセンス問題 - WordPress.org レポジトリに掲載されるテーマは必ず GPL ライセンスにて配布される必要があります。これには PHP ファイルや CSS ファイル及び画像ファイルも含まれます。現在このテーマ内の readme ファイルには画像の再配布が出来ないという旨の記述がありますが、必ず再配布を認めない限りテーマをディレクトリに掲載することは出来ません。また、各画像ごとのライセンスを必ず明記してください。例えば以下のような形です:
* default-x.jpg - license: CC0 from http:// 画像へのリンク
* default-x.jpg - license: CC0 from http:// 画像へのリンク
* default-x.jpg - license: CC0 from http:// 画像へのリンク

#34 @tijaji
2 months ago

いつもわかりやすくレビューしていただきありがとうございます。
また修正してご連絡させていただきますm(_ _)m

#35 @themetracbot
7 weeks ago

  • Summary changed from THEME: TIJAJI – 1.18 to THEME: TIJAJI – 1.19

TIJAJI - 1.19

日本人が作ったスマホ対応のシンプルなレスポンシブテンプレート。テーマカスタマイザーにてカラム数や文字サイズ、フォントなどカスタムが可能。Simple responsive template made by Japanese. Suitable for smartphone. With theme customizer, you can customize the number of columns, font size, font and so on.

Theme URL - http://tijaji.jp/
Author URL - http://www.tjerji.com

Trac Browser - https://themes.trac.wordpress.org/browser/tijaji/1.19

SVN - https://themes.svn.wordpress.org/tijaji/1.19
ZIP - https://wordpress.org/themes/download/tijaji.1.19.zip?nostats=1

Diff with previous version: https://themes.trac.wordpress.org/changeset?old_path=tijaji/1.18&new_path=tijaji/1.19

History:

Ticket Summary Status Resolution Owner
#46669 THEME: TIJAJI – 1.0 closed not-approved acosmin
#46672 THEME: TIJAJI – 1.01 closed not-approved acosmin
#46682 THEME: TIJAJI – 1.02 closed not-approved rabmalin
#46717 THEME: TIJAJI – 1.24 closed live Mirucon

(this ticket)

#49344 THEME: TIJAJI – 1.25 closed live themetracbot
#49350 THEME: TIJAJI – 1.26 closed live themetracbot


https://themes.svn.wordpress.org/tijaji/1.19/screenshot.png

#36 @tijaji
7 weeks ago

ご指摘いただいた箇所の修正が終わりました。
よろしくお願いいたします。

#37 @Mirucon
7 weeks ago

だいぶ改善されてきています :) 残りは細かいエラー程度です。

Required:

  • サニタイズ関数 - 色設定のサニタイズには maybe_hash_hex_color() ではなく sanitize_hex_color() を使用してください。
  • ライセンス - Readme ファイル内に以下の例のようにライセンスの宣言をしてください。
Fred WordPress Theme, Copyright 2012 Joe Smith
Fred is distributed under the terms of the GNU GPL
  • author.php line: 137, get_comment_date() 関数をエスケープするか代わりに comment_date() 関数を使ってください。
  • author.php line: 117, wp_reset_query_() ではなく wp_reset_postdata() を使用してください。
  • functions.php line: 120, echo get_comment_author_link() の代わりに comment_author_link() を使用してください。
  • inc/header-image.php 内にて、極力インラインスタイルを使わず CSS 内で記述してください。

#38 @tijaji
7 weeks ago

今回も丁寧でわかりやすいレビューありがとうございます。
了解しました!
また修正してご連絡させていただきます。

#39 @themetracbot
7 weeks ago

  • Summary changed from THEME: TIJAJI – 1.19 to THEME: TIJAJI – 1.20

TIJAJI - 1.20

日本人が作ったスマホ対応のシンプルなレスポンシブテンプレート。テーマカスタマイザーにてカラム数や文字サイズ、フォントなどカスタムが可能。Simple responsive template made by Japanese. Suitable for smartphone. With theme customizer, you can customize the number of columns, font size, font and so on.

Theme URL - https://tijaji.jp/
Author URL - http://www.tjerji.com

Trac Browser - https://themes.trac.wordpress.org/browser/tijaji/1.20

SVN - https://themes.svn.wordpress.org/tijaji/1.20
ZIP - https://wordpress.org/themes/download/tijaji.1.20.zip?nostats=1

Diff with previous version: https://themes.trac.wordpress.org/changeset?old_path=tijaji/1.19&new_path=tijaji/1.20

History:

Ticket Summary Status Resolution Owner
#46669 THEME: TIJAJI – 1.0 closed not-approved acosmin
#46672 THEME: TIJAJI – 1.01 closed not-approved acosmin
#46682 THEME: TIJAJI – 1.02 closed not-approved rabmalin
#46717 THEME: TIJAJI – 1.24 closed live Mirucon

(this ticket)

#49344 THEME: TIJAJI – 1.25 closed live themetracbot
#49350 THEME: TIJAJI – 1.26 closed live themetracbot


https://themes.svn.wordpress.org/tijaji/1.20/screenshot.png

#40 @tijaji
7 weeks ago

ご指摘いただいた箇所の修正が終わりました。
よろしくお願いいたします。

#41 @Mirucon
7 weeks ago

細かいですけど、以下だけ修正お願いします。あとは問題なさそうです。

  • author.php Line 43, esc_html() の代わりに esc_attr() を使ってください。(二つほどあります)

#42 @tijaji
6 weeks ago

こちらこそ細かいミスして申し訳ないです。
了解です!
また修正してご連絡させていただきます!

#43 @themetracbot
6 weeks ago

  • Summary changed from THEME: TIJAJI – 1.20 to THEME: TIJAJI – 1.21

TIJAJI - 1.21

日本人が作ったスマホ対応のシンプルなレスポンシブテンプレート。テーマカスタマイザーにてカラム数や文字サイズ、フォントなどカスタムが可能。Simple responsive template made by Japanese. Suitable for smartphone. With theme customizer, you can customize the number of columns, font size, font and so on.

Theme URL - https://tijaji.jp/
Author URL - http://www.tjerji.com

Trac Browser - https://themes.trac.wordpress.org/browser/tijaji/1.21

SVN - https://themes.svn.wordpress.org/tijaji/1.21
ZIP - https://wordpress.org/themes/download/tijaji.1.21.zip?nostats=1

Diff with previous version: https://themes.trac.wordpress.org/changeset?old_path=tijaji/1.20&new_path=tijaji/1.21

History:

Ticket Summary Status Resolution Owner
#46669 THEME: TIJAJI – 1.0 closed not-approved acosmin
#46672 THEME: TIJAJI – 1.01 closed not-approved acosmin
#46682 THEME: TIJAJI – 1.02 closed not-approved rabmalin
#46717 THEME: TIJAJI – 1.24 closed live Mirucon

(this ticket)

#49344 THEME: TIJAJI – 1.25 closed live themetracbot
#49350 THEME: TIJAJI – 1.26 closed live themetracbot


https://themes.svn.wordpress.org/tijaji/1.21/screenshot.png

#44 @tijaji
6 weeks ago

修正しました。
それともう一度ソースを見直し、CSSファイルでマウスオーバーアクションの一部と、single.php、page.phpにてページ分割時のページナビゲーションの表示動作を修正しました。
いつもお手数をおかけし恐縮ですがよろしくお願いいたします。

#45 @Mirucon
6 weeks ago

問題が後ちょっとだけ残ってます。残りはおそらく問題と思います。

  • readme 内のコピーライトの宣言が消えています。必ずこのフォーマットでのコピーライト・ライセンスの宣言をお願いします。
TIJAJI WordPress Theme, Copyright 2017 tijaji 
TIJAJI is distributed under the terms of the GNU GPL 
  • single.php と page.php のページャー部分の以下の変数を esc_url() を使ってエスケープしてください。
    • $tijaji_get_previousurl[1]
    • $tijaji_get_nexturl[1]
    • $tijaji_get_nexturl[1]
    • $tijaji_get_previousurl[1]
  • wp_reset_postdata が必要になるのはメインループ以外のループです。author.php 内で、一つ目のループがメイン、2つ目のループがセカンドループであるように見受けられます。wp_reset_postdata を endforeach の後に使用し、最初のループの終わりである endwhile の直後のwp_reset_postdata を削除してください。(具体的には L117 の wp_reset_postdata を削除・L148 行目辺りに追加してください)

#46 @tijaji
6 weeks ago

ファイルの更新ミスがあったようです何度もすいません。
了解です!
また修正してご連絡させていただきますm(_ _)m

#47 @themetracbot
5 weeks ago

  • Summary changed from THEME: TIJAJI – 1.21 to THEME: TIJAJI – 1.22

TIJAJI - 1.22

日本人が作ったスマホ対応のシンプルなレスポンシブテンプレート。テーマカスタマイザーにてカラム数や文字サイズ、フォントなどカスタムが可能。Simple responsive template made by Japanese. Suitable for smartphone. With theme customizer, you can customize the number of columns, font size, font and so on.

Theme URL - https://tijaji.jp/
Author URL - http://www.tjerji.com

Trac Browser - https://themes.trac.wordpress.org/browser/tijaji/1.22

SVN - https://themes.svn.wordpress.org/tijaji/1.22
ZIP - https://wordpress.org/themes/download/tijaji.1.22.zip?nostats=1

Diff with previous version: https://themes.trac.wordpress.org/changeset?old_path=tijaji/1.21&new_path=tijaji/1.22

History:

Ticket Summary Status Resolution Owner
#46669 THEME: TIJAJI – 1.0 closed not-approved acosmin
#46672 THEME: TIJAJI – 1.01 closed not-approved acosmin
#46682 THEME: TIJAJI – 1.02 closed not-approved rabmalin
#46717 THEME: TIJAJI – 1.24 closed live Mirucon

(this ticket)

#49344 THEME: TIJAJI – 1.25 closed live themetracbot
#49350 THEME: TIJAJI – 1.26 closed live themetracbot


https://themes.svn.wordpress.org/tijaji/1.22/screenshot.png

#48 @tijaji
5 weeks ago

修正させていただきました。
よろしくお願いいたします。

#49 @themetracbot
5 weeks ago

  • Summary changed from THEME: TIJAJI – 1.22 to THEME: TIJAJI – 1.23

TIJAJI - 1.23

日本人が作ったスマホ対応のシンプルなレスポンシブテンプレート。テーマカスタマイザーにてカラム数や文字サイズ、フォントなどカスタムが可能。Simple responsive template made by Japanese. Suitable for smartphone. With theme customizer, you can customize the number of columns, font size, font and so on.

Theme URL - https://tijaji.jp/
Author URL - http://www.tjerji.com

Trac Browser - https://themes.trac.wordpress.org/browser/tijaji/1.23

SVN - https://themes.svn.wordpress.org/tijaji/1.23
ZIP - https://wordpress.org/themes/download/tijaji.1.23.zip?nostats=1

Diff with previous version: https://themes.trac.wordpress.org/changeset?old_path=tijaji/1.22&new_path=tijaji/1.23

History:

Ticket Summary Status Resolution Owner
#46669 THEME: TIJAJI – 1.0 closed not-approved acosmin
#46672 THEME: TIJAJI – 1.01 closed not-approved acosmin
#46682 THEME: TIJAJI – 1.02 closed not-approved rabmalin
#46717 THEME: TIJAJI – 1.24 closed live Mirucon

(this ticket)

#49344 THEME: TIJAJI – 1.25 closed live themetracbot
#49350 THEME: TIJAJI – 1.26 closed live themetracbot


https://themes.svn.wordpress.org/tijaji/1.23/screenshot.png

#50 @tijaji
5 weeks ago

度々すいません。
スライダーの表示を一部修正しました。
よろしくお願いいたします。

#51 @Mirucon
5 weeks ago

問題の指摘が小出しになってしまい申し訳ないです、おそらくこれで最後になるかと思います !

Required:

  • tijaji_admin_style() は現在すべての管理画面ページで読み込まれてしまっています。tijaji-info.php ページの表示中にのみインラインスタイルが読み込まれるようにしてください。
  • excerpt_length フックを編集することは可能ですが、管理画面部分には影響を及ぼさないようになっている必要があります。具体的には、
if ( is_admin() ) {
  return;
}

のようなコードを関数内に追加してください。

参考: https://github.com/WordPress/WordPress/blob/826c589f119a463df68824887e432801b52724f3/wp-content/themes/twentyseventeen/functions.php#L346

#52 @tijaji
5 weeks ago

いえいえ!いつも本当に勉強になります!ありがとうございます!
また修正してご連絡させていただきます。

#53 @themetracbot
4 weeks ago

  • Summary changed from THEME: TIJAJI – 1.23 to THEME: TIJAJI – 1.24

TIJAJI - 1.24

日本人が作ったスマホ対応のシンプルなレスポンシブテンプレート。テーマカスタマイザーにてカラム数や文字サイズ、フォントなどカスタムが可能。Simple responsive template made by Japanese. Suitable for smartphone. With theme customizer, you can customize the number of columns, font size, font and so on.

Theme URL - https://tijaji.jp/
Author URL - http://www.tjerji.com

Trac Browser - https://themes.trac.wordpress.org/browser/tijaji/1.24

SVN - https://themes.svn.wordpress.org/tijaji/1.24
ZIP - https://wordpress.org/themes/download/tijaji.1.24.zip?nostats=1

Diff with previous version: https://themes.trac.wordpress.org/changeset?old_path=tijaji/1.23&new_path=tijaji/1.24

History:

Ticket Summary Status Resolution Owner
#46669 THEME: TIJAJI – 1.0 closed not-approved acosmin
#46672 THEME: TIJAJI – 1.01 closed not-approved acosmin
#46682 THEME: TIJAJI – 1.02 closed not-approved rabmalin
#46717 THEME: TIJAJI – 1.24 closed live Mirucon

(this ticket)

#49344 THEME: TIJAJI – 1.25 closed live themetracbot
#49350 THEME: TIJAJI – 1.26 closed live themetracbot


https://themes.svn.wordpress.org/tijaji/1.24/screenshot.png

#54 @tijaji
4 weeks ago

ご指摘いただいた箇所を修正させていただきました。
それとは別にテーマカスタマイザーに機能追加版へのリンクも追加しました。
よろしくお願いいたします。

#55 @Mirucon
4 weeks ago

  • Resolution set to live
  • Status changed from reviewing to closed

お疲れ様でした ! テーマを live としてマークしました。およそ一時間ほどでテーマがディレクトリに表示されるようになります !

#56 @tijaji
4 weeks ago

ミルコンさんには感謝の言葉しかありません!
いろいろと教えていただき本当にありがとうございました!

Note: See TracTickets for help on using tickets.