From 2da1349fcb090663ab01b4167aaa393689f7a9db Mon Sep 17 00:00:00 2001
From: Andreia Gaita <shana@spoiledcat.net>
Date: Tue, 1 Sep 2015 14:04:51 +0200
Subject: [PATCH 1/2] Fix the styling of the checkbox so it binds properly

Fixes #62
---
 src/GitHub.VisualStudio/SharedDictionary.xaml | 27 ++++++-------------
 .../Controls/RepositoryPublishControl.xaml    | 19 ++++++++++++-
 2 files changed, 26 insertions(+), 20 deletions(-)

diff --git a/src/GitHub.VisualStudio/SharedDictionary.xaml b/src/GitHub.VisualStudio/SharedDictionary.xaml
index 8c24d463bd..3b69a1d0df 100644
--- a/src/GitHub.VisualStudio/SharedDictionary.xaml
+++ b/src/GitHub.VisualStudio/SharedDictionary.xaml
@@ -10,25 +10,14 @@
 
   <Style x:Key="VSStyledButton" TargetType="{x:Type Button}" BasedOn="{StaticResource VsButtonStyleKey}">
   </Style>
-  <Style x:Key="VSStyledCheckBox" TargetType="{x:Type CheckBox}">
-    <Setter Property="Control.Template">
-      <Setter.Value>
-        <ControlTemplate TargetType="{x:Type CheckBox}">
-          <CheckBox Foreground="{DynamicResource VsBrush.ToolWindowText}" Background="{DynamicResource VsBrush.ToolWindowBackground}">
-              <TextBlock TextWrapping="Wrap" Foreground="{DynamicResource VsBrush.ToolWindowText}" Background="{DynamicResource VsBrush.ToolWindowBackground}">
-                <TextBlock.Inlines>
-                    <Run Text="{TemplateBinding Content}" />
-                </TextBlock.Inlines>
-              </TextBlock>
-            </CheckBox>
-            <ControlTemplate.Triggers>
-              <Trigger Property="UIElement.IsMouseOver" Value="true">
-                <Setter Property="Background" Value="{DynamicResource VsBrush.ToolWindowBackground}"/>
-              </Trigger>
-            </ControlTemplate.Triggers>
-          </ControlTemplate>
-        </Setter.Value>
-      </Setter>
+  <Style x:Key="VSStyledCheckBox" BasedOn="{StaticResource VsCheckBoxStyleKey}" TargetType="{x:Type CheckBox}">
+    <Setter Property="Foreground" Value="{DynamicResource VsBrush.ToolWindowText}" />
+    <Setter Property="Background" Value="{DynamicResource VsBrush.ToolWindowBackground}" />
+    <Style.Triggers>
+      <Trigger Property="UIElement.IsMouseOver" Value="true">
+        <Setter Property="Background" Value="{DynamicResource VsBrush.ToolWindowBackground}"/>
+      </Trigger>
+    </Style.Triggers>
   </Style>
   <Style x:Key="VSStyledComboBox" TargetType="{x:Type ComboBox}" BasedOn="{StaticResource VsComboBoxStyleKey}">
   </Style>
diff --git a/src/GitHub.VisualStudio/UI/Views/Controls/RepositoryPublishControl.xaml b/src/GitHub.VisualStudio/UI/Views/Controls/RepositoryPublishControl.xaml
index 263921c77f..9dad97678e 100644
--- a/src/GitHub.VisualStudio/UI/Views/Controls/RepositoryPublishControl.xaml
+++ b/src/GitHub.VisualStudio/UI/Views/Controls/RepositoryPublishControl.xaml
@@ -123,7 +123,24 @@
 
         <CheckBox x:Name="makePrivate"
                   Margin="0,8,0,0"
-                  Style="{StaticResource VSStyledCheckBox}">Private Repository</CheckBox>
+                  Style="{StaticResource VSStyledCheckBox}">
+          <TextBlock TextWrapping="Wrap"
+                     Foreground="{DynamicResource VsBrush.ToolWindowText}"
+                     Background="{DynamicResource VsBrush.ToolWindowBackground}">
+            <TextBlock.Resources>
+              <Style TargetType="{x:Type TextBlock}">
+                <Style.Triggers>
+                  <Trigger Property="IsEnabled" Value="False">
+                    <Setter Property="Opacity" Value="0.5" />
+                  </Trigger>
+                </Style.Triggers>
+              </Style>
+            </TextBlock.Resources>
+            <TextBlock.Inlines>
+              <Run>Private Repository</Run>
+            </TextBlock.Inlines>
+          </TextBlock>
+        </CheckBox>
         
         <Label x:Name="upgradeToMicroPlanWarning" HorizontalAlignment="Left" Visibility="Collapsed">
             <StackPanel Orientation="Horizontal" HorizontalAlignment="Stretch">

From 3f70f80ad7f24477e1ca200f358bafe1feea1119 Mon Sep 17 00:00:00 2001
From: Andreia Gaita <shana@spoiledcat.net>
Date: Tue, 1 Sep 2015 14:06:03 +0200
Subject: [PATCH 2/2] Revert "Merge pull request #64 from
 shiftkey/checked-does-not-propagate"

This reverts commit a523bf9675f927706cebcaa5d66cf003cf48345f, reversing
changes made to 56511e561d1844cab37ab753955cd006f00ae2fb.
---
 .../Controls/RepositoryPublishControl.xaml.cs | 28 +------------------
 1 file changed, 1 insertion(+), 27 deletions(-)

diff --git a/src/GitHub.VisualStudio/UI/Views/Controls/RepositoryPublishControl.xaml.cs b/src/GitHub.VisualStudio/UI/Views/Controls/RepositoryPublishControl.xaml.cs
index 69c641fcd9..3c2334be33 100644
--- a/src/GitHub.VisualStudio/UI/Views/Controls/RepositoryPublishControl.xaml.cs
+++ b/src/GitHub.VisualStudio/UI/Views/Controls/RepositoryPublishControl.xaml.cs
@@ -30,33 +30,7 @@ public RepositoryPublishControl()
                 d(this.Bind(ViewModel, vm => vm.RepositoryName, v => v.nameText.Text));
                 
                 d(this.Bind(ViewModel, vm => vm.Description, v => v.description.Text));
-
-                d(this.WhenAnyValue(x => x.ViewModel.KeepPrivate)
-                    .Subscribe(keepPrivate =>
-                    {
-                        // because we removed this.Bind, set this by hand
-                        if (keepPrivate != makePrivate.IsChecked)
-                        {
-                            makePrivate.IsChecked = keepPrivate;
-                        }
-                    }));
-
-                // BEGIN DANGER ZONE
-                //
-                // This replaces the default Bind behaviour as the Checkbox control
-                // does not raise an event here when it is hosted in the Team Explorer
-                // view.
-                //
-                // We've used this.Bind in other places (popups) so it's something
-                // we need to investigate further
-                //
-                d(Observable.FromEventPattern<RoutedEventArgs>(makePrivate, "Checked")
-                    .Subscribe(_ => ViewModel.KeepPrivate = true));
-                d(Observable.FromEventPattern<RoutedEventArgs>(makePrivate, "Unchecked")
-                    .Subscribe(_ => ViewModel.KeepPrivate = false));
-                // END DANGER ZONE
-
-
+                d(this.Bind(ViewModel, vm => vm.KeepPrivate, v => v.makePrivate.IsChecked));
                 d(this.OneWayBind(ViewModel, vm => vm.CanKeepPrivate, v => v.makePrivate.IsEnabled));
 
                 //d(this.OneWayBind(ViewModel, vm => vm.ShowUpgradeToMicroPlanWarning, v => v.upgradeToMicroPlanWarning.Visibility));