Hack to display avatars at 100x100.

We need a better avatar cache...

Author: grokys

src/GitHub.App/Models/Account.cs

@@ -80,7 +80,7 @@ public Account(Octokit.Account account, IObservable<BitmapSource> bitmapSource)
 
         public long PrivateReposInPlan { get; private set; }
 
-        public string AvatarUrl { get; private set; }
+        public string AvatarUrl { get; set; }
 
         public BitmapSource Avatar
         {

src/GitHub.App/SampleData/ForkRepositorySelectViewModelDesigner.cs

@@ -14,11 +14,11 @@ public ForkRepositorySelectViewModelDesigner()
         {
             Accounts = new[]
             {
-                new AccountDesigner { Login = "Myself" },
-                new AccountDesigner { Login = "MyOrg1" },
-                new AccountDesigner { Login = "MyOrg2" },
-                new AccountDesigner { Login = "MyOrg3" },
-                new AccountDesigner { Login = "a-long-org-name" },
+                new AccountDesigner { Login = "Myself", AvatarUrl = "https://identicons.github.com/myself.png" },
+                new AccountDesigner { Login = "MyOrg1", AvatarUrl = "https://identicons.github.com/myorg1.png" },
+                new AccountDesigner { Login = "MyOrg2", AvatarUrl = "https://identicons.github.com/myorg2.png"  },
+                new AccountDesigner { Login = "MyOrg3", AvatarUrl = "https://identicons.github.com/myorg3.png"  },
+                new AccountDesigner { Login = "a-long-org-name", AvatarUrl = "https://identicons.github.com/a-long-org-name.png"  },
             };
 
             ExistingForks = new[]

src/GitHub.App/ViewModels/Dialog/ForkRepositorySelectViewModel.cs

@@ -108,6 +108,14 @@ void BuildAccounts(IReadOnlyList<IAccount> accessibleAccounts, ILocalRepositoryM
 
             Accounts = readOnlyList.Where(arg => arg.Fork == null).Select(arg => arg.Account).ToList();
             ExistingForks = readOnlyList.Where(arg => arg.Fork != null).Select(arg => arg.Fork).ToList();
+
+            // HACK: Our avatar cache only provides avatars in a very small size, but we want to 
+            // display them 100x100 in the Fork view. For now, wse the AvatarUrl directly to get
+            // the avatar, appending "s=100" to the URL to get the correct size.
+            foreach (Account account in Accounts)
+            {
+                account.AvatarUrl += "&s=100";
+            }
         }
     }
 }

src/GitHub.VisualStudio/Views/Dialog/ForkRepositorySelectView.xaml

@@ -65,10 +65,11 @@
                                 Text="{Binding Login, StringFormat=@{0}}"
                                 TextAlignment="Center"
                                 TextTrimming="CharacterEllipsis"/>
-              <Image
-                              Margin="4"
-                              MaxWidth="100"
-                              Source="{Binding Avatar}" />
+              <Image Margin="4"
+                     Width="100"
+                     Height="100"
+                     Stretch="Uniform"
+                     Source="{Binding AvatarUrl}" />
             </DockPanel>
           </Border>
         </DataTemplate>