-
Notifications
You must be signed in to change notification settings - Fork 27
ModernWPF UI update #27
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
Merged
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
…ngly named). Added local Dark/Light.xaml for any extra colors that we need that are not provided by ModernWPF. Fixed TrendGraph so it uses these new colors.
fixed the Accounts view so it shows the closed accounts in a disabled color by re-instating the AccountNameColorConverter Added AppTheme.GetThemedBrush Some work on TransactionView TransactionCell colors.
Fix MoneyColorConverter to used themed colors. Add Light/Dark colors for PositiveCurrencyForegroundBrush and NegativeCurrencyForegroundBrush so dark theme can be less bright. Color DataGrid colors in AliasesView, SecurituesView and CurrenciesView. Fix colors in Splits DataGrid and Securities StockSplits grid.
…act. Fix color of status cell in transaction view. Fix background of Report search bar. Fix background & color of of report expandall button.
Why now have light/dark themed hyperlink colors. Fix expand/collapse button on reports.
…A Dark theme should use SystemAccentColorLight1 and a Light theme should use SystemAccentColorDark1
…s are automatically computed and spaced.
Add a border on the AppSettings panel.
…he top row of buttons.
Improve the margins a bit in the transaction grid view so labels are not touching the vertical lines.
… the Memo into edit mode. This one was a bit tricky unfortunately, because the row expands a few pixels the hit test was failing because it was operating on the editable row and finding different things that the user saw on the original click...
…n order to better see future diff changes
… "Standard" to "Recycling"
Fix background on TrendGraph tooltips.
Make the SecuritiesView show a stock chart for active selection.
Member
|
Greatly Approved |
jpdup
approved these changes
Feb 18, 2022
Member
jpdup
left a comment
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.
Looks good
Fix unit test so it runs clean.
Publish version 1.1.0.88.
Collaborator
Author
|
@jpdup merged and released https://github.com/clovett/MyMoney.Net/releases/tag/1.1.0.88 Phew... that was a lot of work. Still have to fix the row headers too, on the dark theme... |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Much thanks to JP for his help on this big new feature. Money.Net is now using lovely new ModernWPF UI themes. With a polished Light and Dark theme which you can changed from the settings or toggle by typing CTRL+L.