This repository was archived by the owner on Jun 21, 2023. It is now read-only.
Commit a26703d
committed
Don't use RxUI for Repositories binding.
As it was sometimes setting the selection on the `ListBox` without user
interaction. Instead, use XAML bindings. Unfortunately this required
binding to a dependency property on the view itself. Better solutions
moving forward would be:
- Declare the collection view in XAML. This would require the Group
classes to be made public, but they have a dependency on state in the
view.
- Declate the collection view in the view model.
I didn't implement either of these solutions as I felt the changes would
be too impactful on the codebase at this point in time.1 parent c9b990f commit a26703d
File tree
2 files changed
+22
-2
lines changed- src/GitHub.VisualStudio/UI/Views/Controls
2 files changed
+22
-2
lines changedLines changed: 1 addition & 1 deletion
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
243 | 243 | | |
244 | 244 | | |
245 | 245 | | |
246 | | - | |
| 246 | + | |
247 | 247 | | |
248 | 248 | | |
249 | 249 | | |
| |||
Lines changed: 21 additions & 1 deletion
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
33 | 33 | | |
34 | 34 | | |
35 | 35 | | |
| 36 | + | |
| 37 | + | |
| 38 | + | |
| 39 | + | |
| 40 | + | |
| 41 | + | |
| 42 | + | |
| 43 | + | |
| 44 | + | |
36 | 45 | | |
37 | 46 | | |
38 | 47 | | |
39 | 48 | | |
40 | 49 | | |
41 | 50 | | |
42 | | - | |
43 | 51 | | |
44 | 52 | | |
45 | 53 | | |
| 54 | + | |
46 | 55 | | |
47 | 56 | | |
48 | 57 | | |
49 | 58 | | |
50 | 59 | | |
| 60 | + | |
| 61 | + | |
| 62 | + | |
| 63 | + | |
| 64 | + | |
| 65 | + | |
| 66 | + | |
| 67 | + | |
51 | 68 | | |
52 | 69 | | |
53 | 70 | | |
54 | 71 | | |
| 72 | + | |
| 73 | + | |
| 74 | + | |
55 | 75 | | |
56 | 76 | | |
57 | 77 | | |
| |||
0 commit comments