Rodney,
Here are a few things to merge from uckelman-merge to the trunk:
2756: Switched some BufferedImages from 4BYTE_AGBR to INT_ARGB.
2758: Adding src dir; VASSAL and org will be moved there later.
2759: mv org src
2760: mv VASSAL src
–
J.
Messages mailing list
Messages@forums.vassalengine.org
forums.vassalengine.org/mailman/ … engine.org
Post generated using Mail2Forum (mail2forum.com)
Additionally:
2774:2778: I/O cleanup and replaced StringBuffer with StringBuilder.
2794: Replacing StringBuffers introduced in 2785.
Hi Rodney, Joel,
I have a couple of bug fixes ready to merge from swampwallaby-merge:
2854: Don’t request to start a log if logging already in progress at end of replay.
2855: Draw selection highlights on all counters in multi-selection drag ghosts.
Do these get merged into the trunk, then to Joel’s regular builds, or into Joel’s and then to trunk when done?
Also, Rodney, how are you going with the integration of the Plugins changes. I found and reapplied them to swampwallaby-merge@2787.
Regards,
Brent.
Thus spake “Brent Easton”:
Thanks for fixing that one. I’ve tried several times myself, with no
success.
I guess that depends on whether Rodney wants them for a 3.0.18 release.
If so, then they should be merged to the trunk.
I’m going to merge them to my branch so I can test them anyhow.
–
J.
Messages mailing list
Messages@forums.vassalengine.org
forums.vassalengine.org/mailman/ … engine.org
Post generated using Mail2Forum (mail2forum.com)
Hi Joel, Rodney,
Have also killed the Deleting Basic Piece bug in swampwallaby-merge@2856
Regards,
Brent.
This is still a bit off.
Example: Select more than one stack, and start a drag. Every stack except
the one over which you started the drag disappears.
Otherwise, it works great. I’m glad that this bug finally got some attention
from someone who knows how to fix it.
Cool, that works.
I just noticed one other thing which we could fix w/r/t this dialog:
The Add button is enabled with BasicPiece selected in the Available
list when the dialog comes up. The Add button is disabled like it
should be if you select something else, an then come back to BasicPiece.
I’m fixing that now.
Come to think of it, is there any reason at all to show BasicPiece in the
Available list when you can’t remove it from the Current list?
Ok, that’s weird. I’ll have another look.
I remember it took me a full week to investigate the Drag Handling stuff the first time I looked at it. Knew where the bodies where buried this time.
Brent Easton
Analyst/Programmer
University of Western Sydney
Email: b.easton@uws.edu.au
Messages mailing list
Messages@forums.vassalengine.org
forums.vassalengine.org/mailman/ … engine.org
Post generated using Mail2Forum (mail2forum.com)
Speaking of Selection highlights, since your looking at/fixing it I noticed this odd behavior with it. See the image attached.
In the first pic you see two pieces side by side with white highlighted selection borders around them. These two pieces are actually one stack that has a stack layer offsetting the vertical the exact width of the counter making them appear to be ‘side by side’, so the piece on the right is in reality the top piece.
In the second pic the stack has been deselected - notice the "yellowout’ on the underneath piece but the top piece still has its selection border. Is it really supposed to do that? I havent noticed this when stacks are not offset by a full piece width - they seem to still display underneath as per normal.
Pics 3 and 4 are of when the stack is expanded (the offset is the same) and the yellowout disappears and the selection border returns to normal behavior per piece selected
Joel,
swampwallaby-merge@2861 has an improved version that fixes this properly.
Cheers,
Brent.
Thus spake “Brent Easton”:
Got it. Works for me. Merged it to uckelman-caching@2862. Bug closed.
Thanks for fixing this!
–
J.
Messages mailing list
Messages@forums.vassalengine.org
forums.vassalengine.org/mailman/ … engine.org
Post generated using Mail2Forum (mail2forum.com)
Actually, I’m still not happy with it. If you stack up about 8 counters, the top edge of the Ghost stack image is missing. It’s something in the change I’ve made, but the change is so simple I can’t see why.
Another minor issue is that the Ghost uses a default StackMetrics, not the current Map’s StackMetrics when drawing the ghosts, so the Stacking offsets in the ghosts can be different to the real counters.
Tim, I could not reproduce your problem in my small test module.
Joel,
swampwallaby-merge@2863 fixes the Stackmetrics problem. The Ghosts now use the map’s metrics and show the stacking correctly.
There is one remaining issue which is not an easy fix. If you lasso a single counter and a stack and move both by dragging the single counter, then the stack is drawn as a single counter (the top one) only. I’ll have a think about that one tomorrow.
Brent.
Brent,
I was able to reproduce it in a clean mod - no custom code. I’ve attached the demo. Didn’t realize this till I went back and looked at my original just now, but it would seem to have to do with something of counters that are created/ built up with layers only w/ a transparent basic piece or null image
Thus spake “Brent Easton”:
Hmm. This might be something which is no longer a problem in my builds.
E.g., I stacked up about 60 pieces in Ben’s Case Blue mod and the
ghost painted properly for them.
nomic.net/~uckelman/tmp/vass … vn2862.zip
–
J.
Messages mailing list
Messages@forums.vassalengine.org
forums.vassalengine.org/mailman/ … engine.org
Post generated using Mail2Forum (mail2forum.com)
While we’re looking at this, Brent, do you know what the difference is
between makeDragImage() and makeDragCursor()? They seem very similar to
me, so much so that you’ve made the same changes in both. I wonder if
it would be possible to refactor and delete some duplicate code here?
Thus spake “Brent Easton”:
Indeed. That’s weird. I’ll keep the bug open.
–
J.
Messages mailing list
Messages@forums.vassalengine.org
forums.vassalengine.org/mailman/ … engine.org
Post generated using Mail2Forum (mail2forum.com)
You only see it if you change the Stacking Options from default, especially reversing the vertical and horizontal offset factors.
Yes, I think they can be refactored and combined. I may need to do a bit of a rewrite anyway to handle this stacking issue. Leave it with me.
Brent.
Tim,
This is in fact the programmed behaviour. Unselected counters below the top counter in unexpanded Stacks are not drawn specifically, but are drawn as a rectangle with the color as set in the Stacking Options properties.
I guess no-one expected you to push the unexpanded stack offset by that much.
Regards,
Brent.
Hmm, ok good to know. Would it redraw the underneath piece if we are able to
set the Stacking Options color to transparent, or would vassal just look
through the lower piece as if non existent displaying underneath map.
Anyway don’t alter it regardless. I can live with it as is.
I’ll see if I can’t find an alternate method of offsetting counters which
allows automatic ‘collapsing’ when a piece is pulled out of middle of an
expanded stack, without using stacks - but may be difficult to pull off 
Post generated using Mail2Forum (mail2forum.com)