MapTool 1.2 Development Release Build 8

[trevor]

I sent the screencast to your e-mail for review.

Cool, I'll take a look

[Phergus]

Really need to get FoW controls disconnected from the drawing controls.

If you are drawing with textures and have large brush sizes set when you switch to FoW, the user experience is less than ideal.

It would probably be enough if the FoW tools just defaulted back to a 1 pixel line, black/black colors, no snap-to-grid, draw mode (not erase mode).

[trevor]

Good call. I had disconnected them, but then I added the pen size and didn't add that to the disconnect list. I'll get that adjusted for the next build

[trevor]

Going to need to catch null strings in the Vision dialog:

java.lang.NumberFormatException: For input string: ""
at java.lang.NumberFormatException.forInputString(Unknown Source)
at java.lang.Integer.parseInt(Unknown Source)
at java.lang.Integer.parseInt(Unknown Source)
at net.rptools.maptool.client.ui.VisionDialog.commit(VisionDialog.java:153)
at net.rptools.maptool.client.ui.VisionDialog.access$0(VisionDialog.java:149)
at net.rptools.maptool.client.ui.VisionDialog$1.actionPerformed(VisionDialog.java:123)

This is fixed for 1.2b9 contributed by fobbo

[trevor]

What if you were running an adventure and the characters one lantern ran out of oil?

Presumably you'd disable the lantern vision instead of deleting it.

When I think about the toggle implementation, it may not be looking for disabled vision. I'll put that check in.

Yeah, you'll need to put that check in because that's what I did.

This is fixed in 1.2b15

[trevor]

It would probably be enough if the FoW tools just defaulted back to a 1 pixel line, black/black colors, no snap-to-grid, draw mode (not erase mode).

This has been updated for topology and fog tools for 1.2b16