New Account with closed group selected offers to rename the group instead of the account #358

Closed
hsoft opened this Issue Jun 22, 2013 · 4 comments

Projects

None yet

1 participant

@hsoft
Owner
hsoft commented Jun 22, 2013

New accounts can be made within existing account groups, based on the currently selected line in the accounts list. If the account group selected is 'closed', then instead of naming the account you'll be asked to rename the group.

Steps:

  1. Make an Account Group
  2. Close the group (so that it's accounts are hidden)
  3. Make sure this account group is selected/highlighted in the list of accounts
  4. Press cmd+N to make a new account

Expected:

  • A new account should be created under the selected account group
  • It should be called "New Account" (possibly with a numeric suffix to ensure uniqueness)
  • The account group should be expanded to show the new account
  • The UI should immediately enter "renaming" mode on the new account, allowing the user to name it

Actual:

  • The new account is created under the selected group, with the appropriate default name ("New Account #")
  • The account group is not expanded
  • The UI enters "renaming" mode on the account group (which is still 'closed') rather than the account (which is hidden)
@hsoft hsoft closed this Jun 22, 2013
@hsoft
Owner
hsoft commented Jun 22, 2013

Yes, I can reproduce the bug under Mac OS X. There's also a similar bug on Qt.

@hsoft
Owner
hsoft commented Jun 22, 2013

(from [7f31c4c20b39]) [#358] Fixed a Qt selection glitch on New Account.
https://bitbucket.org/hsoft/moneyguru/changeset/7f31c4c20b39/

@hsoft
Owner
hsoft commented Jun 22, 2013

(from [e1020c61a4d0]) [#358] Fix selection glitch when adding account in group (OS X).
https://bitbucket.org/hsoft/moneyguru/changeset/e1020c61a4d0/

@hsoft
Owner
hsoft commented Jun 22, 2013

The bug has been fixed in the latest commit.

It's another of these bugs caused by the change in the way NSOutline item references work in 10.7.

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