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.

Sign up for GitHub

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

Jump to bottom

Fix hexagon geometry #4831

Merged
merged 13 commits into from
Jun 27, 2024
Merged

Fix hexagon geometry #4831

Changes from 1 commit
Commits
Show all changes
13 commits
Select commit Hold shift click to select a range
727629d
Fix edge projection for stretched hex grids
kwvanderlinde Jun 11, 2024
e90a627
Mark Grid fields as transient
kwvanderlinde Jun 11, 2024
5a108b6
Change responsibilities: Grid.createGridArea() is no longer responsib…
kwvanderlinde Jun 11, 2024
011304b
Clear hex grid cache on changes too by not using the field directly
kwvanderlinde Jun 11, 2024
0b02e18
For hex grids, define scaled area in terms of the boundary
kwvanderlinde Jun 11, 2024
8641d34
Avoid int truncation when making hex shapes
kwvanderlinde Jun 11, 2024
4f861f1
Avoid mild precision issue in hex grid offsets
kwvanderlinde Jun 11, 2024
5e26705
Fix horizontal vs vertical offset coords; they got mixed up when movi…
kwvanderlinde Jun 11, 2024
93218d9
Remove obsolete note about grid offsets
kwvanderlinde Jun 11, 2024
008973a
Grid.createCellShape() is now only for the current size
kwvanderlinde Jun 11, 2024
f116809
Switch to scaled hexagons instead of equal-length edges
kwvanderlinde Jun 11, 2024
afc988d
Fix loading of existing campaigns to calculate shape after dimensions…
kwvanderlinde Jun 11, 2024
862f67c
Horizontal and vertical hex grids do their own grid offsets
kwvanderlinde Jun 11, 2024
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Prev Previous commit
Next Next commit
Fix horizontal vs vertical offset coords; they got mixed up when movi… 
…ng the implementation into HexGrid
  • Loading branch information
@kwvanderlinde
kwvanderlinde committed Jun 11, 2024
commit 5e26705dee54af786dc102c24ea4a8a51abccce3
4 changes: 2 additions & 2 deletions src/main/java/net/rptools/maptool/model/HexGrid.java
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -626,9 626,9 @@ protected AffineTransform getGridOffset(Token token) {
double coordinateOffsetU = -0.5 * (edgeProjection edgeLength);
if (isHexHorizontal()) {
// Swap x and y;
at.translate(coordinateOffsetU, coordinateOffsetV);
} else {
at.translate(coordinateOffsetV, coordinateOffsetU);
} else {
at.translate(coordinateOffsetU, coordinateOffsetV);
}
}

Expand Down