BugTrack/2266
をテンプレートにして作成
[
トップ
] [
新規
|
一覧
|
検索
|
最終更新
|
ヘルプ
|
ログイン
]
開始行:
* 編集中に「テキスト整形のルールを表示」しても、メッセー...
- ページ: [[BugTrack]]
- 投稿者: [[ぃぉぃぉ]]
- 優先順位: 低
- 状態: 完了
- カテゴリー: 本体新機能
- 投稿日: 2007-08-04 (土) 10:35:44
- バージョン: 1.4.7
- リリース予定バージョン: 1.5.2
----
#contents
----
**編集フォームの「テキスト整形のルールを表示する」をPOST...
使い始めた頃、編集し始めてから「表はどう記載するんだっけ...
周り(パソコンに不慣れな人たち)に自分のPukiWikiを紹介して...
#ref(BugTrack2-266_ioio_01_buttonA.png,nolink)
#br
#ref(BugTrack2-266_ioio_01_buttonB.png,nolink)
***フォーム生成部改造 [#zf9d8829]
-html.php
--function edit_form()
---L.166あたり
- global $_btn_preview, $_btn_repreview, $_btn_update, $...
+ global $_btn_preview, $_btn_repreview, $_btn_update, $...
---L.240あたり&br;
//value=$_msg_help(_off)を""でくくり忘れていたので修正。(...
$vars['helpoff']を使用しない様に修正。代わりに$vars['help...
+ // Is help shown?
+ $help_on = isset($vars['help_on']) ? $vars['help_on'] ...
+ // If help button is clicked, change help showing state.
+ $help_on = ($help_on xor isset($vars['help']));
+ // Show help if (help showing and help button is not p...
+ if ($help_on) {
+ $help_button = "<input type=\"submit\" name=\"help\"...
+ } else {
+ $help_button = "<input type=\"submit\" name=\"help\"...
+ }
+
$body = <<<EOD
<div class="edit_form">
<form action="$script" method="post" style="margin-bot...
$template
$addtag
<input type="hidden" name="cmd" value="edit" />
<input type="hidden" name="page" value="$s_page" />
<input type="hidden" name="digest" value="$s_digest" />
<textarea name="msg" rows="$rows" cols="$cols">$s_pos...
<br />
<div style="float:left;">
<input type="submit" name="preview" value="$btn_prev...
<input type="submit" name="write" value="$_btn_upd...
$add_top
$add_notimestamp
- </div>
<textarea name="original" rows="1" cols="1" style="di...
- </form>
- <form action="$script" method="post" style="margin-top...
- <input type="hidden" name="cmd" value="edit" />
- <input type="hidden" name="page" value="$s_page" />
<input type="submit" name="cancel" value="$_btn_cance...
+ </div>
+ <br />
+ <div style="clear:left;">
+ $help_button
+ <input type="hidden" name="help_on" value="$help_on" />
+ </div>
</form>
</div>
EOD;
- if (isset($vars['help'])) {
- $body .= $hr . catrule();
- } else {
- $body .= '<ul><li><a href="' .
- $script . '?cmd=edit&help=true&page=' . $r_...
- '">' . $_msg_help . '</a></li></ul>';
- }
-
- return $body;
+ if ($help_on) {
+ $body .= $hr . catrule();
+ }
return $body;
***edit.inc.php呼び出し時に表示するtextareaの中身決定。 [...
-global
- global $vars, $_title_edit, $load_template_func;
+ global $post, $vars, $_title_edit, $load_template_func;
-L.30あたり&br;
プレビューかフォーマットルール表示の場合には引数から渡さ...
+ if (!isset($post['msg'])) {
$postdata = @join('', get_source($page));
if ($postdata == '') $postdata = auto_template($page);
+ } else {
+ $postdata = $post['msg'];
+ }
***ボタン表示切り替えに、消すときのメッセージ作成 [#gd8eb...
-ja.lng.php l.57あたり
$_msg_help = 'テキスト整形のルールを表示する';
+$_msg_help_off = 'テキスト整形のルールを隠す';
$_msg_week = array('日','月','火','水','木','...
-en.lng.php l.54あたり
$_msg_help = 'View Text Formatting Rules';
+$_msg_help_off = 'Hide Text Formatting Rules';
$_msg_week = array('Sun','Mon','Tue','Wed','Thu'...
--------
**コメント [#kf831b9b]
- こんにちは :) 確かに私もついクリックしてしまい、「戻る...
-- 失礼しました (^^; クリックするまでの話ではなく、クリッ...
-- 挙動的にはプレビューの亜種なのだなあ。 -- [[henoheno]]...
-- そうですね。送るデータはsubmitがpreviewかhelp関係かと...
- 自分の場合はついついクリックしたのではなく、Formatting ...
-- 最近は消えることが分かっていますから、shift+クリックで...
--- これならツールバー(? 最上部のメニュー)や下部のアイコ...
-- つまり、現状では編集モードに入った''直後''に「テキスト...
--- 一度プレビュー表示をした後なら、プレビュー表示に「戻...
--- プレビューの後でも、プレビューの''直後''しかだめです...
-- ということで、このBugTrackのように、「テキスト整形のル...
- アイデアだけな関連 : [[BugTrack/570]]の注釈
- 今までの実装が $vars['help'] を見ているということは、cm...
-- 今でもL.240あたりの「+ if (isset($vars['helpoff']) || ...
- なんだかシングルクォートがエスケープされてました。spam...
- とりあえず、こちらにデモページを%%用意しました%%設置し...
-- 助かります。ページの先頭に、外見と動作が理解できるよう...
--- 動作確認ありがとうございました。&br;画像作成、添付あ...
- edit.inc.php の部分ですが、msgを POSTのみから得ることを...
-- [[cvs:plugin/edit.inc.php]] (r1.42)
- この機能は本体に組み込まれる可能性はありそうでしょうか?...
-- [[ぃぉぃぉ]]さんが話されたアイデアやその発想はすぐに理...
-- 了解です。私が提案したまんまの仕様では組み込まれる可能...
- 随分細かいやり取りが続いていますが、こうした部分がクリ...
- どこに書けばいいかわからなかったので、必要に応じて移動...
-- 「整形ルールを表示する」ボタンの挙動とは直接関係がなさ...
- これがうまくいけば、[[追加で整形ルールを表示すると編集...
-- addもedit_form()を使用しているので「ヘルプを呼び出すと...
- シンプルに、「テキスト整形のルール」ページを別ウィンド...
#comment
** コメント: 資源の消費量 (理解するコストを減らし、結果的...
- html.phpの差分は、$vars['helpoff'] の無い、よりシンプル...
-- [[cvs:../devel/followup_html]] (1.1)
-- デモ作成お疲れ様です。拝見致しました。なるほど、すっき...
-- PHPマニュアルの変数の項を参照下さい。初期化さえしっか...
-- マニュアル見てみました。とりあえずNoticeも何も出ていな...
- こちらの(というか、そちらのお手元の)ソース、$vars['help...
- $vars['help_on']を使うようにしました。$vars['showhelp']...
- お疲れ様です。こちらの件、もう少し簡素にできないでしょ...
$help_on = isset($vars['help_on']) ? $vars['help_on'] : ...
$help_on = $help_on ^ isset($vars['help']);
-- 一行目でstringかbool(FALSE)が変数に入っていますから、F...
$help_on = isset($vars['help_on']) ? (bool)$vars['help_o...
$help_on = (isset($vars['help']) xor $help_on); // bool ...
-- 論理演算子とbit演算子を区別していませんでした。失礼し...
-- どうして [[cvs:../devel/followup_html/form_submit.php]...
-- boolの値を収めた変数をechoしたとしてもユーザーに届く値...
#comment
** コメント: ユーザーインターフェース同士の関係性 [#v16b...
- 「プレビューのボタン」と、「プレビューの亜種であるとこ...
-- 「一つのボタンと二つのチェックボックス」で表現は簡単そ...
-- 機能ごとにボタンが一つあるというのは確かにそこだけ見る...
-- レイアウトについてはこんなイメージでいました。集約させ...
[ リロード ] □プレビュー □整形ルール [ ページの更新 ] □...
[ キャンセル ]
-- 「整形ルールを表示する」というのをわかりやすく省略する...
□タイムスタンプを維持[ ページの更新 ]
□プレビューON □整形ルール表示 [ 実行 ]
[ キャンセル ]
というのは考えてみましたが、この長さなら現状の方がよいな...
#comment
** コメント: 抽象化による展開例: 衝突検知など [#l911d7bf]
- 「プレビューのボタン」と、「プレビューの亜種であるとこ...
-- なるほど、編集の衝突のチェックですか。衝突対策の良い案...
-- 衝突検知は今でもできる((だから衝突した事を知らせる事だ...
-- 衝突検知に関しては、確かに実用的なものというのは難しい...
#comment
** メタコメント: 既存実装とのバッティング (cancel時のpayl...
- lib/html.php の edit_form() で、cancelが本文をPOSTしな...
-- 動作的にはplugin/edit.inc.phpで対応済みの様です。$vars...
-- お疲れ様です。BugTrack2/160 の工夫 というのは、"「キャ...
-- なるほど。cancel時はPOSTデータのほとんどが不要というこ...
+++「テキスト整形のルールを表示する」ボタンをCANCELボタン...
+++CANCELをボタンではなくリロードと同じリンクとする&br;
という方法が考えられますね。自分としてはCANCELボタンの使...
- ふむふむ、考え方が理解できました。edit_form の payload ...
-- よろしくお願いします。 -- [[ぃぉぃぉ]] &new{2007-08-19...
- こちらの件、上に画像で提示したレイアウトから考えるに、...
-- 少し上のa.「テキスト整形のルールを表示する」ボタンを(...
-- ? (^^; あ、そうか・・・ちょっと適当言ってたみたいです...
-- 了解っす^^; -- [[ぃぉぃぉ]] &new{2007-08-30 (木) 01:50...
#comment
** メタコメント: パッチ(差分)に関して [#z0182e6c]
- 見た目を含めて検討したかったのですが、上記のパッチはそ...
-- 元になっているのはPukiWiki Ver.1.4.7_notbです。&br;$he...
--- はい、$help_on はこのパッチで用意された変数ですよね。...
if ($help_on) {
$body .= $hr . catrule();
}
--- [[フォーム生成部改造のhtml.phpのfunction edit_form() ...
--- はい、そしてそのパッチの最後の部分にある、 + でも - ...
--- 失礼しました。その3行も+です。(修正しました) -- [[ぃ...
#comment
終了行:
* 編集中に「テキスト整形のルールを表示」しても、メッセー...
- ページ: [[BugTrack]]
- 投稿者: [[ぃぉぃぉ]]
- 優先順位: 低
- 状態: 完了
- カテゴリー: 本体新機能
- 投稿日: 2007-08-04 (土) 10:35:44
- バージョン: 1.4.7
- リリース予定バージョン: 1.5.2
----
#contents
----
**編集フォームの「テキスト整形のルールを表示する」をPOST...
使い始めた頃、編集し始めてから「表はどう記載するんだっけ...
周り(パソコンに不慣れな人たち)に自分のPukiWikiを紹介して...
#ref(BugTrack2-266_ioio_01_buttonA.png,nolink)
#br
#ref(BugTrack2-266_ioio_01_buttonB.png,nolink)
***フォーム生成部改造 [#zf9d8829]
-html.php
--function edit_form()
---L.166あたり
- global $_btn_preview, $_btn_repreview, $_btn_update, $...
+ global $_btn_preview, $_btn_repreview, $_btn_update, $...
---L.240あたり&br;
//value=$_msg_help(_off)を""でくくり忘れていたので修正。(...
$vars['helpoff']を使用しない様に修正。代わりに$vars['help...
+ // Is help shown?
+ $help_on = isset($vars['help_on']) ? $vars['help_on'] ...
+ // If help button is clicked, change help showing state.
+ $help_on = ($help_on xor isset($vars['help']));
+ // Show help if (help showing and help button is not p...
+ if ($help_on) {
+ $help_button = "<input type=\"submit\" name=\"help\"...
+ } else {
+ $help_button = "<input type=\"submit\" name=\"help\"...
+ }
+
$body = <<<EOD
<div class="edit_form">
<form action="$script" method="post" style="margin-bot...
$template
$addtag
<input type="hidden" name="cmd" value="edit" />
<input type="hidden" name="page" value="$s_page" />
<input type="hidden" name="digest" value="$s_digest" />
<textarea name="msg" rows="$rows" cols="$cols">$s_pos...
<br />
<div style="float:left;">
<input type="submit" name="preview" value="$btn_prev...
<input type="submit" name="write" value="$_btn_upd...
$add_top
$add_notimestamp
- </div>
<textarea name="original" rows="1" cols="1" style="di...
- </form>
- <form action="$script" method="post" style="margin-top...
- <input type="hidden" name="cmd" value="edit" />
- <input type="hidden" name="page" value="$s_page" />
<input type="submit" name="cancel" value="$_btn_cance...
+ </div>
+ <br />
+ <div style="clear:left;">
+ $help_button
+ <input type="hidden" name="help_on" value="$help_on" />
+ </div>
</form>
</div>
EOD;
- if (isset($vars['help'])) {
- $body .= $hr . catrule();
- } else {
- $body .= '<ul><li><a href="' .
- $script . '?cmd=edit&help=true&page=' . $r_...
- '">' . $_msg_help . '</a></li></ul>';
- }
-
- return $body;
+ if ($help_on) {
+ $body .= $hr . catrule();
+ }
return $body;
***edit.inc.php呼び出し時に表示するtextareaの中身決定。 [...
-global
- global $vars, $_title_edit, $load_template_func;
+ global $post, $vars, $_title_edit, $load_template_func;
-L.30あたり&br;
プレビューかフォーマットルール表示の場合には引数から渡さ...
+ if (!isset($post['msg'])) {
$postdata = @join('', get_source($page));
if ($postdata == '') $postdata = auto_template($page);
+ } else {
+ $postdata = $post['msg'];
+ }
***ボタン表示切り替えに、消すときのメッセージ作成 [#gd8eb...
-ja.lng.php l.57あたり
$_msg_help = 'テキスト整形のルールを表示する';
+$_msg_help_off = 'テキスト整形のルールを隠す';
$_msg_week = array('日','月','火','水','木','...
-en.lng.php l.54あたり
$_msg_help = 'View Text Formatting Rules';
+$_msg_help_off = 'Hide Text Formatting Rules';
$_msg_week = array('Sun','Mon','Tue','Wed','Thu'...
--------
**コメント [#kf831b9b]
- こんにちは :) 確かに私もついクリックしてしまい、「戻る...
-- 失礼しました (^^; クリックするまでの話ではなく、クリッ...
-- 挙動的にはプレビューの亜種なのだなあ。 -- [[henoheno]]...
-- そうですね。送るデータはsubmitがpreviewかhelp関係かと...
- 自分の場合はついついクリックしたのではなく、Formatting ...
-- 最近は消えることが分かっていますから、shift+クリックで...
--- これならツールバー(? 最上部のメニュー)や下部のアイコ...
-- つまり、現状では編集モードに入った''直後''に「テキスト...
--- 一度プレビュー表示をした後なら、プレビュー表示に「戻...
--- プレビューの後でも、プレビューの''直後''しかだめです...
-- ということで、このBugTrackのように、「テキスト整形のル...
- アイデアだけな関連 : [[BugTrack/570]]の注釈
- 今までの実装が $vars['help'] を見ているということは、cm...
-- 今でもL.240あたりの「+ if (isset($vars['helpoff']) || ...
- なんだかシングルクォートがエスケープされてました。spam...
- とりあえず、こちらにデモページを%%用意しました%%設置し...
-- 助かります。ページの先頭に、外見と動作が理解できるよう...
--- 動作確認ありがとうございました。&br;画像作成、添付あ...
- edit.inc.php の部分ですが、msgを POSTのみから得ることを...
-- [[cvs:plugin/edit.inc.php]] (r1.42)
- この機能は本体に組み込まれる可能性はありそうでしょうか?...
-- [[ぃぉぃぉ]]さんが話されたアイデアやその発想はすぐに理...
-- 了解です。私が提案したまんまの仕様では組み込まれる可能...
- 随分細かいやり取りが続いていますが、こうした部分がクリ...
- どこに書けばいいかわからなかったので、必要に応じて移動...
-- 「整形ルールを表示する」ボタンの挙動とは直接関係がなさ...
- これがうまくいけば、[[追加で整形ルールを表示すると編集...
-- addもedit_form()を使用しているので「ヘルプを呼び出すと...
- シンプルに、「テキスト整形のルール」ページを別ウィンド...
#comment
** コメント: 資源の消費量 (理解するコストを減らし、結果的...
- html.phpの差分は、$vars['helpoff'] の無い、よりシンプル...
-- [[cvs:../devel/followup_html]] (1.1)
-- デモ作成お疲れ様です。拝見致しました。なるほど、すっき...
-- PHPマニュアルの変数の項を参照下さい。初期化さえしっか...
-- マニュアル見てみました。とりあえずNoticeも何も出ていな...
- こちらの(というか、そちらのお手元の)ソース、$vars['help...
- $vars['help_on']を使うようにしました。$vars['showhelp']...
- お疲れ様です。こちらの件、もう少し簡素にできないでしょ...
$help_on = isset($vars['help_on']) ? $vars['help_on'] : ...
$help_on = $help_on ^ isset($vars['help']);
-- 一行目でstringかbool(FALSE)が変数に入っていますから、F...
$help_on = isset($vars['help_on']) ? (bool)$vars['help_o...
$help_on = (isset($vars['help']) xor $help_on); // bool ...
-- 論理演算子とbit演算子を区別していませんでした。失礼し...
-- どうして [[cvs:../devel/followup_html/form_submit.php]...
-- boolの値を収めた変数をechoしたとしてもユーザーに届く値...
#comment
** コメント: ユーザーインターフェース同士の関係性 [#v16b...
- 「プレビューのボタン」と、「プレビューの亜種であるとこ...
-- 「一つのボタンと二つのチェックボックス」で表現は簡単そ...
-- 機能ごとにボタンが一つあるというのは確かにそこだけ見る...
-- レイアウトについてはこんなイメージでいました。集約させ...
[ リロード ] □プレビュー □整形ルール [ ページの更新 ] □...
[ キャンセル ]
-- 「整形ルールを表示する」というのをわかりやすく省略する...
□タイムスタンプを維持[ ページの更新 ]
□プレビューON □整形ルール表示 [ 実行 ]
[ キャンセル ]
というのは考えてみましたが、この長さなら現状の方がよいな...
#comment
** コメント: 抽象化による展開例: 衝突検知など [#l911d7bf]
- 「プレビューのボタン」と、「プレビューの亜種であるとこ...
-- なるほど、編集の衝突のチェックですか。衝突対策の良い案...
-- 衝突検知は今でもできる((だから衝突した事を知らせる事だ...
-- 衝突検知に関しては、確かに実用的なものというのは難しい...
#comment
** メタコメント: 既存実装とのバッティング (cancel時のpayl...
- lib/html.php の edit_form() で、cancelが本文をPOSTしな...
-- 動作的にはplugin/edit.inc.phpで対応済みの様です。$vars...
-- お疲れ様です。BugTrack2/160 の工夫 というのは、"「キャ...
-- なるほど。cancel時はPOSTデータのほとんどが不要というこ...
+++「テキスト整形のルールを表示する」ボタンをCANCELボタン...
+++CANCELをボタンではなくリロードと同じリンクとする&br;
という方法が考えられますね。自分としてはCANCELボタンの使...
- ふむふむ、考え方が理解できました。edit_form の payload ...
-- よろしくお願いします。 -- [[ぃぉぃぉ]] &new{2007-08-19...
- こちらの件、上に画像で提示したレイアウトから考えるに、...
-- 少し上のa.「テキスト整形のルールを表示する」ボタンを(...
-- ? (^^; あ、そうか・・・ちょっと適当言ってたみたいです...
-- 了解っす^^; -- [[ぃぉぃぉ]] &new{2007-08-30 (木) 01:50...
#comment
** メタコメント: パッチ(差分)に関して [#z0182e6c]
- 見た目を含めて検討したかったのですが、上記のパッチはそ...
-- 元になっているのはPukiWiki Ver.1.4.7_notbです。&br;$he...
--- はい、$help_on はこのパッチで用意された変数ですよね。...
if ($help_on) {
$body .= $hr . catrule();
}
--- [[フォーム生成部改造のhtml.phpのfunction edit_form() ...
--- はい、そしてそのパッチの最後の部分にある、 + でも - ...
--- 失礼しました。その3行も+です。(修正しました) -- [[ぃ...
#comment
ページ名: