Реализация «Переименовать» из ContextMenu

Question

Каков наилучший способ реализации функции переименования из контекстного меню в WPF? Что мне нужно, так это функции, такие как проводник Windows, где вы щелкаете правой кнопкой мыши элемент, получаете контекстное меню, и если вы выбираете переименовать, текст становитсяредактируемые.

Пока я пробовал следующее, но я думаю, что должен быть лучший способ.Практически любой аспект этого решения может быть изменен:)

<ListBox Grid.Row="0" MinWidth="200" MinHeight="75" ItemsSource="{Binding Path=DataSetsVM.DataSets, Source={StaticResource vmLocator}}" x:Name="dataSets">
<ListBox.ItemTemplate>
    <DataTemplate>
        <StackPanel x:Name="nameRoot">
            <TextBlock x:Name="nameBlock" Text="{Binding Path=Name}" HorizontalAlignment="Stretch" MinWidth="200" Tag="{Binding DataContext, ElementName=dataSets}">
            <TextBlock.ContextMenu> 
                <ContextMenu >
                    <ContextMenu.Resources>
                        <Style TargetType="Image">
                            <Setter Property="Height" Value="24"/>
                            <Setter Property="Width" Value="24"/>
                        </Style>
                    </ContextMenu.Resources>
                    <MenuItem Header="Rename" x:Name="RenameDatasetContext"
                              Command="{Binding PlacementTarget.Tag.CoreCommands.RenameDatasetCommand, RelativeSource={RelativeSource AncestorType=ContextMenu}}"
                              CommandParameter="{Binding PlacementTarget.DataContext, 
                              RelativeSource={RelativeSource FindAncestor, 
                              AncestorType={x:Type ContextMenu}}}">
                        <MenuItem.Icon>
                            <Image Source="{StaticResource RenameLargeIcon}"/>
                        </MenuItem.Icon>
                    </MenuItem>
                </ContextMenu>
            </TextBlock.ContextMenu>
            </TextBlock>
            <TextBox x:Name="nameBox"  Text="{Binding Path=Name}" HorizontalAlignment="Stretch" MinWidth="200" Visibility="Collapsed" />
        </StackPanel>
        <DataTemplate.Triggers>                                            
            <DataTrigger Binding="{Binding Path=IsEditable}" Value="True">
                <Setter TargetName="nameBlock" Property="Visibility" Value="Collapsed"/>
                <Setter TargetName="nameBox" Property="Visibility" Value="Visible"/>
            </DataTrigger>
        </DataTemplate.Triggers>
    </DataTemplate>
</ListBox.ItemTemplate> 
</ListBox>

Установщик для Name делает это:

set
{
    if (IsEditable)
    {
        IsEditable = false;   
        if (_name == value)
        {
            RaisePropertyChanged(NamePropertyName);
            return;
        }
        // Do some back end stuff
        if (back end stuff worked)
        {
            var oldValue = _name;
            _name = value;
            RaisePropertyChanged(NamePropertyName);
        }
        else
        {
            // Do something about the error in the back end
        }
    }   
}

Все, что делает команда, устанавливает IsEditable в true.

Это работает, если вы на самом деле не изменяете имя, и в этот момент ничто не может вернуть IsEditable, так как никогда не вызывается установщик для Name, так что вы можете получить кучу элементов в режиме Rename.

Могу ли я установить свойство на виртуальной машине через триггер или есть лучший способ сделать это?

Answer 1

это было бы больше MVVM-иш .. Я упростил привязку, чтобы образец был коротким и простым, и я просто подключил ViewModel к CS. файл сзади для быстрого тестирования, все компилируется / запускается.

<Window x:Class="Sample.MainWindow"
    xmlns="http://schemas.microsoft.com/winfx/2006/xaml/presentation"
    xmlns:x="http://schemas.microsoft.com/winfx/2006/xaml" 
    Height="350" Width="525" 
    x:Name="wnd">

<ListBox ItemsSource="{Binding ViewModel.Items, ElementName=wnd}"
         SelectedItem="{Binding ViewModel.SelectedItem, ElementName=wnd}">
    <ListBox.ItemTemplate>
        <DataTemplate>
            <StackPanel>
                <TextBlock x:Name="nameBlock" Text="{Binding Name}" Margin="4">
                    <TextBlock.ContextMenu> 
                        <ContextMenu>
                            <MenuItem Header="Rename" 
                                      Command="{Binding EditItemCommand}">
                            </MenuItem>
                        </ContextMenu>
                    </TextBlock.ContextMenu>
                </TextBlock>
                <TextBox x:Name="nameBox" Text="{Binding Name}" Visibility="Collapsed" Margin="2,0"/>
            </StackPanel>
            <DataTemplate.Triggers>
                <DataTrigger Binding="{Binding IsEditable}" Value="True">
                    <Setter TargetName="nameBlock" Property="Visibility" Value="Collapsed"/>
                    <Setter TargetName="nameBox" Property="Visibility" Value="Visible"/>
                </DataTrigger>
            </DataTemplate.Triggers>
        </DataTemplate>
    </ListBox.ItemTemplate>
</ListBox>

код сзади (обратите внимание, вид полностью пассивен):

public partial class MainWindow : Window
{
    public ViewModel ViewModel { get; set; }

    public MainWindow()
    {
        ViewModel = new ViewModel();
        InitializeComponent();
    }
}

viewModel (имеет всю логику манипуляции):

public class ViewModel : INotifyPropertyChanged
{
    private ObservableCollection<SomeClass> _items;
    private SomeClass _selectedItem;

    public ViewModel()
    {
        SelectedItem = Items.First(); 
    }

    public ObservableCollection<SomeClass> Items
    {
        get
        {
            if (_items == null)
            {
                _items = new ObservableCollection<SomeClass>();
                for (int i = 0; i < 10; i++) _items.Add(new SomeClass(string.Format("name {0}", i)));
            }

            return _items;
        }
    }

    public SomeClass SelectedItem
    {
        get { return _selectedItem; }
        set
        {
            if (_selectedItem == value)
                return;

            _selectedItem = value;

            foreach (var item in Items) item.IsEditable = false;

            RaisePropertyChanged("SelectedItem");
        }
    }

    private void RaisePropertyChanged(string propertyName)
    {
        if (PropertyChanged != null)
            PropertyChanged(this, new PropertyChangedEventArgs(propertyName));
    }

    public event PropertyChangedEventHandler PropertyChanged;
}

Модель (совершенно глупая по отношению к другим моделям, знает только, как установить свое собственное поведение) со свойствами, полностью очищенными от любой логики:

public class SomeClass : INotifyPropertyChanged
{
    private string _name;
    private bool _isEditable;
    private DelegateCommand _editItemCmd;

    public SomeClass(string name) { _name = name;}

    public DelegateCommand EditItemCommand
    {
        get
        {
            return _editItemCmd ?? (_editItemCmd = new DelegateCommand(() => { IsEditable = true; }));
        }
    }

    public string Name
    {
        get { return _name; }
        set
        {
            if (_name == value)
                return;
            _name = value;
            RaisePropertyChanged("Name");
        }
    }

    public bool IsEditable
    {
        get { return _isEditable; }
        set
        {
            if(_isEditable == value)
                return;
            _isEditable = value;
            RaisePropertyChanged("IsEditable");
        }
    }

    private void RaisePropertyChanged(string propertyName)
    {
        if(PropertyChanged != null)
            PropertyChanged(this, new PropertyChangedEventArgs(propertyName));
    }

    public event PropertyChangedEventHandler PropertyChanged;
}

Answer 2

это странно, но, учитывая ваш код (так, как он написан), вы попадаете в причудливую ситуацию, поэтому самое простое / быстрое решение - сбросить все элементы ListBox перед началом редактирования нового:

в XAML добавлено к событию nameBlock mousedown:

<TextBlock x:Name="nameBlock" MouseDown="nameBlock_MouseDown"...

в коде CS:

private object current;

private void nameBlock_MouseDown(object sender, MouseButtonEventArgs e)
{
    if (current != null && current == sender)
    return;
    foreach (var item in dataSets.ItemsSource)
        ((SomeClass)item).IsEditable = false;
}

также убрал проверку IsEditable из установщика

set
        {
            //if (IsEditable)
            {
                //IsEditable = false;
                if (_name == value)
                {
                    RaisePropertyChanged("Name");
                    return;
                }
                // Do some back end stuff
                if (back //end stuff //worked)
                {
                    var oldValue = _name;
                    _name = value;
                    RaisePropertyChanged("Name");
                }
            }

Answer 3

Пара вещей.

Во-первых, если вы пытаетесь заставить его вести себя как Windows Explorer, то вам нужно добавить связывание клавиш в Esc, которое устанавливает IsEditable = false.

Во-вторых, вам нужно добавить триггер наTextBox для LostFocus, который устанавливает IsEditable = false.