Load character dialog should remember last directory

Description

The dialog to load a character should remember the last directory used in the current session. Currently it always opens into the directory defined as default for character in the settings. If you are working on several characters in on of several campaigns, it can become cumbersome to have to navigate to the correct sub-folder for every character.

My proposed behavior: when brining up this dialog, use the directory from which the last character was loaded. If no character has yet been loaded in the current session, use the default as defined in the program settings.

If I remember correctly, that was the also behavior in the old GUI.

Environment

None

Activity

Show:
Andrew Maitland
February 7, 2013, 7:25 AM

I don't mind this proposal, however, I believe the change was due to paths to saved companions. Might want to check with James before changing this behavior so we don't break things.

James Dempsey
February 7, 2013, 4:32 PM

Andrew is correct, a lot of work went into keeping the link between companions and masters intact. Part of this was fixing the previous behaviour of changing the base character folder every time you opened or saved a character.

Your proposed behaviour sounds good to me, but it will need new storage of the current folder, not the scheme that was used previously.

Stefan Radermacher
February 7, 2013, 5:56 PM

Yes I remember the old problem, but this happened because this path was weirdly mixed up with the path actually used to load a character or companion, and this was a certainly a bug. I think this should not be difficult to implement correctly. I can do this but I need a description of what exactly should not happen.

Stefan Radermacher
February 8, 2013, 10:23 AM

Implemented as of Subversion revision 19376.

This change only affects the actual file chooser dialog used to open and save characters, which will now behave like this: when first called, it will open in the default directory configured in the settings dialog. On subsequent callings it will open in the last directory used to load or save a character.

Tested positively not to interfere with the loading of companions.

Andrew Maitland
February 8, 2013, 10:37 AM

Fix version needs to be an actual version, not a placeholder.

Assignee

Stefan Radermacher

Reporter

Stefan Radermacher

Labels

None

Theme

New User Interface

Epic/Theme

None

Pending User Input

None

Components

Fix versions

Affects versions

Priority

Minor
Configure