[Ohrrpgce] SVN: james/9279 Add find_plank_at_screen_pos() which, in contrast to find_plank_nearest_

[James Paige]

Oh, right! I see. Yes, I can fix that. It should indeed be finding the
first one, not the one nearest to center. I understand now.

On Wed, Sep 6, 2017 at 9:15 AM, Ralph Versteegen <teeemcee at gmail.com> wrote:

> The way find_plank_nearest_screen_pos works sounds fine for that edgecase.
> But I was talking about finding the plank under the mouse, which also uses
> the (copy-pasted) 'nearest' logic.
>
> On 7 September 2017 at 03:08, James Paige <Bob at hamsterrepublic.com> wrote:
>
>> The purpose of getting the nearest was for when a selected plank is
>> completely removed, and the selection focus needs to jump somewhere sane,
>> so it is only used in an edge-case.
>>
>> But in that edge case, I probably should do find_plank_at_screen_pos()
>> first, and only fall back to find_plank_nearest_screen_pos() if the
>> first fails.
>>
>>
>> On Wed, Sep 6, 2017 at 6:12 AM, Ralph Versteegen <teeemcee at gmail.com>
>> wrote:
>>
>>> Do you really want to select the "nearest" selectable slice (which means
>>> "with the nearest center", which is irrelevant in this case and isn't the
>>> nearest slice!) when there are multiple overlapping? Wouldn't it be better
>>> to select the top-most one?
>>>
>>>
>>> On 6 September 2017 at 10:15, <subversion at hamsterrepublic.com> wrote:
>>>
>>>> james
>>>> 2017-09-05 15:15:08 -0700 (Tue, 05 Sep 2017)
>>>> 158
>>>> Add find_plank_at_screen_pos() which, in contrast to
>>>> find_plank_nearest_screen_pos()
>>>> requires the slice to be visible and to actually collide with the
>>>> targpos
>>>> ---
>>>> U   wip/plankmenu.bas
>>>> U   wip/plankmenu.bi
>>>> _______________________________________________
>>>> Ohrrpgce mailing list
>>>> ohrrpgce at lists.motherhamster.org
>>>> http://lists.motherhamster.org/listinfo.cgi/ohrrpgce-motherhamster.org
>>>>
>>>
>>>
>>> _______________________________________________
>>> Ohrrpgce mailing list
>>> ohrrpgce at lists.motherhamster.org
>>> http://lists.motherhamster.org/listinfo.cgi/ohrrpgce-motherhamster.org
>>>
>>>
>>
>> _______________________________________________
>> Ohrrpgce mailing list
>> ohrrpgce at lists.motherhamster.org
>> http://lists.motherhamster.org/listinfo.cgi/ohrrpgce-motherhamster.org
>>
>>
>
> _______________________________________________
> Ohrrpgce mailing list
> ohrrpgce at lists.motherhamster.org
> http://lists.motherhamster.org/listinfo.cgi/ohrrpgce-motherhamster.org
>
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.motherhamster.org/pipermail/ohrrpgce-motherhamster.org/attachments/20170906/6672b15c/attachment.html>