[adnan-bhaldar]

Tip

Start the title with a prefix so the PR is labeled automatically: feat: fix: docs: ci: build: i18n:
Example: fix(hwp): keep tables when converting to DOCX

What does this change?

Adds a button to the main window's sidebar, just above the update, GitHub and Sponsor icons, that collapses it to an icon-only rail and expands it again. The logo and app name stay at the top.

• Collapsed: titles, tagline and version are hidden. The nav items, Update, GitHub and Sponsor show only their icon, each with a tooltip.
• The width animates, and doesn't animate when reduced motion is on.
• The state is saved in settings.json (AppSettings.NavCollapsed), so it survives the window going to the tray and coming back. Settings from before this change start expanded, so no schema step is needed.
• Accessibility: while collapsed, each nav row keeps its title as its screen-reader name (AutomationProperties.Name), because the visible text is hidden.
• While in here: hovering a nav item now nudges only the icon, so the highlighted background no longer slides with it.
• MainWindowViewModel now takes a UserDataStore, so the existing tests that construct it were updated to pass one.

Type of change

• Bug fix
• New feature or improvement
• New format or engine
• Translation
• Documentation
• Build, CI or dependencies

How was it tested?

• dotnet format --verify-no-changes
• dotnet test
• Tried it in the app (what you converted or clicked, and on which OS):
  • OS: Windows 11
  • Collapsed and expanded the sidebar, hovered every item and the bottom icons, switched pages while collapsed, closed to the tray and reopened (state is kept), switched language (tooltips follow), tried with an update available.

New tests in Filee.App.Tests:

• NavCollapseTests (new file):
  • Collapsing saves NavCollapsed, and a new view model starts collapsed.
  • Expanding saves it too.
  • Settings from before this change start expanded.
• AutomationNameTests: while collapsed, every nav item has its title as both its tooltip and its automation name. Expanded, there is no tooltip.
• 14 existing test files were updated for the new MainWindowViewModel constructor argument.

Screenshots (UI changes)

State	Screenshots
Expanded sidebar
Collapsed sidebar
Collapsed, tooltip on a nav item
Collapsed, update available
Hovering a nav item (expanded)
Dark theme

Checklist

• New UI strings are in all src/Filee.App/Assets/i18n/*.json files (including Indonesian, rebased onto i18n: add Indonesian language support (UI translation and platform resources) #790), and each file ends as it does on main
• New third-party code or binaries are listed in THIRD-PARTY-NOTICES.md (none added)
• Tests added for the new behavior, and existing tests updated so the test project builds
• The PR does one thing (the sidebar; the hover tweak, translations and tests are separate commits)

[KnifeLemon]

Thanks, @adnan-bhaldar! The rail looks tidy in both themes, and it's good that it keeps its state, uses tooltips and respects reduced motion. A few things before it can go in:

Must fix

1. The tests don't build. MainWindowViewModel now needs a UserDataStore, but about a dozen test files still construct it with three arguments (AutomationNameTests, BrowserDropTests, CloseActionTests, WhatsNewTests, WhatsNewMedia, DonutFlipTests, DonutOpacityTests, …), so CI fails at compile time. Please update them and run dotnet test. The template's dotnet test box is unchecked.
2. Indonesian: id.json reached main just now (#790). Please rebase and add nav.collapse / nav.expand there too.
3. File endings: every i18n/*.json loses its final newline. Please keep the files ending as they do on main.

Tests for the feature

Please add a test or two in Filee.App.Tests:

• collapsing saves NavCollapsed and a new view model starts collapsed;
• while collapsed, every nav item has its title as a tooltip and an automation name.

AutomationNameTests is a good place to start.

Next time

For a change to the main window like this one, please open an issue first, so the design can be agreed before the code is written and nobody spends time on a version that changes.

[adnan-bhaldar]

Thanks for the review, @KnifeLemon! I've pushed the changes:

1. Tests build again. All the MainWindowViewModel call sites in the tests now pass a UserDataStore. dotnet test passes locally.
2. Indonesian. Rebased onto main and added nav.collapse / nav.expand to id.json.
3. File endings. The i18n files now end the same way as on main (it.json had gained a final newline, which I removed).

New tests:

• NavCollapseTests: collapsing saves NavCollapsed, a new view model starts collapsed, and settings from before this change start expanded.
• AutomationNameTests: while collapsed, every nav item has its title as its tooltip and its automation name.

Writing the second test showed that the collapsed items had no accessible name, because the title text is hidden. I added AutomationProperties.Name to the nav row to fix that.

And noted on opening an issue first for main-window changes. I'll do that next time so the design can be agreed before I write the code.

I also updated the PR description with the tests and the changes above.

[KnifeLemon]

Thanks, @adnan-bhaldar! Everything from the review is in:

• the tests build again;
• id.json has the two strings;
• the file endings match main.

The new tests caught a real gap: collapsed items had no accessible name, and you fixed it. CI is green on all platforms.