<div dir="ltr"><div>It feels like they got repurposed for something other than the reason they were originally added - as evidenced by the name, which only makes much sense for things like the X and Y setting, which are a group which can also be edited together. It's confusing that there's a parallel set of rule.mode constants, eg erIntgrabber, with sometimes the rule.mode handling enter_space_click, and sometimes the rule.group handler, but I guess there's a valid reason for it. And the I made it worse by adding a third parallel track, slice_edit_updates.</div><div>Not easy to see how to make it cleaner, though. Maybe just not putting all the logic in one function.</div><div><br></div><div>A large part of the problem is that the slice editor bypasses the accessor/mutator functions in slices.bas, like ChangeRectSlice, so has to reimplement all that logic (and generally fails to match it). And that safe mutator functions don't exist for a lot of the complex stuff anyway (eg fill mode)<br></div><div><br></div></div><div class="gmail_extra"><br><div class="gmail_quote">On 11 October 2017 at 03:36, James Paige <span dir="ltr"><<a href="mailto:Bob@hamsterrepublic.com" target="_blank">Bob@hamsterrepublic.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div dir="ltr"><div>The rule groups were probably a bad idea. ;)<br><br></div></div><div class="HOEnZb"><div class="h5"><div class="gmail_extra"><br><div class="gmail_quote">On Tue, Oct 10, 2017 at 7:11 AM,  <span dir="ltr"><<a href="mailto:subversion@hamsterrepublic.com" target="_blank">subversion@hamsterrepublic.<wbr>com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">teeemcee<br>
2017-10-10 07:11:13 -0700 (Tue, 10 Oct 2017)<br>
183<br>
Slice editor now enforces mutual exclusion of Cover and Fill, and disables Cover when disallowed or trying to resize<br>
<br>
I'm finding slice_edit_updates more convenient then 'rule groups'<br>
---<br>
U   wip/sliceedit.bas<br>
U   wip/slices.bas<br>
______________________________<wbr>_________________<br>
Ohrrpgce mailing list<br>
<a href="mailto:ohrrpgce@lists.motherhamster.org" target="_blank">ohrrpgce@lists.motherhamster.o<wbr>rg</a><br>
<a href="http://lists.motherhamster.org/listinfo.cgi/ohrrpgce-motherhamster.org" rel="noreferrer" target="_blank">http://lists.motherhamster.org<wbr>/listinfo.cgi/ohrrpgce-motherh<wbr>amster.org</a><br>
</blockquote></div><br></div>
</div></div><br>______________________________<wbr>_________________<br>
Ohrrpgce mailing list<br>
<a href="mailto:ohrrpgce@lists.motherhamster.org">ohrrpgce@lists.motherhamster.<wbr>org</a><br>
<a href="http://lists.motherhamster.org/listinfo.cgi/ohrrpgce-motherhamster.org" rel="noreferrer" target="_blank">http://lists.motherhamster.<wbr>org/listinfo.cgi/ohrrpgce-<wbr>motherhamster.org</a><br>
<br></blockquote></div><br></div>