Skip to content
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.

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

feat: add mock UI for file syncing listing #60

Merged
merged 5 commits into from
Mar 25, 2025

Conversation

deansheather
Copy link
Member

@deansheather deansheather commented Mar 19, 2025

Adds a file sync listing window with mocked data. Since it's difficult (I think) to disable sections of XAML based on release/debug builds, this mocked window replaces the Sign out button on debug builds.

  • Adds FileSyncListWindow, FileSyncListViewModel, FileSyncListMainPage
  • Adds MutagenSessionModel as well as useful helpers
  • Adds DependencyObjectSelector for mapping enum values in XAML
    • This is necessary as there doesn't seem to be another way of binding to a ThemeResource with a map

image
image

Verified

This commit was created on GitHub.com and signed with GitHub’s verified signature.
@@ -132,11 +122,10 @@
_sessionCount += 1;
}

return session;
throw new NotImplementedException();
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This breaks the UTs --- and doesn't seem to be related to the UI view. Did it get added by accident to this PR?

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

My bad, with the type changes I made I had to change the output of this function and wasn't thinking about UTs lol.

@deansheather deansheather merged commit e0a11dd into main Mar 25, 2025
3 checks passed
@deansheather deansheather deleted the dean/file-sync-ui-mock branch March 25, 2025 11:33
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants