プロジェクト

全般

プロフィール

Backport(バックポート) #3714

スマートフォン向けUIで、ホーム画面以外から「PC表示に切り替え」ボタンを押した場合の挙動が正しくない

isao sano9年以上前に追加. 7年以上前に更新.

ステータス:
Fixed(完了)
優先度:
Normal(通常)
担当者:
対象バージョン:
開始日:
2013-06-17
期日:
進捗率:

100%


説明

現象

スマートフォン端末で、メンバー検索 (/member/search) やフレンドリスト (/friend/list) のようにURLのパスが2階層以上のページで「PC表示に切り替え」ボタンを押すと、PC向け画面に切り替わらずにホーム画面に移動する。また、「PC表示に切り替え」ボタンを押したページの階層以下でのみPC向け画面で表示される。

  1. スマートフォン端末から、コミュニティ検索 (/community/search) を開く
  2. 「PC表示に切り替え」ボタンを押す
  3. ホーム画面に移動するが、スマートフォン向け画面のまま表示される
  4. メンバー検索 (/member/search) など異なる階層のページでもスマートフォン向け画面となる
  5. コミュニティ検索 (/community/search) やコミュニティ作成 (/community/edit) ではPC向け画面に切り替わっている

原因

source:web/js/smt_main.js の「PC向け表示」ボタンに関するコードは下記のようになっている。

  $('#smt-switch')
    .show()
    .click(function() {
      document.cookie = "disable_smt=1";
      location.href = $(this).attr('href');
    });

document.cookie によって Cookie を設定しているが path が指定されていない。path が指定されていない場合のデフォルト値は、表示中のページのパスとなる。つまり、コミュニティ検索 (/community/search) ページの場合は /community/ が path にセットされる。よって、この場合は /community/ 以下の階層のページで disable_smt の値が有効となり、それ以外のページでは依然としてスマートフォン向け画面が表示されることになる。

修正内容

#3365 によって openpne.baseUrl を追加した上で、Cookie 設定時の path に openpne.baseUrl を指定することによって SNS 内の全てのページで disable_smt の値が有効となるようにする。


関連するチケット

コピー元 OpenPNE 3 - Bug(バグ) #3364: スマートフォン向けUIで、ホーム画面以外から「PC表示に切り替え」ボタンを押した場合の挙動が正しくない Won't fix(対応せず) 2013-06-17

関係しているリビジョン

リビジョン e570f5a7 (差分)
isao sano9年以上前に追加

fix bug in switching smartphone/PC layout (fix #3714, BP from #3364)

リビジョン 1d548adf
kaoru n7年以上前に追加

Merge pull request #212 from isaosano/t-3714

fix bug in switching smartphone/PC layout (fix #3714, BP from #3364)

履歴

#1 isao sano9年以上前に更新

  • コピー元 Bug(バグ) #3364: スマートフォン向けUIで、ホーム画面以外から「PC表示に切り替え」ボタンを押した場合の挙動が正しくない を追加

#2 isao sano9年以上前に更新

  • ステータスPending Review(レビュー待ち) から New(新規) に変更
  • 担当者 を削除 (Youichi Kimura)
  • 進捗率50 から 0 に変更

#3 isao sano9年以上前に更新

  • ステータスNew(新規) から Accepted(着手) に変更

#4 isao sano9年以上前に更新

  • ステータスAccepted(着手) から Pending Review(レビュー待ち) に変更
  • 進捗率0 から 50 に変更

https://github.com/openpne/OpenPNE3/pull/212
にてプルリクエストを行いました

#5 Shinichi Urabe9年以上前に更新

  • 担当者isao sano にセット

担当を設定ください(差し戻し対応が誰か不明になるため)

#6 isao sano9年以上前に更新

  • 対象バージョンOpenPNE 3.8.14 から OpenPNE 3.8.x に変更

#7 kaoru n8年以上前に更新

  • 対象バージョンOpenPNE 3.8.x から OpenPNE 3.8.18 に変更

#8 kaoru n8年以上前に更新

  • 対象バージョンOpenPNE 3.8.18 から OpenPNE 3.8.x に変更

#9 kaoru n8年以上前に更新

  • 対象バージョンOpenPNE 3.8.x から OpenPNE 3.8.19 に変更

#10 kaoru n約8年前に更新

  • 対象バージョンOpenPNE 3.8.19 から OpenPNE 3.8.x に変更

#11 kaoru n7年以上前に更新

  • 対象バージョンOpenPNE 3.8.x から OpenPNE 3.8.21 に変更

#12 kaoru n7年以上前に更新

  • ステータスPending Review(レビュー待ち) から Rejected(差し戻し) に変更
  • 担当者isao sano から Youichi Kimura に変更

#13 kaoru n7年以上前に更新

  • ステータスRejected(差し戻し) から Pending Review(レビュー待ち) に変更

https://redmine.openpne.jp/issues/3364#note-7
により、レビュー待ちに変更

#14 kaoru n7年以上前に更新

  • ステータスPending Review(レビュー待ち) から Pending Testing(テスト待ち) に変更
  • 進捗率50 から 70 に変更

レビューOKです

#15 isao sano7年以上前に更新

  • ステータスPending Testing(テスト待ち) から Pending Merge(マージ待ち) に変更
  • 進捗率70 から 80 に変更

単体試験完了しました。
問題ありません。

#16 kaoru n7年以上前に更新

  • ステータスPending Merge(マージ待ち) から Fixed(完了) に変更
  • 進捗率80 から 100 に変更

マージしました

他の形式にエクスポート: Atom PDF