Skip to content

Commit 1101f28

Browse files
committed
Refactored ItemContainerGenerator.
A bunch of tests still failing, and some code commented out, but outlines the new API shape.
1 parent 5d1f9f4 commit 1101f28

34 files changed

+2670
-3772
lines changed

src/Avalonia.Controls/ComboBox.cs

Lines changed: 9 additions & 17 deletions
Original file line numberDiff line numberDiff line change
@@ -158,15 +158,6 @@ public VerticalAlignment VerticalContentAlignment
158158
set { SetValue(VerticalContentAlignmentProperty, value); }
159159
}
160160

161-
/// <inheritdoc/>
162-
protected override IItemContainerGenerator CreateItemContainerGenerator()
163-
{
164-
return new ItemContainerGenerator<ComboBoxItem>(
165-
this,
166-
ComboBoxItem.ContentProperty,
167-
ComboBoxItem.ContentTemplateProperty);
168-
}
169-
170161
protected override void OnAttachedToVisualTree(VisualTreeAttachmentEventArgs e)
171162
{
172163
base.OnAttachedToVisualTree(e);
@@ -449,14 +440,15 @@ private void UpdateFlowDirection()
449440

450441
private void SelectFocusedItem()
451442
{
452-
foreach (ItemContainerInfo dropdownItem in ItemContainerGenerator.Containers)
453-
{
454-
if (dropdownItem.ContainerControl.IsFocused)
455-
{
456-
SelectedIndex = dropdownItem.Index;
457-
break;
458-
}
459-
}
443+
throw new NotImplementedException();
444+
////foreach (ItemContainerInfo dropdownItem in ItemContainerGenerator.Containers)
445+
////{
446+
//// if (dropdownItem.ContainerControl.IsFocused)
447+
//// {
448+
//// SelectedIndex = dropdownItem.Index;
449+
//// break;
450+
//// }
451+
////}
460452
}
461453

462454
private void SelectNext()

src/Avalonia.Controls/ContextMenu.cs

Lines changed: 0 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -315,11 +315,6 @@ event Action<IPopupHost?>? IPopupHostProvider.PopupHostChanged
315315
remove => _popupHostChangedHandler -= value;
316316
}
317317

318-
protected override IItemContainerGenerator CreateItemContainerGenerator()
319-
{
320-
return new MenuItemContainerGenerator(this);
321-
}
322-
323318
private void Open(Control control, Control placementTarget, bool requestedByPointer)
324319
{
325320
if (IsOpen)

src/Avalonia.Controls/Flyouts/MenuFlyoutPresenter.cs

Lines changed: 0 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -35,11 +35,6 @@ public override void Open()
3535
throw new NotSupportedException("Use MenuFlyout.ShowAt(Control) instead");
3636
}
3737

38-
protected override IItemContainerGenerator CreateItemContainerGenerator()
39-
{
40-
return new MenuItemContainerGenerator(this);
41-
}
42-
4338
protected override void OnDetachedFromVisualTree(VisualTreeAttachmentEventArgs e)
4439
{
4540
base.OnDetachedFromVisualTree(e);

src/Avalonia.Controls/Generators/IItemContainerGenerator.cs

Lines changed: 0 additions & 114 deletions
This file was deleted.

src/Avalonia.Controls/Generators/ITreeItemContainerGenerator.cs

Lines changed: 0 additions & 18 deletions
This file was deleted.

src/Avalonia.Controls/Generators/ItemContainerEventArgs.cs

Lines changed: 0 additions & 49 deletions
This file was deleted.

0 commit comments

Comments
 (0)