Tweaking Reconcile window #99
Merged
Conversation
This file contains 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
We've decided in the past that we won't treat "checks" any different than an income/spending transaction. Several changes in that regard were already made, e.g. not react on numbers entered into the "Type" field, which itself was removed. This commit removes the last remnants of "checks" from the app, including the last visible place, the Reconcile window.
* There's enough space to have all textboxes in one row. * Put totals above scrollview lists to serve as a kind of header. * Center "Unreconciled total" below lists. * Start with a larger window.
Similar to the transactions list of the main window.
* Get the widest label of all text fields at the top of the window and let them all be the same width. * Set explicit size for deposit/charge totals to avoid resize-jumping when changing the total amount by double-clicking to reconcile an item in the list.
Do the currency formatting after creating the text boxes and when the user entered a value by calling the CurrencyBox's Validate(). Remove left-over SetExplicitMaxSize() calls.
|
Great, the layout jump when selecting items in the lists was annoying, now the window looks much cleaner. |
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.
Remove the "Checks" view, as we've decided to not special-trat those and various tweaks.
Screenshots:
old:
new:
