Skip to content
This repository was archived by the owner on Jun 21, 2023. It is now read-only.

Fixing validation display on Create Repository dialog #2308

Merged
merged 5 commits into from
Apr 5, 2019

Conversation

StanleyGoldman
Copy link
Contributor

@StanleyGoldman StanleyGoldman commented Mar 29, 2019

Related #2264

Depends on:

image

image

image

image

@@ -107,17 +107,21 @@
Grid.Column="1"
MaxLength="{x:Static ghfvs:Constants.MaxRepositoryNameLength}"
Text="{Binding RepositoryName, UpdateSourceTrigger=PropertyChanged}"
SpellCheck.IsEnabled="True"
Copy link
Collaborator

Choose a reason for hiding this comment

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

It looks like spell checking has been disabled. Was this by design?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Yea, I disabled it, when I couldn't type VisualStudio without it calling that an error, I decided that it was not useful...

Copy link
Collaborator

Choose a reason for hiding this comment

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

Lol, makes sense. Though I'm perfectly capable of misspelling the name of my repo. 😉

Copy link
Collaborator

Choose a reason for hiding this comment

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

It look like dotcom also has spell checking disabled fro this field.

image

Choose a reason for hiding this comment

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

ok

@StanleyGoldman
Copy link
Contributor Author

Added screenshots

Copy link
Collaborator

@jcansdale jcansdale left a comment

Choose a reason for hiding this comment

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

Looks good. 😄

I tested with the dark theme...
image

@jcansdale jcansdale merged commit 56c347b into master Apr 5, 2019
@jcansdale jcansdale deleted the fixes/repository-creation-view-errors branch April 5, 2019 16:46
@StanleyGoldman StanleyGoldman changed the title Fixing validation on create repository dialog Fixing validation display on Create Repository dialog Apr 11, 2019
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants