Skip to content

[#6925] Fix tile & region shape rotation in V14#6932

Merged
arbron merged 5 commits into
5.3.xfrom
fix/6925
Apr 20, 2026
Merged

[#6925] Fix tile & region shape rotation in V14#6932
arbron merged 5 commits into
5.3.xfrom
fix/6925

Conversation

@arbron

@arbron arbron commented Apr 13, 2026

Copy link
Copy Markdown
Collaborator
  • Ignored size when rotating tiles because they are now positioned using their center points rather than their top-left corners
  • Added handling for all of the new shape types supported by regions except GridShapeData, because there doesn't seem to be a way to rotate those without potentially losing data
  • Fixed some calls to deprecated wall edge methods in V14
  • Fixed animating tile movement in V14

Closes #6925

- Ignored size when rotating tiles because they are now positioned
  using their center points rather than their top-left corners
- Added handling for all of the new shape types supported by
  regions except `GridShapeData`, because there doesn't seem to be
  a way to rotate those without potentially losing data
- Fixed some calls to deprecated wall edge methods in V14
- Fixed animating tile movement in V14

Closes #6925
@arbron arbron added this to the D&D5E 5.3.2 milestone Apr 13, 2026
@arbron
arbron requested a review from Fyorl April 13, 2026 21:17
@arbron arbron self-assigned this Apr 13, 2026
@arbron arbron added compatibility Relating to compatibility issues with new core versions system: region behaviors labels Apr 13, 2026
Comment thread module/data/region-behavior/rotate-area.mjs Outdated
Comment thread module/data/region-behavior/rotate-area.mjs Outdated
@Fyorl
Fyorl requested a review from dev7355608 April 14, 2026 01:06
@arbron
arbron requested a review from Fyorl April 14, 2026 17:18

@dev7355608 dev7355608 left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This behavior should probably be its own module, I think. It doesn't seem like it essential to play dnd5e or dnd5e specific to me.

Comment thread module/data/region-behavior/rotate-area.mjs Outdated
Comment thread module/data/region-behavior/rotate-area.mjs
Comment thread module/data/region-behavior/rotate-area.mjs Outdated
Comment thread module/data/region-behavior/rotate-area.mjs
Comment thread module/data/region-behavior/rotate-area.mjs Outdated
Comment thread module/data/region-behavior/rotate-area.mjs Outdated
- Use core's shape rotation in V14 rather than home-grown version,
  revert improvements to `rotateRegionShapes` since they aren't
  needed anymore
- Swap token movement to use `moveTokens` in V14 rather than
  direct update
- Use `shape.center` in V14 rather than custom logic
- Split token animations into their own `Token#animate` calls
  rather than handling manually
- Don't call refresh to `canvas.perception` in V14
@arbron
arbron requested a review from dev7355608 April 14, 2026 21:19
@Freeze023

Copy link
Copy Markdown

just tested this, works great except for lights that don't have their emission angle set to 360, those lights get moved by an offset.

Comment thread module/data/region-behavior/rotate-area.mjs
@arbron
arbron requested a review from dev7355608 April 17, 2026 17:43
@arbron
arbron merged commit 602457e into 5.3.x Apr 20, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

compatibility Relating to compatibility issues with new core versions system: region behaviors

Projects

None yet

Development

Successfully merging this pull request may close these issues.

in v14 the Rotate Area Region Behavior is bugged for Tiles and animations

4 participants