Project

General

Profile

Backport(バックポート) #3714

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

Added by isao sano over 5 years ago. Updated over 3 years ago.

Status:
Fixed(完了)
Priority:
Normal(通常)
Target version:
Start date:
2013-06-17
Due date:
% Done:

100%


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

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

Associated revisions

Revision e570f5a7 (diff)
Added by isao sano over 5 years ago

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

Revision 1d548adf
Added by kaoru n over 3 years ago

Merge pull request #212 from isaosano/t-3714

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

History

#1 Updated by isao sano over 5 years ago

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

#2 Updated by isao sano over 5 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 5 years ago

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

#4 Updated by isao sano over 5 years ago

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

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

#5 Updated by Shinichi Urabe over 5 years ago

  • Assignee set to isao sano

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

#6 Updated by isao sano over 5 years ago

  • Target version changed from OpenPNE 3.8.14 to OpenPNE 3.8.x

#7 Updated by kaoru n over 4 years ago

  • Target version changed from OpenPNE 3.8.x to OpenPNE 3.8.18

#8 Updated by kaoru n over 4 years ago

  • Target version changed from OpenPNE 3.8.18 to OpenPNE 3.8.x

#9 Updated by kaoru n over 4 years ago

  • Target version changed from OpenPNE 3.8.x to OpenPNE 3.8.19

#10 Updated by kaoru n about 4 years ago

  • Target version changed from OpenPNE 3.8.19 to OpenPNE 3.8.x

#11 Updated by kaoru n almost 4 years ago

  • Target version changed from OpenPNE 3.8.x to OpenPNE 3.8.21

#12 Updated by kaoru n almost 4 years ago

  • Status changed from Pending Review(レビュー待ち) to Rejected(差し戻し)
  • Assignee changed from isao sano to Youichi Kimura

#13 Updated by kaoru n almost 4 years ago

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

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

#14 Updated by kaoru n almost 4 years ago

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

レビューOKです

#15 Updated by isao sano over 3 years ago

  • Status changed from Pending Testing(テスト待ち) to Pending Merge(マージ待ち)
  • % Done changed from 70 to 80

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

#16 Updated by kaoru n over 3 years ago

  • Status changed from Pending Merge(マージ待ち) to Fixed(完了)
  • % Done changed from 80 to 100

マージしました

Also available in: Atom PDF