Skip to content

Commit 7556205

Browse files
authored
Feature: Added support for previewing AVI files in the Preview Pane (#18419)
Signed-off-by: Josh65-2201 <26977075+Josh65-2201@users.noreply.github.com>
1 parent 1425850 commit 7556205

3 files changed

Lines changed: 5 additions & 3 deletions

File tree

src/Files.App/ViewModels/Properties/BasePropertiesPage.cs

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -62,8 +62,9 @@ protected override void OnNavigatedTo(NavigationEventArgs e)
6262
BaseProperties = new CombinedFileProperties(ViewModel, np.CancellationTokenSource, DispatcherQueue, items, AppInstance);
6363

6464
ViewModel.IsEditAlbumCoverVisible =
65+
items.All(item => item.FileExtension is not ".avi") && (
6566
items.All(item => FileExtensionHelpers.IsVideoFile(item.FileExtension)) ||
66-
items.All(item => FileExtensionHelpers.IsAudioFile(item.FileExtension));
67+
items.All(item => FileExtensionHelpers.IsAudioFile(item.FileExtension)));
6768
}
6869
// Selection includes folders
6970
else

src/Files.App/ViewModels/Properties/Items/FileProperties.cs

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -46,8 +46,9 @@ public override void GetBaseProperties()
4646
ViewModel.LoadFileIcon = Item.LoadFileIcon;
4747
ViewModel.IsDownloadedFile = Win32Helper.ReadStringFromFile($"{Item.ItemPath}:Zone.Identifier") is not null;
4848
ViewModel.IsEditAlbumCoverVisible =
49+
Item.FileExtension is not ".avi" && (
4950
FileExtensionHelpers.IsVideoFile(Item.FileExtension) ||
50-
FileExtensionHelpers.IsAudioFile(Item.FileExtension);
51+
FileExtensionHelpers.IsAudioFile(Item.FileExtension));
5152

5253
if (!Item.IsShortcut)
5354
return;

src/Files.Shared/Helpers/FileExtensionHelpers.cs

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -87,7 +87,7 @@ public static bool IsAudioFile(string? fileExtensionToCheck)
8787
/// <returns><c>true</c> if the fileExtensionToCheck is a video file; otherwise, <c>false</c>.</returns>
8888
public static bool IsVideoFile(string? fileExtensionToCheck)
8989
{
90-
return HasExtension(fileExtensionToCheck, ".mp4", ".webm", ".ogg", ".mov", ".qt", ".m4v", ".mp4v", ".3g2", ".3gp2", ".3gp", ".3gpp", ".mkv");
90+
return HasExtension(fileExtensionToCheck, ".avi", ".mp4", ".webm", ".ogg", ".mov", ".qt", ".m4v", ".mp4v", ".3g2", ".3gp2", ".3gp", ".3gpp", ".mkv");
9191
}
9292

9393
/// <summary>

0 commit comments

Comments
 (0)