Skip to content

Feature: Updated the edit tags button in the Details Pane #12978

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 7 commits into from
Jul 19, 2023
Merged

Feature: Updated the edit tags button in the Details Pane #12978

Show file tree
Hide file tree
Changes from 1 commit
Commits
Show all changes
7 commits
Select commit Hold shift + click to select a range
60f9774
Preview Pane: Move Edit Tags Button
ferrariofilippo Jul 19, 2023
91c630f
Update PathIcons.xaml
yaira2 Jul 19, 2023
e60ad59
Merge branch 'main' into Change_Edit_Tags_Button_Position
yaira2 Jul 19, 2023
6f8994e
Update PreviewPane.xaml
yaira2 Jul 19, 2023
a53ab7a
Update PathIcons.xaml
yaira2 Jul 19, 2023
4e3c923
Update PathIcons.xaml
yaira2 Jul 19, 2023
3173206
Update PathIcons.xaml
yaira2 Jul 19, 2023
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
43 changes: 43 additions & 0 deletions src/Files.App/Data/Items/TagsListItem.cs
View file
Open in desktop
Original file line number Diff line number Diff line change
@@ -0,0 +1,43 @@
// Copyright (c) 2023 Files Community
// Licensed under the MIT License. See the LICENSE.

using Microsoft.UI.Xaml.Controls;

namespace Files.App.Data.Items
{

public class TagsListItem
{
public bool IsTag
=> this is TagItem;

public TagItem? AsTag
=> this as TagItem;

public bool IsFlyout
=> this is FlyoutItem;

public FlyoutItem? AsFlyout
=> this as FlyoutItem;
}

public class TagItem : TagsListItem
{
public TagViewModel Tag { get; set; }

public TagItem(TagViewModel tag)
{
Tag = tag;
}
}

public class FlyoutItem : TagsListItem
{
public MenuFlyout Flyout { get; set; }

public FlyoutItem(MenuFlyout flyout)
{
Flyout = flyout;
}
}
}
68 changes: 32 additions & 36 deletions src/Files.App/UserControls/Pane/PreviewPane.xaml
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -7,9 +7,9 @@
xmlns:converters="using:CommunityToolkit.WinUI.UI.Converters"
xmlns:d="http://schemas.microsoft.com/expression/blend/2008"
xmlns:helpers="using:Files.App.Helpers"
xmlns:items="using:Files.App.Data.Items"
xmlns:mc="http://schemas.openxmlformats.org/markup-compatibility/2006"
xmlns:properties="using:Files.App.ViewModels.Properties"
xmlns:tags="using:Files.Core.ViewModels.FileTags"
xmlns:triggers="using:CommunityToolkit.WinUI.UI.Triggers"
xmlns:usercontrols="using:Files.App.UserControls"
x:Name="Root"
Expand Down Expand Up @@ -285,7 +285,7 @@

<GridView
HorizontalAlignment="Stretch"
ItemsSource="{x:Bind ViewModel.SelectedItem.FileTagsUI, Mode=OneWay}"
ItemsSource="{x:Bind ViewModel.Items, Mode=OneWay}"
SelectionMode="None">
<GridView.ItemsPanel>
<ItemsPanelTemplate>
Expand All @@ -294,45 +294,45 @@
</GridView.ItemsPanel>

<GridView.ItemTemplate>
<DataTemplate x:Name="TagTemplate" x:DataType="tags:TagViewModel">
<StackPanel
AutomationProperties.Name="{x:Bind Name, Mode=OneWay}"
Orientation="Horizontal"
Spacing="8"
ToolTipService.ToolTip="{x:Bind Name, Mode=OneWay}">
<PathIcon Data="{StaticResource ColorIconFilledTag}" Foreground="{x:Bind Color, Mode=OneWay}" />
<TextBlock
HorizontalAlignment="Stretch"
Style="{StaticResource Local.FileDetailsTextBlockStyle}"
Text="{x:Bind Name, Mode=OneWay}"
TextTrimming="CharacterEllipsis" />
<DataTemplate x:Name="TagTemplate" x:DataType="items:TagsListItem">
<StackPanel>
<StackPanel
x:Name="DetailsTagItem"
x:Load="{x:Bind IsTag, Mode=OneWay}"
AutomationProperties.Name="{x:Bind AsTag.Tag.Name, Mode=OneWay}"
Orientation="Horizontal"
Spacing="8"
ToolTipService.ToolTip="{x:Bind AsTag.Tag.Name, Mode=OneWay}">
<PathIcon Data="{StaticResource ColorIconFilledTag}" Foreground="{x:Bind AsTag.Tag.Color, Mode=OneWay}" />
<TextBlock
HorizontalAlignment="Stretch"
Style="{StaticResource Local.FileDetailsTextBlockStyle}"
Text="{x:Bind AsTag.Tag.Name, Mode=OneWay}"
TextTrimming="CharacterEllipsis" />
</StackPanel>

<!-- Edit Tags Button -->
<Button
x:Name="DetailsEditTags"
HorizontalAlignment="Left"
x:Load="{x:Bind IsFlyout, Mode=OneWay}"
AutomationProperties.Name="{helpers:ResourceString Name=EditTags}"
Flyout="{x:Bind AsFlyout.Flyout, Mode=OneWay}"
ToolTipService.ToolTip="{helpers:ResourceString Name=EditTags}">
<Button.Content>
<FontIcon FontSize="12" Glyph="&#xE710;" />
</Button.Content>
</Button>
</StackPanel>
</DataTemplate>
</GridView.ItemTemplate>
</GridView>
</StackPanel>

<!-- Edit Tags Button -->
<Button
x:Name="DetailsEditTags"
Margin="8,0,8,8"
HorizontalAlignment="Left"
Flyout="{x:Bind ViewModel.TagsFlyout, Mode=OneWay}"
ToolTipService.ToolTip="{helpers:ResourceString Name=EditTags}"
Visibility="Collapsed">
<Button.Content>
<StackPanel Orientation="Horizontal" Spacing="8">
<FontIcon FontSize="12" Glyph="&#xE8EC;" />
<TextBlock Text="{helpers:ResourceString Name=EditTags}" />
<FontIcon FontSize="12" Glyph="&#xE70D;" />
</StackPanel>
</Button.Content>
</Button>

<!-- Open Properties Button -->
<Button
x:Name="DetailsOpenProperties"
Margin="8,0,8,8"
Margin="8,8,8,8"
HorizontalAlignment="Left"
Command="{x:Bind Commands.OpenProperties}"
IsEnabled="{x:Bind Commands.OpenProperties.IsExecutable, Mode=OneWay}"
Expand Down Expand Up @@ -364,7 +364,6 @@
<Setter Target="PreviewErrorText.Visibility" Value="Visible" />
<Setter Target="PreviewErrorText.Text" Value="{x:Bind GetLocalizedResource('DetailsPanePreviewNotAvaliableText')}" />
<Setter Target="FileDetailsRepeater.Visibility" Value="Visible" />
<Setter Target="DetailsEditTags.Visibility" Value="Visible" />
<Setter Target="DetailsOpenProperties.Visibility" Value="Visible" />
<Setter Target="DetailsListHeader.Visibility" Value="Visible" />
<Setter Target="PreviewControlPresenter.Visibility" Value="Collapsed" />
Expand All @@ -378,7 +377,6 @@
<Setter Target="PreviewErrorText.Visibility" Value="Visible" />
<Setter Target="PreviewErrorText.Text" Value="{x:Bind GetLocalizedResource('DetailsPanePreviewNotAvaliableText')}" />
<Setter Target="FileDetailsRepeater.Visibility" Value="Collapsed" />
<Setter Target="DetailsEditTags.Visibility" Value="Collapsed" />
<Setter Target="DetailsOpenProperties.Visibility" Value="Collapsed" />
<Setter Target="PreviewControlPresenter.Visibility" Value="Collapsed" />
</VisualState.Setters>
Expand All @@ -390,7 +388,6 @@
<VisualState.Setters>
<Setter Target="PreviewErrorText.Visibility" Value="Collapsed" />
<Setter Target="FileDetailsRepeater.Visibility" Value="Visible" />
<Setter Target="DetailsEditTags.Visibility" Value="Visible" />
<Setter Target="DetailsOpenProperties.Visibility" Value="Visible" />
<Setter Target="PreviewControlPresenter.Visibility" Value="Visible" />
<Setter Target="DetailsListHeader.Visibility" Value="Visible" />
Expand All @@ -403,7 +400,6 @@
<VisualState.Setters>
<Setter Target="PreviewErrorText.Visibility" Value="Collapsed" />
<Setter Target="FileDetailsRepeater.Visibility" Value="Collapsed" />
<Setter Target="DetailsEditTags.Visibility" Value="Collapsed" />
<Setter Target="DetailsOpenProperties.Visibility" Value="Collapsed" />
<Setter Target="DetailsListHeader.Visibility" Value="Visible" />
<Setter Target="PreviewControlPresenter.Visibility" Value="Collapsed" />
Expand Down
17 changes: 13 additions & 4 deletions src/Files.App/ViewModels/UserControls/PreviewPaneViewModel.cs
View file
Open in desktop
Original file line number Diff line number Diff line change
@@ -1,7 +1,6 @@
// Copyright (c) 2023 Files Community
// Licensed under the MIT License. See the LICENSE.

using Files.App.Data.Contexts;
using Files.App.UserControls.FilePreviews;
using Files.App.ViewModels.Previews;
using Microsoft.UI.Xaml;
Expand Down Expand Up @@ -49,7 +48,7 @@ public ListedItem SelectedItem

if (SetProperty(ref selectedItem, value))
{
OnPropertyChanged(nameof(TagsFlyout));
UpdateTagsItems();
OnPropertyChanged(nameof(LoadTagsList));

if (value is not null)
Expand Down Expand Up @@ -88,8 +87,7 @@ public bool LoadTagsList
PreviewPaneState is PreviewPaneStates.NoPreviewAvailable ||
PreviewPaneState is PreviewPaneStates.PreviewAndDetailsAvailable;

public MenuFlyout TagsFlyout
=> new Files.App.UserControls.Menus.FileTagsContextMenu(new List<ListedItem>() { SelectedItem });
public ObservableCollection<TagsListItem> Items { get; } = new();

public PreviewPaneViewModel(IPreviewPaneSettingsService previewSettings, IContentPageContext contentPageContextService = null)
{
Expand Down Expand Up @@ -374,6 +372,17 @@ private void SelectedItem_PropertyChanged(object? sender, PropertyChangedEventAr
{
if (e.PropertyName is nameof(ListedItem.HasTags))
OnPropertyChanged(nameof(LoadTagsList));
else if (e.PropertyName is nameof(ListedItem.FileTagsUI))
UpdateTagsItems();
}

private void UpdateTagsItems()
{
Items.Clear();

SelectedItem?.FileTagsUI?.ForEach(tag => Items.Add(new TagItem(tag)));

Items.Add(new FlyoutItem(new Files.App.UserControls.Menus.FileTagsContextMenu(new List<ListedItem>() { SelectedItem })));
}

public void Dispose()
Expand Down