Fix: Column view trace not updating on theme change#18056
Fix: Column view trace not updating on theme change#18056R0h1tAnand wants to merge 1 commit intofiles-community:mainfrom
Conversation
…12577) Added ActualThemeChanged event listener to ColumnLayoutPage to detect theme changes and reapply folder background colors. - Subscribe to ActualThemeChanged in OnNavigatedTo - Unsubscribe in OnNavigatingFrom to prevent memory leaks - Reapply folder background when theme changes - Fixes issue where trace was invisible when app theme differs from system theme Follows the same pattern as SpeedGraph.cs for handling theme changes.
| currentIconSize = LayoutSizeKindHelper.GetIconSize(FolderLayoutModes.ColumnView); | ||
|
|
||
| UserSettingsService.LayoutSettingsService.PropertyChanged += LayoutSettingsService_PropertyChanged; | ||
| this.ActualThemeChanged += ColumnLayoutPage_ActualThemeChanged; |
There was a problem hiding this comment.
Can you bind to the event directly from xaml instead of manually subscribing and unsubscribing from code behind?
Unfortunately, this doesn't appear to resolve the issue. What issues are you having with compiling the project? |
yair100
added
changes requested
Extremly Sorry for the Late reply and forgive me. This might be something versy silly but I might be doing something very dumb , I am new Can you guide me where am i Making the mistake. |
|
Did you set the packing project as the startup project? |
Yes sir I did |
|
After deploying, does Files Dev come up in the Start Menu? |
|
I looked it up in my system it didn't come when tried, Is any native
windows settings blocking this ??
…On Mon, 2 Feb, 2026, 10:26 pm Yair, ***@***.***> wrote:
*yaira2* left a comment (files-community/Files#18056)
<#18056 (comment)>
After deploying, does Files Dev come up in the Start Menu?
—
Reply to this email directly, view it on GitHub
<#18056 (comment)>,
or unsubscribe
<https://github.com/notifications/unsubscribe-auth/BGRDF4OARRNZMHJEXBECBOT4J56VVAVCNFSM6AAAAACRSTUZ6SVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZTQMZWGQ3TSMJXGM>
.
You are receiving this because you authored the thread.Message ID:
***@***.***>
|
|
It sounds like it wasn't deployed. Please share a screenshot of Visual Studio. |
I will try again and will update you ASAP |
2 participants
Resolved / Related Issues
Quality Checklist
Steps used to test these changes
Due to system constraints with UWP packaging and deployment, I was unable to run the full app locally to test the fix at runtime. However, I have completed the following verification:
SpeedGraph.cs(lines 88-95)Implementation Details
The fix adds an
ActualThemeChangedevent handler toColumnLayoutPage.xaml.csthat reapplies the folder background color when the app theme changes. This ensures the visual trace (highlighted folder background) updates correctly when switching between Light/Dark themes.Changes Made:
OnNavigatedTo():this.ActualThemeChanged += ColumnLayoutPage_ActualThemeChanged;OnNavigatingFrom():this.ActualThemeChanged -= ColumnLayoutPage_ActualThemeChanged;SetFolderBackground()to reapply the theme-appropriate background colorTesting Request
Due to the system limitations mentioned above, I was unable to verify the fix in the running application. It would be greatly appreciated if a maintainer or reviewer could test this change to confirm it resolves the issue as expected. I am eager to address any feedback and make additional changes if needed to ensure this fix works correctly.
The solution builds successfully and the implementation follows established patterns in the codebase, so I'm confident in the approach, but runtime verification would be valuable.