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

Themes.xaml --> Sould it be Gray or Grey? #3575

Closed
timunie opened this issue Aug 19, 2019 · 7 comments · Fixed by #3583
Closed

Themes.xaml --> Sould it be Gray or Grey? #3575

timunie opened this issue Aug 19, 2019 · 7 comments · Fixed by #3583
Milestone

Comments

@timunie
Copy link
Collaborator

timunie commented Aug 19, 2019

Describe the bug
I really like the way the theme resources are called now, because it is more intuitive. But what i don't understand is, the Name of the Gray Brushes.

Have a look in the file Theme.xaml:

    <SolidColorBrush x:Key="MahApps.Brushes.SemiTransparentGrey" Color="#40808080" options:Freeze="True" />

[....]

    <SolidColorBrush x:Key="MahApps.Brushes.Gray1" Color="{StaticResource MahApps.Colors.Gray1}" options:Freeze="True" />

To Reproduce
Just have a look here:
https://github.com/MahApps/MahApps.Metro/blob/develop/src/MahApps.Metro/Styles/Themes/Theme.Template.xaml

Expected behavior
I am not a native english speaker, so I cannot tell you which spelling for gray or grey is the correct version. But we should use only one version imo.

Screenshots
If applicable, add screenshots to help explain your problem.

Environment(please complete the following information):

  • MahApps.Metro version 2.0.0 latest alpha

Happy coding
Tim

@makingbloke
Copy link

makingbloke commented Aug 19, 2019 via email

@amkuchta
Copy link
Contributor

@makingbloke the fact that you have "bloke" in your username confirms that you are, in fact, British.

As an American, I actually side with @makingbloke - I personally prefer the British spelling of the word.

@punker76
Copy link
Member

@timunie The Grey was introduced by this PR.

I would prefer changing this to Gray because it's just these 2 places.

@timunie
Copy link
Collaborator Author

timunie commented Aug 26, 2019

@punker76 I think you are right. Then it would be easier for other people, too.

Happy coding
Tim

@Evangelink
Copy link
Contributor

Usually in IT american english wins over british ;)

@makingbloke
Copy link

makingbloke commented Aug 27, 2019 via email

@punker76 punker76 added this to the 2.0.0 milestone Sep 2, 2019
punker76 pushed a commit to timunie/MahApps.Metro that referenced this issue Sep 2, 2019
Renamed it this way to reduce the effort of renaming all other Resources
punker76 added a commit to timunie/MahApps.Metro that referenced this issue Sep 2, 2019
@daiplusplus
Copy link

daiplusplus commented Jun 17, 2020

(I'll preface my answer by saying I'm British myself, albeit living in Microsoft's back-yard in Redmond)

Putting national-pride aside - the official .NET Framework Style Guidelines do make it clear that en-US spelling should always be preferred, and because it's consistent with the main .NET class libraries which use en-US spelling, especially in System.Drawing.Colors where it's Gray, not Grey, sorry: https://docs.microsoft.com/en-us/dotnet/api/system.drawing.color.gray?view=netcore-3.1

I feel a small insignificant cringe in my head whenever I review Unity (game-engine) code because they named one their main interfaces Behaviour rather than Behavior - not at all because it's not en-GB (or en-AU, en-CA etc), but because of the fact it's inconsistent - and I'd rather not feel the same way when I'm using the otherwise wonderful MahApps.Metro library.

I can imagine a non-native English speaker who might be completely oblivious to the differences between Grey and Gray who is converting WinForms+GDI code to WPF+MahApps.Metro at 3am and just pounding their head on the keyboard because their code won't compile because they didn't expect something as simple as colour names to be arbitrarily different compared to the rest of the .NET library ecosystem.

I vote for grey being a British English speaker :) too much Americanisation of our language going on lol ;)

If we want MahApps.Metro to counter Americanism (I won't go as far as saying "cultural imperialism"[1]) in the software industry it should be by embodying the best parts of British values, not something as patently silly as spelling. Pink Floyd said it best in 1973:

Hanging on in quiet desperation is the English way

...but I'll also settle for the MahApps.Metro library including an animated GIF of Queen Liz and her corgis as an easter-egg hidden inside the main assembly.

[1] And that would be inexcusable coming from me as someone who directly-indirectly benefited from the spoils of the days of Empire...

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Development

Successfully merging a pull request may close this issue.

6 participants