-
-
Notifications
You must be signed in to change notification settings - Fork 361
wxGUI: Make GUI workspace part of mapset #3113
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
base: main
Are you sure you want to change the base?
Conversation
- Automatically create workspace file in a mapset when GUI is notified about the change if it does not exist. - If workspace exists in a mapset, open it. - If workspace does not exists and but the previous mapset was in the same location, it keeps the open mapdisplays as before. - The updates to workspace are not saved automatically (in time interval, before switch, or at exit), so user needs to save the workspace (Ctrl+S) in order to keep the state. This is a proof of concept and not meant for merging. If someone wants to pick it up, I'm more than fine with that.
A possible extension or variation of the proposed change is to explicitly support more than one workspace file directly in the mapset directory. Allows for more than one workspace per mapset, but does not require much additional logic or understanding as a separate subdirectory in the mapset directory. Here is a whiteboard after discussion with @veroandreo (green lines mark the new parts for workspaces): |
Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>
Solved conflict in import |
I'm not quite sure if it makes sense to combine mapsets and workspaces in the same place. The Data tab is really just about data, while a workspace represents the state of the GUI. I’ve put the idea into a short presentation: I know this is different from the direction of the current PR, but since it hasn’t been merged yet, I thought now might be a good time to step back and consider things from another perspective. |
Maybe move this to GH discussion, it might get lost here. |
I have put it here: #5916. |
This is a proof of concept and not meant for merging. If someone wants to pick it up, I'm more than fine with that.