<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><div class="gmail_extra"><br><div class="gmail_quote">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><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><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>______________________________<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>