ec-cube icon indicating copy to clipboard operation
ec-cube copied to clipboard

商品詳細ページにSNS共有ボタンを表示する

Open matsuoshi opened this issue 4 years ago • 2 comments

概要(Overview・Refs Issue)

商品詳細ページほかに、SNSシェアボタンを追加します

#4849

方針(Policy)

Twitter および Facebook のシェアボタンを、各ページに配置します。 ただし、以下のページでは表示されません。

  • マイページ系
  • カートページ系
  • 会員登録ページ系

実装に関する補足(Appendix)

シェアボタンは新規ブロック social_buttons.twig として作成し、デフォルト状態ではレイアウトのフッター上部に入るように設定しています。

CSS はスタイルガイド内の "2.3 その他ボタン" 内に追加しました。

テスト(Test)

期待するページに、SNSシェアボタンのブロックが出力されていることを確認する Webテストを追加しました。

相談(Discussion)

マイナーバージョン互換性保持のための制限事項チェックリスト

  • [x] 既存機能の仕様変更
  • [x] フックポイントの呼び出しタイミングの変更
  • [x] フックポイントのパラメータの削除・データ型の変更
  • [x] twigファイルに渡しているパラメータの削除・データ型の変更
  • [x] Serviceクラスの公開関数の、引数の削除・データ型の変更
  • [x] 入出力ファイル(CSVなど)のフォーマット変更

レビュワー確認項目

  • [ ] 動作確認
  • [ ] コードレビュー
  • [ ] E2E/Unit テスト確認(テストの追加・変更が必要かどうか)
  • [ ] 互換性が保持されているか
  • [ ] セキュリティ上の問題がないか

matsuoshi avatar Feb 12 '21 08:02 matsuoshi

デフォルト状態から表示されるようになっていますが利用しないサイトもありますので、 デフォルトは未設定が良いと思います。

k-yamamura avatar Dec 03 '21 05:12 k-yamamura

現在、TwitterとFacebookの追加を頂いておりますが、この2つで良いかという点と Twitter→Xに変更しているという点から 追加の改修がまだ必要だと感じています。

また別の問題として、そもそも需要がないというのもあります 参考: https://twitter.com/EC_CUBE/status/1765283570686521420

こちらもう少しディスカッションを重ねたいです。

shinya avatar Apr 17 '24 07:04 shinya