【保留・相談待ち】「続きを読む」リンクテキストの背景色を削除してみました

sysbird commented 10 months ago

チケットへのリンク / 変更の理由（元のissueがあればリンクを貼り付ければOK）

【相談】クエリーループの中に抜粋を入れて、「続きを読む」リンクテキストにテキストを入れると背景色がついてしまう #220

どういう変更をしたか？

クエリーループの抜粋の「続きを読む」に背景色があってボタンみたいに表示されていた箇所を、ただの下線リンクにしました

実装者はレビュワーに回す前に以下の事を確認してチェックをつけてください。

ソースコードについて

[x] 複数の意図の変更（機能の不具合修正 + 別の機能追加など）を含んでいないか？
[ ] 関数名 / 変数名 / クラス名 / 保存値名はそれだけで内容が想像できるものになっているか？紛らわしい命名になっていないか？
[ ] 関数名 / 変数名 / クラス名 / 保存値名は既存のコードの命名規則に沿ったものになっているか？

プログラムの変更の場合

テストを書かないのは普通ではありません。書けるテストは極力書くようにしてください。書いていない場合は書かない理由を記載してください。

[ ] 書けそうなテストは書いたか？

その他

[x] readme.txt に変更内容は書いたか？
[x] Files changed (変更ファイル)の内容は目視でちゃんと確認したか？
[x] このチェック項目を機械的にチェックするのではなく本当にちゃんと確認をしたか？
[x] レビュワーが確認しないでリリースしてしまっても問題ないレベルまでちゃんと作りこみ・確認をしたか？

変更内容について何を確認したか、どういう方法で確認をしたかなど

クエリーループの抜粋の「続きを読む」にあった背景色とパディングを css で削除し、下線のみリンクで表示されるようにしました
箇所は、トップページ、アーカイブ、検索結果など投稿の一覧と、投稿でのクエリーブロックなど

レビュワーの確認方法・確認する内容など

上記と同じ

レビュワーに回す前の確認事項

[ ] このテンプレートのチェック項目をちゃんと確認してチェックしたか？

.wp-block-post-excerpt__more-link {
    transition: all 0.1s ease-in;
    background-color: var(--wp--preset--color--primary);
    color: #fff;
    text-decoration: none;
    border-radius: var(--wp--custom--radius--button);
}

.wp-block-post-excerpt__more-link:hover {
    --wp--preset--color--primary: var(--wp--preset--color--primary-hover);
}

.wp-block-query .wp-block-post-excerpt__more-text a {
    padding: var(--wp--custom--spacing--button-sm);
}

sysbird commented 10 months ago

@goutetsuguma @kurudrive 対応してみました、これで合ってますかね？ボタンみたいに表示されていたのが急に変わってもよいかどうか？もとの issue も【相談】の段階ですので、ご確認をお願いします

goutetsuguma commented 10 months ago

@sysbird @kurudrive とりさんありがとうございます！すみません。。。今確認していたのですが、VKパターンライブラリのX-T9にした時のiframe内で、続きを読むボタンの周りのpaddingがうまくきいていなかっただけのようでした、、、

続きを読むテキストにpaddingの余白がつかなかったので、背景色が無い方が良いと思い、このissueをたてさせていただいたのですが、今わたしのローカルでみたところ、余白もちゃんとついていてボタンの様に表示されていました。。。

すみません、、私の勘違いだった様です、、、

ボタンみたいに表示されていたのが急に変わってもよいかどうか？

私の確認ミスで申し訳ないです、、たしかに今までボタンになっていたので、そのままの方が良さそうです。。ほんとにすみません、、

@kurudrive VKパターンライブラリのX-T9に変更した時にiframe内のcssがうまく当たっておらず、わたしの確認不足のissueでした。とりさんがせっかく実装してくださったのですが、Close でよいでしょうか、、？

sysbird commented 10 months ago

@goutetsuguma おぉ、そうだったんですね、確認できてよかったです close でOKです〜

個人的には「続きを読む」」インライン表示のときは背景なしがいいなーと感じます

goutetsuguma commented 10 months ago

@sysbird @kurudrive

個人的には「続きを読む」」インライン表示のときは背景なしがいいなーと感じます

テキストだけの時は背景無しの方が良いですよねぇ。。。

あ、記事の続きを読むボタンの背景色はテーマ側にかいてあると、角丸のボタンにしたい時にcss書かないと調整できないから、やっぱり背景は無い方が良いかもしれないと思えてきました、、！

そしてブロック側でここのボタンの背景を設定できる様になるかもしれないですよね。

@kurudrive ↑ 石川さんはいかがでしょうか、、？（インストール数1,000の今ならまだ変えやすい、、？）

一応、ボタンにする場合のcssのお知らせもいれつつ、背景無しの方向で行くとか、、？

https://demo.dev3.biz/x-t9/

sysbird commented 10 months ago

続きを読むボタンの周りのpaddingがうまくきいていなかった

もしかしたら記事のなかにクエリーループブロックを置くと、わたしもそうなる気がします

kurudrive commented 4 months ago

これそもそも抜粋ブロックで続きを読むを非表示にできれば、そこで非表示にして、「続きを読む」ブロックで色とか付ければいいのにね...

次回ミーティングでどうするか相談しましょう...（＝ｗ＝；

vektor-inc / x-t9