Skip to content
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

[Bug]: Restore Layout malfunctioning, showing empty macro frames from past games on right side #3215

Closed
melek opened this issue Nov 26, 2021 · 5 comments · Fixed by #3623
Closed
Assignees
Labels
bug tested This issue has been QA tested by someone other than the developer.

Comments

@melek
Copy link
Collaborator

melek commented Nov 26, 2021

Describe the Bug

In 1.10.3+, when I use 'Restore Layout', I'm getting unpopulated frames with titles from macro-generated frames created during prior sessions in a tabbed panel on the right hand side. I have not tested on older versions.

In 1.11.0a5 I get the Campaign panel tabbed with the Library panel on the top left, I think this is correct in 1.11.x thanks to #3135.

To Reproduce

  1. Open a new campaign on a computer which has had prior sessions which generated macro frames.
  2. Select Window → Restore layout.
  3. See error - right-hand side is occupied by a panel with tabs for old frames.

Expected Behaviour

I expect the restore layout button to behave consistently regardless of prior macro panels opened on the computer.

Screenshots

image

Here is a partial list of the tabs that come up (I closed about half so you could see the titles:
image

MapTool Info

1.10.3, 1.11.0a5, each installed in their own directory.

Desktop

Windows 10

Additional Context

Still hoping for #2712 and #1510 ... :)

@melek melek added the bug label Nov 26, 2021
@bubblobill
Copy link
Collaborator

I only had 25 empty frames when I found this
discord conversation #hacktoberfest 24/9/2021

@Irarara
Copy link
Contributor

Irarara commented Sep 12, 2022

Just noticed this. This is something that got missed when fixing #2485. I have something for it.

@Phergus
Copy link
Contributor

Phergus commented Sep 14, 2022

@melek - test the fix when you get a chance please

@melek
Copy link
Collaborator Author

melek commented Sep 15, 2022

After much consternation trying to build 'develop', indeed I can confirm that 'Restore Layout' works without creating the weird empty frame + tabs.

Success!

@Phergus
Copy link
Contributor

Phergus commented Sep 15, 2022

Thanks!

@Phergus Phergus added the tested This issue has been QA tested by someone other than the developer. label Sep 15, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug tested This issue has been QA tested by someone other than the developer.
Projects
None yet
Development

Successfully merging a pull request may close this issue.

4 participants