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

Fix Grid Size labelling - Again #4350

Closed
wants to merge 12 commits into from
Closed

Conversation

bubblobill
Copy link
Collaborator

@bubblobill bubblobill commented Oct 28, 2023

Identify the Bug or Feature request
Fixes #4349

Description of the Change
Had to delete unapproved Hide Token Stack controls on preferences dialog in order to get this to run.
Preferences, Map, and Grid Adjust dialogues now consistently refer to Grid Size with a tooltip detailing it as cells size in pixels

Possible Drawbacks
none

Documentation Notes
Fix inconsistency in UI reference to grid size

Release Notes
Fix inconsistency in UI reference to grid size


This change is Reviewable

@bubblobill bubblobill changed the title Again Fix Grid Size labelling - Again Oct 29, 2023
added GridRenderStyle to ZoneRenderer
changed default grid colour to translucent yellow
draw half hexes not full hexes
draw two colours
Introduced some enums and classes to support interesting grids

Signed-off-by: bubblobill <[email protected]>
Moved grid related things to package net.rptools.maptool.model.grid
Introduced some enums and classes to support interesting grids

Signed-off-by: bubblobill <[email protected]>
optimising
created assorted i18n keys for Grids

Signed-off-by: bubblobill <[email protected]>
@thelsing
Copy link
Collaborator

thelsing commented Nov 2, 2023

This PRs does much more than specified according to the file diff:

  • changes package of Grid (this might be problematic when loading old campaigns because of xstream)
  • changes several defaults for grids: defaultgrid, default grid color,...
  • adds some other options to grids

As the changes are not related to #4350 this was probably not intended?

Signed-off-by: bubblobill <[email protected]>
Signed-off-by: bubblobill <[email protected]>
@bubblobill bubblobill closed this Nov 3, 2023
@bubblobill
Copy link
Collaborator Author

Oh FFS. I hate you GItHub and your IntelliJ friend.

@bubblobill bubblobill deleted the again branch November 4, 2023 09:19
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants