<div dir="ltr"><br><div class="gmail_extra"><br><div class="gmail_quote">On 6 April 2018 at 14:28, Ralph Versteegen <span dir="ltr"><<a href="mailto:teeemcee@gmail.com" target="_blank">teeemcee@gmail.com</a>></span> wrote:<br><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>I remember this problem. It's caused by the following code in init_menu_state (for MenuDef):<br><br>  .size = menu.maxrows - 1<br>  IF .size = -1 THEN .size = 17<br><br></div>This hardcoded value of 17 is only suitable at 320x200 and only if the itemspacing and borders haven't been changed. Obviously this needs to be fixed, but last time I was makingchanges to the menu sizing code I left it alone for because I didn't want to deal with it then, out of caution (eg backcompat).<br></div>I don't think we should expect any backcompat problems changing it, because few games run at a different resolution. If you did something like reducing the item spacing to squeeze your 18 menu items together to leave space to display something above or below the menu, only then would changing it be a problem, but that sounds very unlikely to have occurred.<br></div></blockquote><div><br></div><div>Oh, and I see this only changes .size, not the size of the menu's box... I think I changed how the latter was calculated without changing the former, so I probably already broke the above hypothetical game relying on current default maxrows behaviour anyway?<br></div><div>Menu sizing really is a mess! I hate the fact that we have MenuState.has_been_drawn, that you have to call draw_menu/standardmenu to correctly calculate the size and position of the menu, which leads to momentary glitches when you enter some menus like the Battle System Options. The way I see to fix it is to put a bunch of arguments like the location at which a menu is drawn in MenuState, rather than passing it as args to draw_menu/standardmenu.<br></div><div> </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><div class="gmail_extra"><br><div class="gmail_quote"><div><div class="gmail-h5">On 6 April 2018 at 09:43, James Paige <span dir="ltr"><<a href="mailto:Bob@hamsterrepublic.com" target="_blank">Bob@hamsterrepublic.com</a>></span> wrote:<br></div></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex"><div><div class="gmail-h5"><div dir="ltr"><div><div><div>I noticed recently that when a menu is set to:<br><br></div>Max rows to display: Default<br><br></div>The sizing always appears to be wrong for menus longer than a certain size. I am not sure if this has ever worked correctly.<br><br></div>I think it is possible that anyone who has made a menu long enough to scroll has always had to just put a specific number in the "Max rows to display" to get it to look right.<br></div>
<br></div></div>______________________________<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>
<br></blockquote></div><br></div>
</blockquote></div><br></div></div>