Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

「続きを読む」リンクテキストの背景色を削除してみました #221

Open
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

sysbird
Copy link
Member

@sysbird sysbird commented Nov 17, 2023

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

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

どういう変更をしたか?

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

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

ソースコードについて

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

プログラムの変更の場合

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

  • 書けそうなテストは書いたか?

その他

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

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

  • クエリーループの抜粋の「続きを読む」にあった背景色とパディングを 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 sysbird marked this pull request as ready for review November 17, 2023 04:39
@sysbird
Copy link
Member Author

sysbird commented Nov 17, 2023

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

@goutetsuguma
Copy link
Contributor

goutetsuguma commented Nov 17, 2023

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

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

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

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

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

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

@sysbird
Copy link
Member Author

sysbird commented Nov 17, 2023

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

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

@goutetsuguma
Copy link
Contributor

goutetsuguma commented Nov 17, 2023

@sysbird @kurudrive

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

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

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

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

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

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

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

@sysbird
Copy link
Member Author

sysbird commented Nov 20, 2023

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

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

@kurudrive
Copy link
Member

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

次回ミーティングでどうするか相談しましょう...(=w=;

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants