ここの情報は古いです。ご理解頂いた上でお取り扱いください。

Opened 12 years ago

Closed 12 years ago

Last modified 12 years ago

#1538 closed defect (fixed)

SoftBankの一部機種でログイン画面のボタンがレイアウト崩れを起こす

Reported by: imoto Owned by: ebihara
Priority: major Milestone: OpenPNE2.10.2
Component: pne-ktai Version: 2.10.x & 2.11.x
Keywords: OpenPNE2.11.2 Cc:

Description (last modified by ebihara)

■現象

  • 旧vodafoneの一部機種で、フォームのボタンが横スクロールをしないと表示できなくなっている場所がある

■原因

SoftBankでは、table要素のwidth属性値に100%を指定した場合、その幅は横方向にスクロール可能な最大幅であるとしていることが原因(■参考資料を参照)。

■修正内容

  • SoftBank? の 3G 端末
    • 変更なし
  • SoftBank? の 3G 以前の端末
    • 拡張HTTPヘッダの x-jphone-display から表示可能幅を取得する
    • Smartyのフィルターで、テンプレート内の <table width="100%" の 100% を 取得した幅に置換する

■参考資料

  • 参照URL
    バージョンアップやってみました。
    
    本日バージョンアップして、2.10.0を入れてみました。
    一回変更箇所を間違えて、うぉ~ぉというエラー表示を見ましたが、
    間違いにすぐ気がついたので、今のところは普通に動いております。
    やはり携帯画面が好評ですよ~★
    
    利用者さんに携帯で色々試して貰ったところ、
    報告で
    
    vodafone2.5Gで、ログイン画面の
    「ログイン」ボタンと「かんたんログイン」ボタンが
    かなり右にスクロールしないと表示されません~という報告を頂きました。
    
    あと、キャリアはボーダフォンで機種はV803Tで、表示が文字化けしてしまうというのがありました。
    
    
    一応、ご報告です<(_ _)>
    また何かありましたらご報告いたします。
    
  • 同じ内容の報告
    • #1489 携帯:ログインボタンが表示されない
    • #1539 vodafoneV803Tで、表示が文字化けする

http://sns.openpne.jp/?m=pc&a=page_c_topic_detail&target_c_commu_topic_id=2809

ソフトバンクの技術資料
http://developers.softbankmobile.co.jp/dp/tool_dl/web/tec...
のHTML編(PDF)
http://www2.developers.softbankmobile.co.jp/dp/tool_dl/do...
から、関連ありそうなのを探してみました。(拾い読みなので外しているかも)
-----
table要素の説明:
  「表の横幅がLCDの横幅を超える場合には、折り返すことなく表示する。表を閲覧する場合には横スクロールする。」(p.124)
 width属性の説明:
  「表の表示する幅を『ピクセル数』または『%』で指定する。横方向にスクロール可能な最大幅を100%とする。…」(p.124)
td要素の説明:
 「1セルの横幅は300Pixelである。」(p.132)
要素の入れ子関係:
 td→centerは禁止される入れ子関係(動作は保証されない)(p.250)
-----

Change History (27)

comment:1 Changed 12 years ago by kiwa

Type: enhancementdefect

comment:2 Changed 12 years ago by kiwa

Component: pne-adminpne-ktai
Keywords: 再現待ち removed
Summary: vodafone2.5Gで、ログイン画面のボタンがレイアウト崩れを起こすSoftBankの一部機種でログイン画面のボタンがレイアウト崩れを起こす

comment:3 Changed 12 years ago by kiwa

Description: modified (diff)

comment:4 Changed 12 years ago by kiwa

Milestone: OpenPNE2.10.2

comment:5 Changed 12 years ago by kiwa

Description: modified (diff)

comment:6 Changed 12 years ago by kiwa

Description: modified (diff)

comment:7 Changed 12 years ago by kiwa

http://sns.openpne.jp/?m=pc&a=page_c_topic_detail&target_c_commu_topic_id=2809
コメントID:13に原因がありました

ソフトバンクの技術資料
http://developers.softbankmobile.co.jp/dp/tool_dl/web/tec...
のHTML編(PDF)
http://www2.developers.softbankmobile.co.jp/dp/tool_dl/do...
から、関連ありそうなのを探してみました。(拾い読みなので外しているかも)
-----
table要素の説明:
  「表の横幅がLCDの横幅を超える場合には、折り返すことなく表示する。表を閲覧する場合には横スクロールする。」(p.124)
 width属性の説明:
  「表の表示する幅を『ピクセル数』または『%』で指定する。横方向にスクロール可能な最大幅を100%とする。…」(p.124)
td要素の説明:
 「1セルの横幅は300Pixelである。」(p.132)
要素の入れ子関係:
 td→centerは禁止される入れ子関係(動作は保証されない)(p.250)
-----

comment:8 Changed 12 years ago by ebihara

Owner: changed from nobody to ebihara
Status: newassigned

以下の方針で実装したいと思います。

  • SoftBank? の 3G 端末
    • 変更なし
  • SoftBank? の 3G 以前の端末
    • 拡張HTTPヘッダの x-jphone-display から表示可能幅を取得する
    • Smartyのフィルターで、テンプレート内の <table width="100%" の 100% を 取得した幅に置換する

comment:9 Changed 12 years ago by ebihara

Keywords: 確認中 added

以下のリビジョンで修正しました。ご確認ください。

comment:10 Changed 12 years ago by ogawa

Keywords: 確認中 removed

HTTP_X_JPHONE_DISPLAY から display_size が取得できなかった場合の処理を書いておいた方が安全でしょう。

comment:11 Changed 12 years ago by ebihara

Keywords: 確認中 added

以下のリビジョンで、x-jphone-displayが取得できなかった場合は置換をおこなわないように修正しました。

確認よろしくお願いします。

comment:12 Changed 12 years ago by ogawa

Keywords: 確認中 removed

$display_size[0] の値を見るべきでしょう。

comment:13 Changed 12 years ago by ebihara

Keywords: 確認中 added

以下のリビジョンで、$display_size[0]が数値かどうかチェックするようにしました。ご確認ください。

  • 2.10.x - r4746
  • trunk - 4747

comment:14 Changed 12 years ago by ebihara

修正リビジョンの指定をミスしてしまいました。リビジョンは以下です。

comment:15 Changed 12 years ago by ogawa

Keywords: 確認中 removed

正の整数値なので、ctype_digit() 通ったらintにキャストして0より大きいかどうか調べるとかやるともっと確実でしょうか。まあせっかくなので。

comment:16 Changed 12 years ago by ebihara

Keywords: 確認中 added

以下のリビジョンで ctype_digit() を使うように変更しました。ctype_digit()が通るのは正の整数値の数値文字列に限られるので、これ以上のチェックはおこなっていません。

確認よろしくお願いします。

comment:17 Changed 12 years ago by ogawa

Keywords: 確認中 removed

0003とか通りますよねー

comment:18 Changed 12 years ago by ebihara

Keywords: 確認中 added

以下のリビジョンで修正しました。

comment:19 Changed 12 years ago by ebihara

以下のリビジョンで不要な括弧を削除しました。

comment:20 Changed 12 years ago by ogawa

Keywords: 確認中 removed

0はNGにした方がいいですよねー

comment:21 Changed 12 years ago by ebihara

Keywords: 確認中 added

以下のリビジョンで修正しました。ご確認ください。

  • 2.10.x - 4786
  • trunk - 4787

comment:22 Changed 12 years ago by ebihara

リビジョン番号の指定を間違えました。以下です。

comment:23 Changed 12 years ago by ogawa

Keywords: テスト待ち added; 確認中 removed

comment:24 Changed 12 years ago by kiwa

Keywords: テスト待ち removed
Resolution: fixed
Status: assignedclosed

現象が発生する端末が無いため動作テストが行えません。 今まで正常に表示されていた端末では問題なく動作しています。

comment:25 Changed 12 years ago by ebihara

Description: modified (diff)

comment:26 Changed 12 years ago by ebihara

Description: modified (diff)

comment:27 Changed 12 years ago by kiwa

Keywords: OpenPNE2.11.2 added
Note: See TracTickets for help on using tickets.