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]: If a map with templates is copied and the copy's [Map -> Edit map… -> Pixels per Cell] is changed, templates sizes change on the original map #4526

Closed
Baaaaaz opened this issue Dec 4, 2023 · 3 comments · Fixed by #4753 or #4763
Assignees
Labels

Comments

@Baaaaaz
Copy link

Baaaaaz commented Dec 4, 2023

Describe the Bug

If a map with templates is copied and the copy's [Map -> Edit map… -> Pixels per Cell] is changed, templates sizes change on the original map.

To Reproduce

Create a map and add all 8 template types.
Make a copy of this map.
Change the map scale using [Map -> Edit map… -> Pixels per Cell] to 50.
Select the original map (which should have a Pixels per Cell of 100)
Observe how the original templates have change size (well apart from Burst and Blast templates (which now in this case #4524 works in this issue's favour!)

Expected Behaviour

Changing the grid size on one map should not affect templates on another map (even if they are copies).
Drawing IDs for templates in a campaign should be unique (even if they are copies).

Screenshots

Original "Grasslands" Map:
image

"Copy of Grasslands" map with changed Pixels per Cell = 50. Templates have changed size (well, apart from Burst & Blast templates)
image

Now select original "Grasslands" map, and see how templates have incorrectly changed size
image

MapTool Info

Occurs in 1.13.2 & 1.14.3

Desktop

Windows

Additional Context

No response

@Baaaaaz Baaaaaz added the bug label Dec 4, 2023
@Baaaaaz
Copy link
Author

Baaaaaz commented Dec 4, 2023

Also note this bug only seems to be occur by changing the Pixels per Cell of the last of copy of a map. Does not seem to be occur when changing the scale of the original or intermediate copies. e.g.

  • take a map with templates on, copy it and then copy the copy. Changing the copy of the copy map's Pixels per Cell will also change the respective templates on both the original and copy maps.
  • take a map with templates on, copy it and then copy the copy. Changing the original map's Pixels per Cell does not affect the templates on the copy or copy of the copy maps.
  • take a map with templates on, copy it and then copy the copy. Changing the copy map's Pixels per Cell does not affect the templates on the original or copy of the copy maps.

@Baaaaaz
Copy link
Author

Baaaaaz commented Dec 4, 2023

Also note: Using Draw Explorer we can see that the Drawing ID for a templates are the same across the copied maps, which may be part of the issue here as adding templates to the original after the map is copied are not affected by changing the Pixels per Cell to the copy map.

@kwvanderlinde
Copy link
Collaborator

This looks like a special case of #2433. But while that more general bug is still present, this particular issue should be fixed in #4753 now that templates no longer hold onto grid size state.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
Status: Merged
2 participants