BugTrack/2126

Category:Design

リスト構造の左マージンに関して + typo†

ページ: BugTrack
投稿者: 0
優先順位: 低
状態: 完了
カテゴリー: 本体新機能
投稿日: 2005-11-25 (金) 16:38:48
バージョン: 1.4.6
リリース予定バージョン: 1.5.2

↑

メッセージ†

ざっと調べてみましたが、既出ならすみません。

別に新機能ってわけでもなく、単なる疑問と提案です。(カテゴリこれでいいんですかね?)

リストのデザインに関して
- default.ini.php
```
/////////////////////////////////////////////////
// リスト構造の左マージン
$_ul_left_margin = 0;   // リストと画面左端との間隔(px)
$_ul_margin = 16;       // リストの階層間の間隔(px)
$_ol_left_margin = 0;   // リストと画面左端との間隔(px)
$_ol_margin = 16;       // リストの階層間の間隔(px)
$_dl_left_margin = 0;   // リストと画面左端との間隔(px)
$_dl_margin = 16;        // リストの階層間の間隔(px)
$_list_pad_str = ' class="list%d" style="padding-left:%dpx;margin-left:%dpx"';
```
  - これはリストの見栄えを簡単に設定するのが目的でしょうか?個人的にはCSSで指定するのが良いと思います。CSSで指定しても“padding-left, margin-left”が無視されるので困ったものです*1。
- menu.inc.php
```
return preg_replace('/<ul[^>]*>/', '<ul>', convert_html($menutext));
```
  - 例えば“recent.inc.php”の“<ul class="recent_list">”のclassが消えます。恐らくこの置き換えは“リスト構造の左マージン”と絡んでいるのではと思います。その他の不具合が思いつかないのですが、あるんでしょうか?多くの場合メニューバーで使用するrecent.inc.phpのclassが消えてしまうので (^^;
ついでに、typoです
- lib/file.php 80行目付近
```
// Modify ogirinal text with user-defined / system-defined rules
```
  - original ですね。

使用者側で簡単に変更できるので特に重要な事項ではありませんが、個人的にものすごく気になるところです。

default.ini.php で設定している左マージンのコントロールについてですが、リストで無茶なレベル指定(例: 3→1→3→2)がされた場合に、妥当なXHTMLを吐きつつも、レベルに合わせた見栄えに整える為に存在していた筈です。開発日記/2003-03-29が参考になるかもしれません。 menu.inc.phpの方はCVSを追いかけたものの分かりませんでした。これはplugin化される前のskinにあったものが引き継がれているようで、skinにはpukiwiki.skin.ja.php の1.15→1.16 で追加されていますが、コメントや開発日記等に記述は見つけられませんでした。 -- にぶんのに 2005-11-26 (土) 00:18:26
- お答えありがとうございます! なるほど、開発日記はそのものズバリですね。*2見出しやリストのレベルを飛ばせば見た目が崩れますね。個人的には“list1”と“2･3”のmargin/paddingを変えたいので(16 * n)ではなく、(16 * (n - 1) + list1の指定)とできればありがたいんですけどね*3。しかし2年半前の議論か・・・ :( -- 0 2005-11-26 (土) 11:24:07
とりあえず、typo修正が終われば保留･却下ですね。 -- 0 2005-11-26 (土) 11:27:43
- typoについて修正しました。cvs:lib/file.php (1.45) -- henoheno 2006-01-09 (月) 19:33:01
BugTrack2/188 -- 2006-07-16 (日) 01:33:33
BugTrack/2188が完了になったのでこちらも完了にします -- umorigu 2017-08-17 (木) 03:03:37
「1. リストのデザインに関して」 - menu.inc.php の実装がそのままなので、保留に変更します。地味ですがこちらの方が影響が大きいかも... -- umorigu 2017-08-17 (木) 08:11:59
```
return preg_replace('/<ul[^>]*>/', '<ul>', convert_html($menutext));
```
menu.inc.php で ul要素の属性を削除している個所を削除しました。 commit:53970ad677 結局のところ、通常List styleよりも優先度の高い div#menubar ul 指定が既にあったので、Menu上のListの見た目は変わりませんでした -- umorigu 2017-08-18 (金) 00:05:09

total:
today:
yesterday:
now: 1

リスト構造の左マージンに関して + typo†

メッセージ†

最新の16件