-
-
Notifications
You must be signed in to change notification settings - Fork 101
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
Added Settings (user preferences) menu #98
Conversation
PixiEditor/Views/MainWindow.xaml
Outdated
@@ -83,6 +83,8 @@ | |||
Command="{Binding FileSubViewModel.SaveDocumentCommand}" CommandParameter="AsNew" /> | |||
<MenuItem Header="_Export" InputGestureText="Ctrl+Shift+Alt+S" Command="{Binding FileSubViewModel.ExportFileCommand}" /> | |||
<Separator /> | |||
<MenuItem Header="_Settings" Command="{Binding MiscSubViewModel.OpenSettingsWindowCommand}" /> |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I would expect it to be in the Edit
tab
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I was thinking about that, some software has this in File
some in Edit
. Not sure which one should we use.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Ok, I moved it to Edit
Executed="CommandBinding_Executed_Close" /> | ||
</Window.CommandBindings> | ||
|
||
<Grid Background="{StaticResource AccentColor}"> |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Might look better if you switch the colors
<Grid Background="{StaticResource AccentColor}"> | |
<Grid Background="{StaticResource MainColor}"> |
<Button Style="{StaticResource DarkRoundButton}" Margin="10 5 10 5" | ||
Command="{Binding SelectCategoryCommand}" CommandParameter="Updates">Updates</Button> | ||
</StackPanel> | ||
<Grid Grid.Row="1" Grid.Column="1" Background="{StaticResource MainColor}"> |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
<Grid Grid.Row="1" Grid.Column="1" Background="{StaticResource MainColor}"> | |
<Grid Grid.Row="1" Grid.Column="1" Background="{StaticResource AccentColor}"> |
Codecov Report
@@ Coverage Diff @@
## master #98 +/- ##
==========================================
- Coverage 52.31% 52.30% -0.01%
==========================================
Files 179 186 +7
Lines 6354 6504 +150
Branches 559 570 +11
==========================================
+ Hits 3324 3402 +78
- Misses 2916 2983 +67
- Partials 114 119 +5
Continue to review full report at Codecov.
|
Added Settings menu in
File -> Settings
with settings:Of course, added
user_preferences.json
reader and writer. The file is stored underAppData\Local\PixiEditor\user_preferences.json