Ticket #1252 (closed バグ指摘: 修正済)

Opened 9 years ago

Last modified 8 years ago

商品一覧 - ページ送りナビのHTML構文誤り

Reported by: habu Owned by: Seasoft
Priority: Milestone: EC-CUBE2.12.0
Component: フロント Version: 2.11.0
Keywords: Cc:
修正済み: yes

Description (last modified by Seasoft) (diff)

商品一覧内、ページ送りナビ内の各要素が<li>タグで構成されていますが、
ところどころ抜けがあります。

また、/data/class/SC_PageNavi.php内にも、
/html/products/list.php内にも、
開始の<ul>タグがありませんでした。

 http://xoops.ec-cube.net/modules/newbb/viewtopic.php?topic_id=7776&forum=13&post_id=37724

リストを使う必要は必ずしもないと思いますので、フラットな HTML に書き換えようと思います。

r21313

Attachments

before.gif Download (2.2 KB) - added by habu 9 years ago.
変更前
after.gif Download (2.3 KB) - added by habu 9 years ago.
変更後
fixed.gif Download (2.2 KB) - added by habu 9 years ago.
CSS修正後

Change History

comment:1 Changed 9 years ago by habu

  • Owner changed from somebody to habu
  • Status changed from new to assigned

Changed 9 years ago by habu

変更前

Changed 9 years ago by habu

変更後

comment:2 follow-up: ↓ 4 Changed 9 years ago by habu

  • Owner habu deleted
  • Status changed from assigned to new

抜けているliタグを追加して、ulタグ(class="navi")で囲む様にすると、
スマートフォンにて、表示が崩れます。
(縦に表示される様になります)

これを回避するため、
html/user_data/packages/sphone/css
に以下のコードを追記すると、とりあえず横に表示される様になります。

.navi li a{
display:inline;
}

しかし、それでも、元々の表示とは異なる見た目になってしまいますので、 この修正をコミットするためには、更なるCSSの修正も必要です。

変更前:
変更前
変更後:
変更後

comment:3 Changed 9 years ago by kajiwara

  • Milestone changed from EC-CUBE2.11.1 to EC-CUBE2.11.2

habuさん

ご指摘と調査、ありがとうございます。

スマホのフロント画面は、小手先で修正してしまうと、色々と問題になりそうですね。 少し、こちら側でもCSSの修正も含めて、再度見直しを行ってみます。
ただし、2.11.1までには間に合わなさそうですので、2.11.2あたりでの修正になると思います。

申し訳ございませんが、マイルストーンを2.11.2に変更します。

Changed 9 years ago by habu

CSS修正後

comment:4 in reply to: ↑ 2 Changed 9 years ago by habu

  • Owner set to habu
  • Status changed from new to assigned

comment:5 Changed 9 years ago by habu

  • 修正済み set

kajiwaraさん

マイルストーンのご調整等、ありがとうございます。
r20915 にて修正してみました。
ご確認の程、宜しくお願い致します。

以下の様な画面です。
CSS修正後
iPhone 3GSだと、ページ送りが少し押しにくかったので
若干文字を大きくしています。

comment:6 Changed 9 years ago by habu

  • 修正済み unset

管理側の「商品選択」画面にも影響する事が分かりました。

もう少し、影響範囲を精査する必要がありそうです。

comment:7 Changed 9 years ago by habu

  • Owner changed from habu to somebody
  • Status changed from assigned to new

影響範囲が大きい事が分かったため、
r20915 にて、r20915 をロールバックしました。

影響範囲は以下の通りです。

【管理画面】
・おすすめ商品管理 商品検索(ポップアップ)
・受注登録 - 顧客検索(ポップアップ)
・受注登録 - 商品の追加画面(ポップアップ)
・ステータス管理
・商品管理>商品並び替え
・商品管理>商品登録 - 関連商品(ポップアップ)
・システム設定>メンバー管理

【フロント】
・商品一覧ページ
・MYページ - お気に入り一覧
・MYページ - 購入履歴一覧
・【携帯】商品検索結果
・【携帯】MYページ - 購入履歴一覧
・【スマホ】商品一覧ページ
・(【スマホ】MYページ - 購入履歴一覧 は、何故かページ送り無し)
・【スマホ】MYページ - お気に入り一覧

(「$tpl_strnavi」、「->strnavi」でGREPして調査しました)

PC、スマートフォンについては、CSSの修正で対応可能ですが、
携帯版についてはCSSでの修正が効かない端末もあるので、この度は断念しました。

comment:8 Changed 9 years ago by kajiwara

habuさん

色々、ありがとうございます。

ちなみに、スマートフォンは、2.11.2で、ページ送り機能無くします。
「もっと見る」のような機能への変更を検討しています。

ご報告まで。

comment:9 Changed 9 years ago by kajiwara

  • Status changed from new to closed
  • 修正済み set
  • Resolution set to 修正済

スマートフォンのリニューアルにて「もっと見る」機能で変わって対応しました。

本チケットはこちらにてクローズさせていただきます。

comment:10 Changed 9 years ago by Seasoft

  • Status changed from closed to reopened
  • 修正済み unset
  • Resolution 修正済 deleted
  • Description modified (diff)
  • Milestone changed from EC-CUBE2.11.2 to EC-CUBE 2.11.5(仮)

現状でも修正されていないようです。
 http://xoops.ec-cube.net/modules/newbb/viewtopic.php?topic_id=9245&forum=8

また /admin/products/product_select.php で、表示乱れが発生しているようです。

comment:11 Changed 9 years ago by Seasoft

  • Owner changed from somebody to Seasoft
  • Status changed from reopened to new
  • Summary changed from 商品一覧 - ページ送りナビのulタグliタグが抜けているところがある to 商品一覧 - ページ送りナビのHTML構文誤り

comment:12 Changed 9 years ago by Seasoft

  • Status changed from new to assigned
  • 修正済み set
  • Description modified (diff)

comment:13 follow-up: ↓ 14 Changed 9 years ago by kotani

  • Milestone changed from EC-CUBE2.11.5 to EC-CUBE2.12.0alpha

2_11-devブランチのコミットキャンセルが必要

comment:14 in reply to: ↑ 13 Changed 9 years ago by Seasoft

kotani への返信

2_11-devブランチのコミットキャンセルが必要

r21411 で対応しました。

comment:15 Changed 8 years ago by kajiwara

  • Status changed from assigned to closed
  • Resolution set to 修正済

本件、 r21313 にて実装済のためクローズさせていただきます。

Note: See TracTickets for help on using tickets.