Backport(バックポート) #3714
スマートフォン向けUIで、ホーム画面以外から「PC表示に切り替え」ボタンを押した場合の挙動が正しくない
100%
Description
現象¶
スマートフォン端末で、メンバー検索 (/member/search) やフレンドリスト (/friend/list) のようにURLのパスが2階層以上のページで「PC表示に切り替え」ボタンを押すと、PC向け画面に切り替わらずにホーム画面に移動する。また、「PC表示に切り替え」ボタンを押したページの階層以下でのみPC向け画面で表示される。
例¶
- スマートフォン端末から、コミュニティ検索 (/community/search) を開く
- 「PC表示に切り替え」ボタンを押す
- ホーム画面に移動するが、スマートフォン向け画面のまま表示される
- メンバー検索 (/member/search) など異なる階層のページでもスマートフォン向け画面となる
- コミュニティ検索 (/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 の値が有効となるようにする。
Related issues
Associated revisions
History
#1
Updated by isao sano over 6 years ago
- Copied from Bug(バグ) #3364: スマートフォン向けUIで、ホーム画面以外から「PC表示に切り替え」ボタンを押した場合の挙動が正しくない added
#2
Updated by isao sano over 6 years ago
- Status changed from Pending Review(レビュー待ち) to New(新規)
- Assignee deleted (
Youichi Kimura) - % Done changed from 50 to 0
#3
Updated by isao sano over 6 years ago
- Status changed from New(新規) to Accepted(着手)
#4
Updated by isao sano over 6 years ago
- Status changed from Accepted(着手) to Pending Review(レビュー待ち)
- % Done changed from 0 to 50
https://github.com/openpne/OpenPNE3/pull/212
にてプルリクエストを行いました
#6
Updated by isao sano over 6 years ago
- Target version changed from OpenPNE 3.8.14 to OpenPNE 3.8.x
#7
Updated by kaoru n over 5 years ago
- Target version changed from OpenPNE 3.8.x to OpenPNE 3.8.18
#8
Updated by kaoru n over 5 years ago
- Target version changed from OpenPNE 3.8.18 to OpenPNE 3.8.x
#9
Updated by kaoru n over 5 years ago
- Target version changed from OpenPNE 3.8.x to OpenPNE 3.8.19
#10
Updated by kaoru n about 5 years ago
- Target version changed from OpenPNE 3.8.19 to OpenPNE 3.8.x
#11
Updated by kaoru n over 4 years ago
- Target version changed from OpenPNE 3.8.x to OpenPNE 3.8.21
#12
Updated by kaoru n over 4 years ago
- Status changed from Pending Review(レビュー待ち) to Rejected(差し戻し)
- Assignee changed from isao sano to Youichi Kimura
#13
Updated by kaoru n over 4 years ago
- Status changed from Rejected(差し戻し) to Pending Review(レビュー待ち)
https://redmine.openpne.jp/issues/3364#note-7
により、レビュー待ちに変更
#14
Updated by kaoru n over 4 years ago
- Status changed from Pending Review(レビュー待ち) to Pending Testing(テスト待ち)
- % Done changed from 50 to 70
レビューOKです
#15
Updated by isao sano over 4 years ago
- Status changed from Pending Testing(テスト待ち) to Pending Merge(マージ待ち)
- % Done changed from 70 to 80
単体試験完了しました。
問題ありません。
#16
Updated by kaoru n over 4 years ago
- Status changed from Pending Merge(マージ待ち) to Fixed(完了)
- % Done changed from 80 to 100
マージしました