Ticket #518 (closed 改善提案: 修正済)
ブロックを置ける場所を増やしてみた
Reported by: | miningbrownie | Owned by: | miningbrownie |
---|---|---|---|
Priority: | 中 | Milestone: | EC-CUBE2.11.0 |
Component: | 管理画面 | Version: | コミュニティ (eccube-comu) |
Keywords: | ブロック、デザイン管理 | Cc: | |
修正済み: |
Attachments
Change History
Changed 14 years ago by miningbrownie
-
attachment
addBlocArea.patch
added
comment:1 Changed 14 years ago by miningbrownie
- Status changed from new to closed
- Resolution set to 修正済
- Description modified (diff)
comment:2 Changed 14 years ago by nanasess
#494 を開発中ですが, 全く同じような機能を作りかけていました.
パッチいただきます. ありがとうございます.
僕の場合は, ヘッダー or フッターにブロックの配置スペースを作っていました.
そこでちょっと相談なのですが, ヘッダー or フッターに配置スペースを作っても, レイアウト管理で配置可能なページにしか適用されません.
ヘッダー or フッターに配置した場合は, すべてのページに適用したいのですが, 良い仕様のアイディアはありませんでしょうか?
「TOP に配置したら, すべてのページに適用される」といった感じでしたら, 簡単にできるのですが, イマイチ美しくないですし...
comment:4 Changed 14 years ago by miningbrownie
nanasessさん コメントありがとうございます。 ヘッダーorフッターに配置で全ページにですかぁ、、、 dtb_blocpositionをどう変えるかが肝になってきそうですね。 -dtb_blocpositionにanywhereというフラグを作る。 -レイアウト設定でブロックにチェックボックスを作って、それによってanywhere=1にする。 -表示時はSC_Helper_PageLayoutで、レイアウト設定にあるなしに関わらず(dtb_blocpostionのpage_id=?をanywhere = 1 or page_id = ?)、表示してしまうとかはどうでしょうか?
ためしに書いてみたので、添付してみます。
comment:5 Changed 14 years ago by nanasess
早速ありがとうございます!
とても良い感じです.
おかげさまで, ちょっと考え中だった個所が一気に解決しそうです.
あと, PostgreSQL ではフラグは int2 を使用しているので, 下記のようにした方が良いかもしれません.
-
html/install/sql/create_table_pgsql.sql
928 928 target_id int4, 929 929 bloc_id int4, 930 930 bloc_row int4, 931 filename text 931 filename text, 932 anywhere int2 932 933 ); 933 934 934 935 CREATE TABLE dtb_csv (
Changed 14 years ago by miningbrownie
-
attachment
addMoreBlocArea.patch
added
<head>内、ヘッダの上下、フッタの上下OK!
comment:7 Changed 14 years ago by Seasoft
- Status changed from closed to reopened
- Resolution 修正済 deleted
なお、フォーラムにて、r18190 と r18194 に不具合報告があります。
http://xoops.ec-cube.net/modules/newbb/viewtopic.php?topic_id=4277&forum=9
shasharaku 様による報告 (抜粋)
[EC-CUBE] r18192 [OS] Linux [PHP] PHP 5.1.6 [データベース] MySQL 5.0.22 [WEBサーバ] apache2.2.3 [ブラウザ] Firefox3 お世話になります。 上記の環境で問題の箇所は正常に表示できました。 が、プレビューボタンを押すと http://○○○○○/admin/design/? が別ウインドウで開き、エラー表示されます。
「チェンジセット 18190」を除外し 再インストールしたところ エラーはなくなりました。 何かが干渉しているようです。
ramrun 様による報告 (抜粋)
r18194を新規にインストールして確認しましたが、なんだかブロックの設定がずれてるっぽいですね(汗)。 プレビューでシステムエラー、登録するとブロックが消えたり飛んだりしています。
comment:8 Changed 14 years ago by miningbrownie
seasoftさま
まとめありがとうございます。 先ほど上記の件修正し、コミットしておきました。 r18202 です。
原因は、SC_Helper_Pagelayout.phpでのtarget_idとその他でのIDのヅレと insert_data.sqlでの配置スペースの名前の付け方が間違っていました。
現在は修正済みです。
comment:9 Changed 14 years ago by Seasoft
新たな不具合箇所を見つけましたが、ブロックのオブジェクトで、従来も同様の症状が発生していたと思いますので、新たなチケットを発行しておきました。
comment:10 follow-up: ↓ 15 Changed 14 years ago by Seasoft
IE6 での表示乱れが大きいようです。ハードコピー添付します。
comment:11 follow-up: ↓ 16 Changed 14 years ago by Seasoft
「 (□全ページ) 」の部分ですが、改行禁止にした方が見やすい気がします。
comment:12 Changed 14 years ago by Seasoft
comment:13 follow-up: ↓ 14 Changed 14 years ago by Seasoft
欄外(Unused)の値が頻繁に変更されているのが気になりました。(アップグレードで支障が発生するケースの懸念があります。)
下記のいずれかの対応を行なうと良いのではないかと感じました。
- IDを「0」や「99」にして、影響を受けにくくする
- dtb_blocposition への登録を不要とする
時間が出来たら、当方にて後者の改訂を行なってみたいと考えています。
comment:14 in reply to: ↑ 13 Changed 14 years ago by Seasoft
r18216 で対応しました。 結局、適用箇所にあわせて、両方の対応を行ないました。
comment:15 in reply to: ↑ 10 Changed 14 years ago by Seasoft
IE6 での表示乱れが大きいようです。ハードコピー添付します。
r18322 で改修してみました。
comment:16 in reply to: ↑ 11 Changed 14 years ago by Seasoft
「 (□全ページ) 」の部分ですが、改行禁止にした方が見やすい気がします。
r18322 で改修してみました。
comment:17 in reply to: ↑ description Changed 13 years ago by Seasoft
「全ページ」に複数チェックを入れると、プレビューでSQLエラーとなるようです。
comment:19 Changed 13 years ago by Seasoft
comment:20 Changed 13 years ago by nanasess
r19663 でヘッダ内にもブロックを配置できるようにしました.
開発合宿 Aチームの成果物です.
comment:22 Changed 12 years ago by nanasess
- Status changed from reopened to closed
- Resolution set to 修正済
Rev:18188とのdiff