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

ショッピングカート画面で、スマホのドロワーメニューが表示されない不具合の修正 #2488

Merged
merged 1 commit into from Sep 11, 2017

Conversation

t-nagahashi
Copy link
Contributor

@t-nagahashi t-nagahashi commented Sep 6, 2017

以下を参考にコメントを作成してください。

概要(Overview・Refs Issue)

方針(Policy)

  • このPullRequestを作るにあたって考慮したものや除外し内容
    なし

実装に関する補足(Appendix)

  • コードだけではわかりづらい点など、実装するにあたってレビューアに追加で伝えておきたいこと
    なし

テスト(Test)

  • テストを行っている範囲など、レビューアが安心できるような情報
    ■動作確認済状況
    PCでの確認(chrome, firefox, safariの最新)
    chromeでのスマホシミュレータでの確認(iPhone6, Nexus)
    ■動作未確認
    スマホ実機での検証

相談(Discussion)

  • 相談したいことや意見を求めたいこと
    なし

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

  • マイナーバージョンでは、機能・プラグイン・デザインテンプレート互換性を損なう変更は原則取り込みません。
  • 既存機能の仕様変更
    フロントのタグ構造を変更しておりますが、
    3.0.14で正常動作していた状態に戻したため問題なしの想定です。
  • フックポイントの呼び出しタイミングの変更
  • フックポイントのパラメータの削除・データ型の変更
  • twigファイルに渡しているパラメータの削除・データ型の変更
  • Serviceクラスの公開関数の、引数の削除・データ型の変更
  • 入出力ファイル(CSVなど)のフォーマット変更

@t-nagahashi t-nagahashi added this to the 3.0.16 milestone Sep 6, 2017
@t-nagahashi t-nagahashi added the affected:template フロントテンプレートの変更 label Sep 6, 2017
@ryo-endo ryo-endo changed the title divタグの数を修正 ショッピングカート画面で、スマホのドロワーメニューが表示されない不具合の修正 Sep 8, 2017
Copy link
Contributor

@ryo-endo ryo-endo left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

DIVタグ対応数をコードで確認。
iPhone(iOS10 Safari)で不具合が解消していることを確認。

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
affected:template フロントテンプレートの変更 bug:High Status: ready-for-merge
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants