Menu

#321 DIFF・履歴ダイアログを可変長サイズ

None
closed
nobody
None
5
2013-02-18
2013-02-05
Moca
No

arokaさんのパッチを見て、DIFF・履歴ダイアログも可変長サイズにしてみました。
また比較ダイアログも上下に伸ばせるようにしました。

1 Attachments

Discussion

  • aroka

    aroka - 2013-02-06

    AnchorStyles いいですね。
    トピックツリーのOnSizeもこの方式に実装を変えてみましたが、ボタンの再描画がうまくいかないことがあります。
    添削していただけると有難いです。

     
  • Moca

    Moca - 2013-02-07

    トピックツリーでもおかしくならないように、トピックツリーの従来の処理を参考に修正しました。
    AnchorStyles -> AnchorStyle
    トピックツリーの順序(&S)の順番を後ろに回した(一覧に最初にフォーカスが当たるように)
    MoveWindowをSetWindowPosとInvalidateRectの組み合わせにした
    トピックツリーをウィンドウにドッキングしたときに下が空いていたのを修正
    などです。

     
  • aroka

    aroka - 2013-02-10

    トピックツリーはよくなったけれど、他のダイアログはちらつきが増えたように思います。
    履歴ダイアログの描画が重いのは CDlgFavorite::OnSize を
    ResizeItem( hwndList, m_ptDefaultSize, ptNew, m_rcListDefault, ANCHOR_ALL, (i==m_nCurrentTab) );
    のようにすると改善します。

     
  • aroka

    aroka - 2013-02-11

    トピックツリーを参考に WindowStyle に WS_CLIPCHILDREN を追加しました。
    このままだとDIFFのグループボックス内の色が抜けるので WS_EX_TRANSPARENT をセットしています。
    あとは上の履歴ダイアログのタブの描画フラグの変更です。

     
  • Uchi

    Uchi - 2013-02-12

    バグの報告をさせていただきます
    現象的には「アウトラインを表示したまま『閉じて開く』を行うと落ちる」です
    原因はv2以降のパッチで"CDlgFuncList.h"のm_rcItems[10];のサイズが小さすぎる為です
    メモリリークが原因の為、現象が出ない或いは別の現象が出るとかは有ると思います

     
  • aroka

    aroka - 2013-02-12

    ありがとうございます。
    p_helpidsと順序をあわせ、抜けていたコントロールも追加しました。

    #位置あわせ不要なANCHOR_LEFTでの追加は性能低下も懸念されますが、漏れ防止には全列挙がいいかな、と。

     
  • aroka

    aroka - 2013-02-12

    再発防止のため、各コンストラクタにassertを追加しました

     
  • aroka

    aroka - 2013-02-18

    [r2608]でコミットしました。

     

    Related

    Commit: [r2608]

  • aroka

    aroka - 2013-02-18
    • status: open --> closed
    • milestone: -->
     

Log in to post a comment.