lmi-commits
[Top][All Lists]
Advanced

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

[lmi-commits] (no subject)


From: Greg Chicares
Subject: [lmi-commits] (no subject)
Date: Thu, 9 Jun 2016 23:17:40 +0000 (UTC)

branch: master
commit 113e7853f69400321b7ea773e941d780822ee426
Author: Vadim Zeitlin <address@hidden>
Date:   Wed Jun 8 20:28:53 2016 +0200

    Don't set minimal size specially for InputSequenceButton
    
    The default size of small buttons with wxBU_EXACTFIT is good enough in
    wxMSW since the commit c4d06e8117f8930b57bffaf6a3323007c9df8d4b there,
    so there is no need to set the minimal size of InputSequenceButton
    manually as it just makes it inconsistent with the buttons used in
    wx{File,Dir}PickerCtrls.
---
 input_sequence_entry.cpp |    5 -----
 1 file changed, 5 deletions(-)

diff --git a/input_sequence_entry.cpp b/input_sequence_entry.cpp
index 91c11df..1e3f347 100644
--- a/input_sequence_entry.cpp
+++ b/input_sequence_entry.cpp
@@ -1366,11 +1366,6 @@ InputSequenceButton::InputSequenceButton(wxWindow* 
parent, wxWindowID id)
     :wxButton(parent, id, "...", wxDefaultPosition, wxDefaultSize, 
wxBU_EXACTFIT)
 {
     SetToolTip("Open sequence editor");
-
-    // Set vertical size to 1px - it's ridiculously small, but the sizers will 
make it as
-    // tall as the text control. Use text extent of "..." for width, because 
standard
-    // buttons use more padding.
-    SetMinSize(wxSize(8 + GetTextExtent(GetLabel()).x, 1));
 }
 
 } // Unnamed namespace.



reply via email to

[Prev in Thread] Current Thread [Next in Thread]