<div dir="ltr"><div dir="ltr"><br></div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Thu, 3 Mar 2022 at 01:39, James Paige <<a href="mailto:Bob@hamsterrepublic.com">Bob@hamsterrepublic.com</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex"><div dir="auto"><div><br><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Wed., Mar. 2, 2022, 12:46 a.m. Ralph Versteegen, <<a href="mailto:teeemcee@gmail.com" target="_blank">teeemcee@gmail.com</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex"><div dir="ltr"><div>Cover Children is still visible in privileged mode only.</div><div>Yes, refreshing is broken, and even more so if you nest it. E.g. in spriteset_resize_menu_rebuild:</div><div><br></div><div>  ' FIXME: CoverChildren is a bit broken, just refreshing doesn't properly position<br>  ' everything, and even have to draw multiple times due to nested CoverChildren.<br>  DrawSlice root, vpage<br>  DrawSlice root, vpage</div><div><br></div><div>To fix this I think I need to make a major change to how recursive drawing and refreshing slices works. In fact I wrote all that documentation at the top of slices.bas in preparation for making that change, and I have more notes.<br></div></div></blockquote></div></div><div dir="auto"><br></div><div dir="auto"><br></div><div dir="auto">Ah, okay</div><div dir="auto"><br></div><div dir="auto"><div class="gmail_quote"><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex"><div dir="ltr"><div></div><div><br></div><div>As for clamping, I couldn't remember why it was a hidden option in the commit message I see I wrote:</div><div><br></div><div>    I didn't mean to check in the slice clamping stuff already! There's a chance I<br>    might change how it works, creating a file format incompatibility if you<br>    actually use it.</div><div><br></div><div>I'm guessing that adding options to clamp in both directions at once (turning it into a 4-way bit field) was the change I was considering making. That would be a nice feature to have. In the case that the child slice is wider than its parent, it could be aligned according to either its anchor or align setting (I really can't figure out which is preferable!)<br></div></div></blockquote></div></div><div dir="auto"><br></div><div dir="auto">I could finish up that feature and enable it.</div><div dir="auto">I guess I'd prefer to use anchor to break the tie when smaller than parent. Falling back to align in the case of a center anchor, and falling back to top left in case of a fully aligned slice</div></div></blockquote><div><br></div><div>If the slice is center-anchored [or aligned], there's no need to pick either the left or right edge: I think it should be centered on the parent instead, sticking out equally on both sides! <br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex"><div dir="auto"><div dir="auto"><br></div><div dir="auto"><div class="gmail_quote"><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex"><div dir="ltr"><div></div><div><br></div><div>Why won't clamping work forkeeping the cursor on the screen? What have you got it parented to? I think it ought to be parented to the root battle slice, not to a BattleSprtie slice, for example, because it needs to appear above everything else, even if there's a large enemy in front of the one you're targetting.<br></div></div></blockquote></div></div><div dir="auto"><br></div><div dir="auto">So the targetting slices are on their own layer, and then each targeting slice is a container that has its pos and size matched to the BattleSprite it is targeting. The actual cursor is a child (or grandchild) of that.</div><div dir="auto"><br></div><div dir="auto">I think I'll want to add a special property for slices that will make Clamp options use ScreenSlice instead of parent</div></div></blockquote><div><br></div><div>That would be useful in general. <br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex"><div dir="auto"><div dir="auto"><br></div><div dir="auto"><div class="gmail_quote"><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex"><div dir="ltr"><div></div></div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Wed, 2 Mar 2022 at 17:35, James Paige <<a href="mailto:Bob@hamsterrepublic.com" rel="noreferrer" target="_blank">Bob@hamsterrepublic.com</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex"><div dir="ltr"><div>I was considering making use of CoverChildren... except it seems slightly broken.</div><div><br></div><div>I see that using it can cause the piston of the children to jump on the first visible tick.</div><div><br></div><div>I can't remember anything about CoverChildren. Looks like it is enabled in the slice editor, but barely used anywhere</div><div><br></div><div>I was also looking at slice clamping. I see it is set to be hidden as privileged in the slice editor, although as far as I can tell it works well. I was thinking of adding an option to clamp in both directions at once, but not sure if I want to go that route or not.</div><div><br></div><div>I need to do some magic to make sure that targeting cursor slices stay onscreen, but clamp won't help me with that, since clamping only respects parent not screen<br></div><div><br></div><div><br></div></div>
_______________________________________________<br>
Ohrrpgce mailing list<br>
<a href="mailto:ohrrpgce@lists.motherhamster.org" rel="noreferrer" target="_blank">ohrrpgce@lists.motherhamster.org</a><br>
<a href="http://lists.motherhamster.org/listinfo.cgi/ohrrpgce-motherhamster.org" rel="noreferrer noreferrer" target="_blank">http://lists.motherhamster.org/listinfo.cgi/ohrrpgce-motherhamster.org</a><br>
</blockquote></div>
_______________________________________________<br>
Ohrrpgce mailing list<br>
<a href="mailto:ohrrpgce@lists.motherhamster.org" rel="noreferrer" target="_blank">ohrrpgce@lists.motherhamster.org</a><br>
<a href="http://lists.motherhamster.org/listinfo.cgi/ohrrpgce-motherhamster.org" rel="noreferrer noreferrer" target="_blank">http://lists.motherhamster.org/listinfo.cgi/ohrrpgce-motherhamster.org</a><br>
</blockquote></div></div></div>
_______________________________________________<br>
Ohrrpgce mailing list<br>
<a href="mailto:ohrrpgce@lists.motherhamster.org" target="_blank">ohrrpgce@lists.motherhamster.org</a><br>
<a href="http://lists.motherhamster.org/listinfo.cgi/ohrrpgce-motherhamster.org" rel="noreferrer" target="_blank">http://lists.motherhamster.org/listinfo.cgi/ohrrpgce-motherhamster.org</a><br>
</blockquote></div></div>