Movatterモバイル変換


[0]ホーム

URL:


Skip to content

Navigation Menu

Sign in
Appearance settings

Search code, repositories, users, issues, pull requests...

Provide feedback

We read every piece of feedback, and take your input very seriously.

Saved searches

Use saved searches to filter your results more quickly

Sign up
Appearance settings

feat: Sign in Window views & view models#20

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 ourterms of service andprivacy statement. We’ll occasionally send you account related emails.

Already on GitHub?Sign in to your account

Merged
deansheather merged 2 commits intomainfromspike/sign-in-views
Feb 7, 2025

Conversation

spikecurtis
Copy link
Collaborator

@spikecurtisspikecurtis commentedFeb 7, 2025
edited
Loading

Adds views for the Sign In dialog window, and a backing view model.

This is still a bit rough around the edges, but I'd like to get some eyes on it and merge to iterate.

It also needs to be integrated with some of the structural changes in#19 including dependency injection. That can happen per or post merge, depending on when#19 lands inmain.

Known issues:

  • Input validation. If you give an invalid URL, then clicking "Next" will crash the app since the hyperlink button won't accept an invalid URI.
  • Relatively janky Error message handling -- doesn't use any consistent styling.

Screenshot 2025-02-07 121242.png

Screenshot 2025-02-07 121347.png

@spikecurtisGraphite App
Copy link
CollaboratorAuthor

This stack of pull requests is managed byGraphite. Learn more aboutstacking.

@spikecurtisspikecurtis marked this pull request as ready for reviewFebruary 7, 2025 08:14
Orientation="Horizontal"
HorizontalAlignment="Center"
Spacing="10">
<Button Content="Back" HorizontalAlignment="Right"
Copy link
Member

Choose a reason for hiding this comment

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

I think the better way of doing forwards/backwards navigation is to use a<RootFrame> in the window rather than settingWindow.Content directly.

One pitfall that might make it hard to use the RootFrame's built-in navigation stuff is that I don't think it lets you instantiate the Window types yourself - which makes it hard to get a ViewModel in there.

In the TrayWindow code in my PR, I ended up just settingRootFrame.Content directly because of this limitation (and because I don't need forwards/backwards navigation). But if you could get that working it'd be more idiomatic I think

Copy link
Member

Choose a reason for hiding this comment

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

I switched it to using a Frame so we're not just touching the window content directly, but I'm not using the forward/backwards functionality yet because of the aforementioned issue.

In the future if we solve it, we could have the "Coder Desktop" text and the navigation buttons be on the Window itself, rather than in each page.

@deansheatherdeansheather merged commit641f1bc intomainFeb 7, 2025
4 checks passed
@deansheatherdeansheather deleted the spike/sign-in-views branchFebruary 7, 2025 15:10
Sign up for freeto join this conversation on GitHub. Already have an account?Sign in to comment
Reviewers

@deansheatherdeansheatherdeansheather approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

2 participants
@spikecurtis@deansheather

[8]ページ先頭

©2009-2025 Movatter.jp