Skip to content

Commit 109f2a9

Browse files
hishitetsuyaira2
andcommitted
Fix: Fixed crash that would occur when ungrouping from zoomed out view (#15639)
Co-authored-by: Yair <[email protected]>
1 parent 016afe0 commit 109f2a9

File tree

5 files changed

+8
-9
lines changed

5 files changed

+8
-9
lines changed

src/Files.App/Views/Layouts/BaseGroupableLayoutPage.cs

Lines changed: 2 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -209,10 +209,9 @@ protected virtual void ItemManipulationModel_StartRenameItemInvoked(object? send
209209
StartRenameItem();
210210
}
211211

212-
protected virtual void ZoomIn(object? sender, GroupOption option)
212+
protected override void ZoomIn()
213213
{
214-
if (option == GroupOption.None)
215-
RootZoom.IsZoomedInViewActive = true;
214+
RootZoom.IsZoomedInViewActive = true;
216215
}
217216

218217
protected virtual void FileList_SelectionChanged(object sender, SelectionChangedEventArgs e)

src/Files.App/Views/Layouts/BaseLayoutPage.cs

Lines changed: 6 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -1314,6 +1314,8 @@ private void UpdateCollectionViewSource()
13141314
}
13151315
else
13161316
{
1317+
ZoomIn();
1318+
13171319
var newSource = new CollectionViewSource()
13181320
{
13191321
IsSourceGrouped = false,
@@ -1323,6 +1325,10 @@ private void UpdateCollectionViewSource()
13231325
}
13241326
}
13251327

1328+
protected virtual void ZoomIn()
1329+
{
1330+
}
1331+
13261332
protected void SemanticZoom_ViewChangeStarted(object sender, SemanticZoomViewChangedEventArgs e)
13271333
{
13281334
if (e.IsSourceZoomedInView)

src/Files.App/Views/Layouts/ColumnLayoutPage.xaml.cs

Lines changed: 0 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -149,8 +149,6 @@ protected override void OnNavigatedTo(NavigationEventArgs eventArgs)
149149

150150
base.OnNavigatedTo(eventArgs);
151151

152-
FolderSettings.GroupOptionPreferenceUpdated -= ZoomIn;
153-
FolderSettings.GroupOptionPreferenceUpdated += ZoomIn;
154152
UserSettingsService.LayoutSettingsService.PropertyChanged += LayoutSettingsService_PropertyChanged;
155153

156154
SetItemContainerStyle();

src/Files.App/Views/Layouts/DetailsLayoutPage.xaml.cs

Lines changed: 0 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -159,7 +159,6 @@ protected override void OnNavigatedTo(NavigationEventArgs eventArgs)
159159
ParentShellPageInstance.FilesystemViewModel.EnabledGitProperties = GetEnabledGitProperties(ColumnsViewModel);
160160

161161
FolderSettings.LayoutModeChangeRequested += FolderSettings_LayoutModeChangeRequested;
162-
FolderSettings.GroupOptionPreferenceUpdated += ZoomIn;
163162
FolderSettings.SortDirectionPreferenceUpdated += FolderSettings_SortDirectionPreferenceUpdated;
164163
FolderSettings.SortOptionPreferenceUpdated += FolderSettings_SortOptionPreferenceUpdated;
165164
ParentShellPageInstance.FilesystemViewModel.PageTypeUpdated += FilesystemViewModel_PageTypeUpdated;
@@ -186,7 +185,6 @@ protected override void OnNavigatingFrom(NavigatingCancelEventArgs e)
186185
{
187186
base.OnNavigatingFrom(e);
188187
FolderSettings.LayoutModeChangeRequested -= FolderSettings_LayoutModeChangeRequested;
189-
FolderSettings.GroupOptionPreferenceUpdated -= ZoomIn;
190188
FolderSettings.SortDirectionPreferenceUpdated -= FolderSettings_SortDirectionPreferenceUpdated;
191189
FolderSettings.SortOptionPreferenceUpdated -= FolderSettings_SortOptionPreferenceUpdated;
192190
ParentShellPageInstance.FilesystemViewModel.PageTypeUpdated -= FilesystemViewModel_PageTypeUpdated;

src/Files.App/Views/Layouts/GridLayoutPage.xaml.cs

Lines changed: 0 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -129,8 +129,6 @@ protected override void OnNavigatedTo(NavigationEventArgs eventArgs)
129129

130130
currentIconSize = FolderSettings.GetRoundedIconSize();
131131

132-
FolderSettings.GroupOptionPreferenceUpdated -= ZoomIn;
133-
FolderSettings.GroupOptionPreferenceUpdated += ZoomIn;
134132
FolderSettings.LayoutModeChangeRequested -= FolderSettings_LayoutModeChangeRequested;
135133
FolderSettings.LayoutModeChangeRequested += FolderSettings_LayoutModeChangeRequested;
136134
UserSettingsService.LayoutSettingsService.PropertyChanged += LayoutSettingsService_PropertyChanged;

0 commit comments

Comments
 (0)