Project

General

Profile

Bug(バグ) #1785

[PATCH] [仕様バグ] テキストボックス・テキストエリアのサイズが適切でない

Added by Kiwa Sakai about 9 years ago. Updated over 7 years ago.

Status:
Rejected(差し戻し)
Priority:
Normal(通常)
Target version:
Start date:
2010-11-16
Due date:
% Done:

50%

3.6 で発生するか:
Yes
3.8 で発生するか:
Unknown (未調査)

Description

Overview (概要)

テキストボックス・テキストエリアのサイズが小さく、文字が入力しづらい。
OpenPNE2に合わせ、デフォルトのサイズを以下のように指定してほしい。

  • テキストボックス: size=40
  • テキストエリア: cols=50 rows=10

Causes (原因)

出力時にサイズを指定していない。

Way to fix (修正手段)

History

#1 Updated by Kousuke Ebihara about 9 years ago

  • Target version set to OpenPNE 3.7.0

#2 Updated by Kousuke Ebihara about 9 years ago

  • Priority changed from Normal(通常) to High(高め)

#3 Updated by Shogo Kawahara almost 9 years ago

  • Status changed from New(新規) to Accepted(着手)
  • Assignee set to Shogo Kawahara

#4 Updated by Shogo Kawahara almost 9 years ago

  • Subject changed from [仕様バグ] テキストボックス・テキストエリアのサイズが適切でない to [PATCH] [仕様バグ] テキストボックス・テキストエリアのサイズが適切でない

#5 Updated by Shogo Kawahara almost 9 years ago

  • Status changed from Accepted(着手) to Pending Review(レビュー待ち)
  • % Done changed from 0 to 50

https://github.com/kawahara/OpenPNE3/commit/bfa4707f105b2558b580af666dcb6ce5d3d12b29

未マージです。懸念事項があるのでチェックをお願いします。

#6 Updated by Minoru Takai over 8 years ago

参考程度にコメントをしておきます。以下の考察に基づいて修正が可能か、あるいは以下の考察が間違っているようであれば対応・指摘等、どなたかお願いします。

問題を解消できているか

テキストボックス(テキストフィールド:HTMLにおける input[type="input"] 要素)とテキストエリア(HTMLにおける textarea 要素)に対してサイズの属性を与える修正としては、修正に誤りなどはないと判断しています。そういう意味ではコードチェックはOKかと思います(が、懸念事項を考慮する必要があります)。

属性値がそれぞれ size=40, cols=50, rows=10 でよいかについても、2系に合わせるという観点からは問題はないと思います。

この修正による副作用がないか

この修正の懸念事項として、widgetで指定していた幾つかの attributes (size, col, rows) を無効化してしまう可能性がある。

と(チケットの概要で)示されていますが、属性値を上書きしてしまうのであれば、この懸念は考慮しなければなりません。

widget で属性が指定されていない場合に限り、デフォルトとなる属性値(size=40 および cols=50, rows=10)を与えるようなことができれば、この問題に対する修正としては適切ではないかと思っています。

「widget で属性が指定されていない場合に限り、デフォルトとなる属性値を与えることができるか」についてはまだ調査していないため、コメントのみを残しておきます。

#7 Updated by Shingo Yamada over 8 years ago

  • 360対象 set to beta13

#8 Updated by Maki Takahashi over 8 years ago

「widget で属性が指定されていない場合に限り、デフォルトとなる属性値を与えることができるか」について、実際にテストしてみました。

opBlogPluginのPC版「BlogのURL設定(member/config?category=blogUrl)」にて、
MemberConfigBlogUrlFormでは「size=57」と属性指定されていますが、上記修正を取り込んだ状態では「size=40」となっています。

普通にインストールした状態で属性指定しているのは、上記とopIntroFriendPluginのPluginIntroFriendFormのみのようですが
IntroFriendの方は、_partsForm.phpを利用していないため、指定した属性('rows' => 8, 'cols' => 88)が反映しています。

#9 Updated by Maki Takahashi over 8 years ago

  • Status changed from Pending Review(レビュー待ち) to Rejected(差し戻し)

#10 Updated by Shingo Yamada over 8 years ago

サイズ指定が上書きされる問題の回避措置についての検討

当初考えていた実装方針

  • サイズの指定の有無をテンプレート側で判別し、サイズの指定がない場合はデフォルト値を設定する。
上記実装方針の問題点

sfWidgetFormTextareaのconfigureでrows, colsのデフォルトを指定しているため、OpenPNEの実装者が指定した値なのかどうかの区別がつかない

「OpenPNEの実装者が指定した値なのかどうかの区別がつかない」問題の回避策

  • デフォルト値を設定しているクラスのサブクラスを作り対応
補足事項
  • partsForm を使用していない箇所の対応についての検討も必要

#11 Updated by Shingo Yamada over 8 years ago

以下の理由で優先度を下げます(対応バージョンも変更)。
  • 優先度が「high」である理由が不明確
  • 対応するにしてもマイナーバージョンアップでも可能なないようであることが予想される

#12 Updated by Shingo Yamada over 8 years ago

  • 360対象 deleted (beta13)

#13 Updated by Kousuke Ebihara about 8 years ago

  • Priority changed from High(高め) to Normal(通常)

優先度が「high」である理由が不明確

ということなので、 Normal に戻します。

#14 Updated by Shouta Kashiwagi over 7 years ago

  • Target version changed from OpenPNE 3.7.0 to 252

#15 Updated by Shouta Kashiwagi over 7 years ago

  • Target version changed from 252 to OpenPNE 3.8.x

Also available in: Atom PDF