Simplicity2.0へに向けて

meta:ogpの後ぐらいに、

<link id="parent-style-css" media="all" type="text/css" href="http://localhost/blog/wp-content/themes/simplicity/style.css" rel="stylesheet">
<link id="child-style-css" media="all" type="text/css" href="http://localhost/blog/wp-content/themes/simplicity-child/style.css" rel="stylesheet">

こんな感じで入っているので、重複はしてますが機能はしています。
関数の中で依存関係が作れるため、優先順位を指定できるのと、例えば独自のSNSボタンの画像とかを使うのでfontawesome以下webフォントは必要ないなんて設定を設けたとして、

1 //親テーマ読み込み
2 //子テーマ読み込み(親テーマ依存)
3 if (enable==on){
　//fontawesome読み込み
//他のwebfont読み込み(fontawesome依存)
}
なんてことができます。elableがoffであればcssは読み込みません。

って事は、他の@import等を利用しているファイルも依存関係を作り、@importをできるだけ無くすことも可能かと。その分。linkタグで書き出すのでmwta部分は肥大するかと思いますが、優先関係が見てすぐわかるのは視覚管理としては良いかも知れません。

まぁsimplicity2.0へ向けての話なので、cssをどのようにまとめて、どのようにして読み込むかの再考には良いかと知れませんね。

twentysixteenのfunctions.php、226行目あたりからが参考になるかと思います。

Notice: Undefined index: pinterest in /usr/share/nginx/html/blog/wp-content/themes/simplicity/lib/customizer.php on line 3780

で、カスタマイザーの該当箇所でダンプしてみたんですが、

$oの配列の中身

Array
(
    [top_btns_visible] => 1
    [obsequence_sns_btns_visible] => 1
    [share_msg] => シェアする
    [follow_msg] => ko-hidekichi
    [twitter] => ko-hidekichi
    [facebook] => 
    [google_plus] => 12345678901
    [instagram] => 
    [fb_admins] => 
    [fb_app_id] => 
    [ogp_description] => excerpt
    [colored_follow_btns] => 1
)

確かにpinterestやらyoutubeやらが見つかりませんね。

もう1件、widget.phpあたりで、

class SimplicityWidgetItem extends WP_Widget {
  //function SimplicityWidgetItem() {
  function __construct() {

かなぁと。これだけではないんですが取り急ぎ、報告まで。

ご意見失礼します。
また、下記のご意見について大変恐縮です。

2.0に向けて、現在数多くのテーマ機能があり、
必要であればチェック or 不要であればチェック外すという事ですが、
ちょこっとしたテーマ容量を考えまして、そこでアンケート等の形式を公開して、
利用者から必要な機能 or 不要な機能の声を集めて必要な機能だけをテーマに組み込むというのはどうでしょうか？

↑このやり方について時間かかって面倒くさい等でしたら、ご意見として受け取っていただければ大変幸いです。

#34764

かしこまりました。
わいひら様の体調等も配慮の上承知しておりましたが、
ご無理を発言してしまい大変申し訳ございません。

#34759 で、カスタマイザーのSNSの中身をダンプしたという話の続きで、例えば、

$tes = get_sns_options();

この$tesをダンプした時に、各SNSの設定のオプションが配列で入っているはずなんですが、customizer.phpをザックリ見なおした感じ、特に書き方に不備があるわけではなさそうなんですが、何故かpinterestとyoutubeのindexが入ってない状態です。

customizer.phpの中身が結構あるため、わかる方はちょっと解析を手伝ってもらいたい感じです。
まぁ無ければナイで、

//PinterestフォローボタンのIDを取得 3773行目あたり
function get_pinterest_follow_id(){
$o = get_sns_options();
return trim($o[‘pinterest’]);
}
で、「無い」と返せばよいわけですが、ちょっと気になる所なので。

widget.phpは、一応noticeがでない所までは、修復できた感じですが、まだちょっと作業中なので後回しにして、header-ogp.phpとheader-twitter.phpのnoticeについては、以下のような感じで直せそうです。
本来の動作かどうかはわからないので、ひとまず確認をお願いしたい感じです。

$searchPattern = ‘/<img.*?src=([“\’])(.+?)\1.*?>/i’;//投稿にイメージがあるか調べる

ここから下の一連の処理の最後、

if ( $ogp_image ) {//使えそうな$ogp_imageがある場合
	echo '<meta property="og:image" content="'.$ogp_image.'">';echo "\n";
}

ここを、

if ( !empty($ogp_image) ) {//使えそうな$ogp_imageがある場合
	echo '<meta property="og:image" content="'.$ogp_image.'">';echo "\n";
} else {
	$ogp_image = get_stylesheet_directory_uri().'/images/og-image.jpg';
	echo '<meta property="og:image" content="'.$ogp_image.'">';echo "\n";
}

こんな感じに。

トップページのリスト表示の時は、$ogp_imageがないので、ひとまずSimplicityのog-image.jpgを当てています。※ 全てのイメージがない時の処理なのでデフォルトで
header-twitter.phpも同様です。

https://github.com/hidekichi/Simplicity/tree/master/original_fix_files

上記にpushしときました。