From 93103bab4080fc001a6a898f8efe30c98ee7caea Mon Sep 17 00:00:00 2001
From: Vadim Zeitlin <vadim@wxwidgets.org>
Date: Sun, 10 Apr 2005 18:07:08 +0000
Subject: [PATCH] removed is_vertical check box, it was duplicating the ticks
 direction radiobox functionality

git-svn-id: https://svn.wxwidgets.org/svn/wx/wxWidgets/trunk@33481 c3d73ce0-8a6f-49c7-b76d-6d57e0e08775
---
 samples/widgets/slider.cpp | 40 ++++++++++++++++----------------------
 1 file changed, 17 insertions(+), 23 deletions(-)

diff --git a/samples/widgets/slider.cpp b/samples/widgets/slider.cpp
index 7b3c89bbc6..3067c60d41 100644
--- a/samples/widgets/slider.cpp
+++ b/samples/widgets/slider.cpp
@@ -76,10 +76,10 @@ enum
 // sides radiobox values
 enum
 {
-    StaticSides_Top,
-    StaticSides_Bottom,
-    StaticSides_Left,
-    StaticSides_Right
+    SliderTicks_Top,
+    SliderTicks_Bottom,
+    SliderTicks_Left,
+    SliderTicks_Right
 };
 
 // ----------------------------------------------------------------------------
@@ -142,7 +142,6 @@ protected:
 
     // the check/radio boxes for styles
     wxCheckBox *m_chkLabels,
-               *m_chkVert,
                *m_chkInverse,
                *m_chkTicks,
                *m_chkBothSides;
@@ -210,7 +209,6 @@ SliderWidgetsPage::SliderWidgetsPage(wxBookCtrl *book,
     m_min = 0;
     m_max = 100;
 
-    m_chkVert =
     m_chkInverse =
     m_chkTicks =
     m_chkLabels =
@@ -227,7 +225,6 @@ SliderWidgetsPage::SliderWidgetsPage(wxBookCtrl *book,
     wxStaticBox *box = new wxStaticBox(this, wxID_ANY, _T("&Set style"));
     wxSizer *sizerLeft = new wxStaticBoxSizer(box, wxVERTICAL);
 
-    m_chkVert = CreateCheckBoxAndAddToSizer(sizerLeft, _T("&Vertical"));
     m_chkInverse = CreateCheckBoxAndAddToSizer(sizerLeft, _T("&Inverse"));
     m_chkTicks = CreateCheckBoxAndAddToSizer(sizerLeft, _T("Show &ticks"));
     m_chkLabels = CreateCheckBoxAndAddToSizer(sizerLeft, _T("Show &labels"));
@@ -326,25 +323,18 @@ SliderWidgetsPage::SliderWidgetsPage(wxBookCtrl *book,
 
 void SliderWidgetsPage::Reset()
 {
-    m_chkVert->SetValue(false);
     m_chkInverse->SetValue(false);
     m_chkTicks->SetValue(true);
     m_chkLabels->SetValue(true);
     m_chkBothSides->SetValue(false);
 
-    m_radioSides->SetSelection(StaticSides_Top);
+    m_radioSides->SetSelection(SliderTicks_Top);
 }
 
 void SliderWidgetsPage::CreateSlider()
 {
     int flags = 0;
 
-    bool isVert = m_chkVert->GetValue();
-    if ( isVert )
-        flags |= wxSL_VERTICAL;
-    else
-        flags |= wxSL_HORIZONTAL;
-
     if ( m_chkInverse->GetValue() )
     {
         flags |= wxSL_INVERSE;
@@ -362,18 +352,22 @@ void SliderWidgetsPage::CreateSlider()
 
     switch ( m_radioSides->GetSelection() )
     {
-        case StaticSides_Top:
+        case SliderTicks_Top:
             flags |= wxSL_TOP;
             break;
-        case StaticSides_Left:
+
+        case SliderTicks_Left:
             flags |= wxSL_LEFT;
             break;
-        case StaticSides_Bottom:
+
+        case SliderTicks_Bottom:
             flags |= wxSL_BOTTOM;
             break;
-        case StaticSides_Right:
+
+        case SliderTicks_Right:
             flags |= wxSL_RIGHT;
             break;
+
         default:
             wxFAIL_MSG(_T("unexpected radiobox selection"));
             // fall through
@@ -410,7 +404,7 @@ void SliderWidgetsPage::CreateSlider()
                             wxDefaultPosition, wxDefaultSize,
                             flags);
 
-    if ( isVert )
+    if ( m_slider->HasFlag(wxSL_VERTICAL) )
     {
         m_sizerSlider->Add(0, 0, 1);
         m_sizerSlider->Add(m_slider, 0, wxGROW | wxALL, 5);
@@ -544,11 +538,11 @@ void SliderWidgetsPage::OnUpdateUIMinMaxButton(wxUpdateUIEvent& event)
 
 void SliderWidgetsPage::OnUpdateUIResetButton(wxUpdateUIEvent& event)
 {
-    event.Enable( m_chkVert->GetValue() ||
-                  m_chkInverse->GetValue() ||
+    event.Enable( m_chkInverse->GetValue() ||
                   !m_chkTicks->GetValue() ||
                   !m_chkLabels->GetValue() ||
-                  m_chkBothSides->GetValue() );
+                  m_chkBothSides->GetValue() ||
+                  m_radioSides->GetSelection() != SliderTicks_Top );
 }
 
 void SliderWidgetsPage::OnCheckOrRadioBox(wxCommandEvent& WXUNUSED(event))
-- 
2.45.2