Project

General

Profile

Bug(バグ) #3364

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

Added by Youichi Kimura over 6 years ago. Updated over 2 years ago.

Status:
Won't fix(対応せず)
Priority:
Normal(通常)
Target version:
Start date:
2013-06-17
Due date:
% Done:

0%

3.6 で発生するか:
No (いいえ)
3.8 で発生するか:
Yes (はい)

Description

現象

スマートフォン端末で、メンバー検索 (/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 の値が有効となるようにする。


Related issues

Blocked by OpenPNE 3 - Enhancement(機能追加・改善) #3365: OpenPNEが設置されているURLをJavaScriptコードから取得できるようにする Won't fix(対応せず) 2013-06-17
Copied to OpenPNE 3 - Backport(バックポート) #3713: スマートフォン向けUIで、ホーム画面以外から「PC表示に切り替え」ボタンを押した場合の挙動が正しくない Invalid(無効) 2013-06-17
Copied to OpenPNE 3 - Backport(バックポート) #3714: スマートフォン向けUIで、ホーム画面以外から「PC表示に切り替え」ボタンを押した場合の挙動が正しくない Fixed(完了) 2013-06-17

Associated revisions

Revision b460dfa0 (diff)
Added by Youichi Kimura over 6 years ago

fix bug in switching smartphone/PC layout (fix #3364)

History

#1 Updated by Youichi Kimura over 6 years ago

  • Description updated (diff)
  • Status changed from New(新規) to Accepted(着手)

#2 Updated by Youichi Kimura over 6 years ago

  • Description updated (diff)

#3 Updated by Youichi Kimura over 6 years ago

  • Status changed from Accepted(着手) to Pending Review(レビュー待ち)
  • % Done changed from 0 to 50

更新履歴 b460dfa0c0e3c7fa022304f448fe15cfd21d26b9 で適用されました。

#4 Updated by isao sano about 5 years ago

  • Copied to Backport(バックポート) #3713: スマートフォン向けUIで、ホーム画面以外から「PC表示に切り替え」ボタンを押した場合の挙動が正しくない added

#5 Updated by isao sano about 5 years ago

  • Copied to Backport(バックポート) #3714: スマートフォン向けUIで、ホーム画面以外から「PC表示に切り替え」ボタンを押した場合の挙動が正しくない added

#6 Updated by kaoru n over 3 years ago

  • Status changed from Pending Review(レビュー待ち) to Rejected(差し戻し)

openpne.baseUrl が定義・設定されていないので、 例の 1, 2, において、path が /community/ に設定されます。

なお、opMessagePlugin の場合は、下記のように定義・設定されています。
https://github.com/tejimaya/opMessagePlugin/blob/master/web/js/smt-message.js#L37-L41

#7 Updated by Youichi Kimura over 3 years ago

  • Status changed from Rejected(差し戻し) to Pending Review(レビュー待ち)
  • 3.6 で発生するか changed from Unknown (未調査) to No (いいえ)
  • 3.8 で発生するか changed from Unknown (未調査) to Yes (はい)

b460dfa0 の修正は #3365 で追加される openpne.baseUrl に依存するため、 #3365 が先に取り込まれている必要があります。
(#3365 の 3.8.x 向けバックポートを #3989 に作成しておきました)

#8 Updated by kaoru n over 3 years ago

  • Status changed from Pending Review(レビュー待ち) to Pending Testing(テスト待ち)
  • % Done changed from 50 to 70

レビューOKです

#9 Updated by isao sano over 2 years ago

  • Status changed from Pending Testing(テスト待ち) to Won't fix(対応せず)
  • % Done changed from 70 to 0

OpenPNE 3.8.21 にて対応済みであったため、対応せずとします。

Also available in: Atom PDF