search

vektor-inc / vk-all-in-one-expansion-unit

wordpress plugin of powerful support
https://ja.wordpress.org/plugins/vk-all-in-one-expansion-unit/
GNU General Public License v2.0
7 stars 1 forks source link

【確認待ち】<title> タグの内容を任意に出力できるように #732 #733

Closed drill-lancer closed 2 years ago

drill-lancer commented 2 years ago

上記に対応したものを実装しておきました。 get_the_title() の挙動が少々おかしい気がしますが （今回追加した post_meta が存在しないものが先頭にある場合 get_the_title() が空を返す） テストも通る形で追加しておきました。

kurudrive commented 2 years ago

@drill-lancer ありがとうございます。

ごめんなさい。 これは「タイトルタグの書き換え」機能なので、「SEO Title」という別の機能として実装するのではなく、 「タイトルタグの書き換え」機能が有効なら使える機能として実装してください。 でないとユーザーからすると「タイトルタグの書き換え」と「SEO Title」と別機能になってると、何がちがうの？と混乱してしまうけれど、実際ユーザーはこの違いは意識する必要はないので。

wp-title.php が関数で書いてあるから、書きにくければ纏めて class 化してもかまいません。

drill-lancer commented 2 years ago

@kurudrive パッケージ化して１つにまとめて置きました。

kurudrive commented 2 years ago

@drill-lancer ありがとうございます。

drill-lancer commented 2 years ago

@kurudrive しておきました。 固定ページの優先順位はこれで OK ですか？

kurudrive commented 2 years ago

@drill-lancer ありがとうございます。 今後過去のファイルも編集する際はお手数ですができるだけコーディング規約に添うように修正よろしくお願いいたします。 ※ なんともならない所とかも多々あると思うので、全部ではなくてもOKですが、努力目標として直せそうな部分は修正する感じでよろしくお願いいたします。