Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

ツイート詳細とツイート入力ボックスを完全に分離 #12

Open
wants to merge 2 commits into
base: develop
Choose a base branch
from

Conversation

aokomoriuta
Copy link
Contributor

Twitter上のとあるユーザーさん(鍵アカウントのため詳細は開示不可)から報告をもらいました。
画像サムネイルを開いた時に発言ボックスが小さくなることが原因で、誤クリックが発生しやすいようです(正確にはクリックというよりパネルタッチに近い操作)。

バグとまでは言えませんが、「ツイート詳細に含まれるはずの画像サムネイルが発言部分に侵食してるのはしっくりこない」という彼の意見に個人的に賛同できる部分もあるので修正の上pull request送ります。

  • 単純にSplitoContainerの順序を入れ替えた
  • 区切りが分かりやすいように詳細と入力ボックスの間のSplitterのスタイルを3Dにした
    ぐらいが主な修正箇所になります。

本家の方でマージご検討いただければ幸いです。

Review on Reviewable

サムネイルを開いた時に発言ボックスが小さくなった時の誤クリック防止などのため
@ghost ghost assigned upsilon Jun 26, 2013
@@ -769,41 +769,21 @@ private void InitializeComponent()
// ProfilePanel
//
resources.ApplyResources(this.ProfilePanel, "ProfilePanel");
this.ProfilePanel.BorderStyle = System.Windows.Forms.BorderStyle.Fixed3D;
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

ProfilePanelは非表示かつどこにも使われていない状態ですが、BorderStyleを変更したのは何か意図があるのでしょうか?

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

すいません、手元ではそもそもProfilePanelは除去していたので(どこからも参照されていないようなので)気づいていませんでした。
むしろ何もないのであればProfilePanelを除去したほうが良いように思うのですが、いかがでしょうか。

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

ProfilePanel は 2df7ca1 で追加されてから放置されているようですね。消しても問題なさそうなので後ほど削除しておきます。

@upsilon
Copy link
Member

upsilon commented Jun 26, 2013

SplitContainer2 の BorderStyle についてですが、手元の環境 (Win8, 120dpi) で試したところ、一度投稿欄を広げて元の大きさに戻そうとすると複数行投稿の状態(投稿欄の右端にスクロールバーが表示される)が解除されない不具合がありました。
SplitContainer2.BorderStyleにFixed3Dを設定した場合のスクリーンショット
そこで、SplitContainer2 ではなく SplitContainer3 の BorderStyle を Fixed3D に設定したところ、複数行投稿の不具合は見られませんでした。(全然関係ないですがAPIレートリミットの表示バグってますね)
SplitContainer3.BorderStyleにFixed3Dを設定した場合のスクリーンショット
SplitContainer3 の場合ですと、外周の線で投稿欄とサムネイル画像の両方の境界が良い具合に識別しやすくなったように見えますが如何でしょうか?

@spx268
Copy link
Member

spx268 commented Jun 27, 2013

複数行投稿のために予め投稿欄を広げて使っていると、サムネイル枠があまりに小さくなりすぎるのですが、旧方式も設定の変更等で選べるようにはできますか?
サムネイル枠のために詳細欄を広げてしまうと、今度はTL表示量が減ってしまってあまり効果的に思えません。

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants