<div dir="ltr"><br><div class="gmail_extra"><br><div class="gmail_quote">On 31 March 2018 at 01:00, 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"><span class=""><br><br>On Thursday, March 29, 2018, Ralph Versteegen <<a href="mailto:teeemcee@gmail.com" target="_blank">teeemcee@gmail.com</a>> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div dir="ltr"><br><div class="gmail_extra"><br><div class="gmail_quote">On 30 March 2018 at 19:21, 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"><br><div class="gmail_extra"><br><div class="gmail_quote"><span>On 30 March 2018 at 16:22, 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:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex"><span><br><br>On Thursday, March 29, 2018, Ralph Versteegen <<a href="mailto:teeemcee@gmail.com" target="_blank">teeemcee@gmail.com</a>> 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"><br><div class="gmail_extra"><br><div class="gmail_quote">On 30 March 2018 at 04:01, 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:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex"><div dir="ltr"><br><div class="gmail_extra"><br><div class="gmail_quote"><span>On Wed, Mar 28, 2018 at 6:59 PM, 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><div><div><div><div>All this additional mouse support is very nice!<br><br></div>But in the Items menu it seems very unnatural that you have to click twice to select Done/Drop/Autosort or to swap an item to a new location. Double clicking is only necessary to select an item (use or start swapping); the extra click shouldn't be necessary for disambiguation in other cases.<br><br></div></div></div></div></div></div></blockquote><div><br></div></span><div>Excellent point. I can fix those things. :)<br></div><span><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><div><div><div></div>Also, after you double click on an item to select it, right clicking should probably deselect it, just as Esc does.<br></div></div></div></div></div></blockquote><div><br></div></span><div>That is also a great idea.<br></div><span><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><div><div></div>Actually, unlike the other menus, right click doesn't exit the Items menu.<br></div>I was initially concerned about right clicking anywhere closing menus, unlike how menus behave in Custom, but I guess it's not that important to be able to select menu items by right clicking. I see from your work on Items that menus which require selecting - Items, Shops, Equip - can easily be made to work with left click only (which is necessary on touch devices anyway)<br><br></div></div></div></blockquote><div><br></div></span><div>Yes, I really like right-click to close on non-scrolling menus, and non-modal menus that don't take up the whole screen, but on something that scrolls like save slots or items, I think right drag to scroll is important, especially for touch, since that will be a two-finger-drag to scroll.<br></div></div></div></div></blockquote><div><br></div><div>Would it be a bad idea to have both right-click to close and right-drag to scroll in the same menu? (It does sound dubious...) I can imagine it working worse with touch than mouse controls.<br></div><div> </div></div></div></div></blockquote><div><br></div></span><div>I think on a non-fullscreen menu it would make sense. The right click close would just work outside the scroll area.</div><span><div><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 class="gmail_extra"><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 class="gmail_extra"><div class="gmail_quote"><div></div><span><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></div>Oh, picksave/load is another menu where right click doesn't close.<br></div>(Also, I was interested to see the scrolling by right-dragging in action, but it seems to be pretty broken)<br></div></blockquote><div><br></div></span><div>Yeah, I don't have it quite right yet, and I might not be able to get it right until I sliceify the save slot picker.<br></div></div></div></div></blockquote><div><br></div><div>Yes, I guess you really need pixel-granularity of scrolling.<br><br></div></div></div></div></blockquote><div><br></div></span><div>Yes, that will help a lot. Also important is storing a dragging flag when readmouse.drag_dist exceeds the threshold for the first time</div></blockquote></span><div><div><br></div><div>Actually, I don't see the point of the ps.drag_scrolling flag. 
drag_dist only increases, never decreasing until the drag ends and it 
gets reset to zero.</div></div></div></div></div></blockquote><div><br></div><div>Also, isn't ps.drag_scroll_start redundant to readmouse.clickstart?<br></div><div><br></div></div></div></div></blockquote><div><br></div></span><div>I had been avoiding readmouse.clickstart because it's definition has a comment that suggests it isn't actually implemented. Maybe I was reading that comment wrong?</div></blockquote><div><br></div><div>Oh, that is confusing. It is implemented. It's just that no backend reports the position of the mouse cursor at the exact instant you clicked rather than the mouse position when the engine polls the mouse on the next frame. Of course, there's only a difference if you clicked and immediately dragged the mouse. Which could be a real problem if the framerate is only 18fps.<br></div><div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div class="HOEnZb"><div class="h5"><div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div dir="ltr"><div class="gmail_extra"><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 class="gmail_extra"><div class="gmail_quote"><div> </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>, making sure right-mouse click doesn't change focus when a dragging is happening, and disabling left-click selection when a dragging is happening</div></blockquote><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><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 class="gmail_extra"><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 class="gmail_extra"><div class="gmail_quote"><div><br></div><div>I plan to add the same right-drag scrolling to the items menu, and hopefully that will work better, since it is already slices.<br></div><span><div><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><div><div><div><br><br></div></div></div></div></div></div><div><div><div class="gmail_extra"><br><div class="gmail_quote">On 29 March 2018 at 11:54,  <span dir="ltr"><<a href="mailto:subversion@hamsterrepublic.com" target="_blank">subversion@hamsterrepublic.co<wbr>m</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">james<br>
2018-03-28 15:54:34 -0700 (Wed, 28 Mar 2018)<br>
104<br>
Mouse support in the Items menu.<br>
<br>
This was tricky with swap support, but I think it works pretty nicely.<br>
---<br>
U   wip/menustuf.rbas<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" 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></span></div><br></div></div>
<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>
<br></blockquote></div><br></div></div>
</blockquote>
</div></div><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>
<br></blockquote></div></div></div><br></div></div>
</blockquote></div><br></div></div>
</blockquote>
</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></div>