Selection from Keyboard Bug

Here's an odd UI bug:

Select an item in a vault.
Hold Shift and press the Up (or Down) keyboard arrow.
The next item is added to the selection, as expected.

While still holding Shift, press the same Up (or Down) arrow again.
The next item is not added to the selection.

The 3rd and subsequent times work as expected.


1Password Version: 80600077
Extension Version: Not Provided
OS Version: 12.3

Comments

  • Huh. That's very interesting, @volts. Thanks for the report! I've filed an issue and will see if I can look into it myself. It's exactly the kind of annoying little logic bug I enjoy sniffing out. :D

    ref: dev/core/core#13532

  • volts
    volts
    Community Member
    edited March 2022

    :-)

    Thanks! It felt weird, and because pressing Up an additional time worked, I didn't really understand at first.

    Try this too -

    Select an item
    Hold Shift and press Up a handful of times, to select a few more items
    Then (still holding Shift) press Down to select fewer items
    Keep going down, past the original item ...

    It will do the same "Pause at two" thing as you pass the first item selected. It's not "The 2nd time you press Up", it's "When there are 2 selected".

  • Yep! I noticed that as well. Here's another one to entertain you:

    1. Select an item.
    2. Press Shift+Down.
    3. Press Shift+Up.

    It should go back to just one item selected, but it actually jumps and selects the original item and the one above it. Same thing happens in reverse.

    Anyway, I just spent the last 90 minutes testing and locating the problem, then writing it up for the team. It turned out to be a weird race condition where we lose track of the most recently selected item when going from 1 to 2 selected items. This just started with a change that was made a couple months ago to improve something else. I have a few ideas for how to fix it, so we just need to decide on the best way and get it done.

    Thanks again for the report!

  • volts
    volts
    Community Member

    I'm easily entertained!
    And thank you for the investigation and sharing the details here. Appreciated.

  • Hey @volts. This has been fixed in the latest nightly (80700019). Thanks again for the report!

  • volts
    volts
    Community Member

    It's much better, thanks!

    There's another weird thing, although it's less likely to happen in normal use.

    • Scroll to the middle of a vault.
    • Select an item.
    • Press & Keep Holding Shift
    • Tap Up, Tap Up, Tap Up
      (Four items are now selected)

    • Press & Hold Down

    Observation -

    • The selection will first reduce back to the original item, as expected.
    • Then, it will extend to additional items, as expected.
    • But! At some point, the selection will reset back to the original item.

    The reset may happen when 4 items are selected; or when the selection reaches the bottom of the screen; or when Down is released.

    I can make a video if that's helpful.

  • volts
    volts
    Community Member

    (I'm learning from this that I continuously make selections and idly "drive" the cursor around the screen.)

  • Interesting! I've reproduced this as well. I'll see if I can take a closer look on Monday. Thanks!

  • volts
    volts
    Community Member

    I think this (partially?) reverted in 80700012. The "resistance at 2 items" issue is back.

  • volts
    volts
    Community Member

    In 80700027 it's back to how it was working in 80700019 again, thanks!

    The other issue I mentioned - if you start going one direction, then reverse and go further in the other direction, the selection will drop - is still present.

  • I think this (partially?) reverted in 80700012. The "resistance at 2 items" issue is back.

    80700012 was before 80700019, where the issue was fixed. Maybe you switched over to the beta channel?

    The other issue I mentioned - if you start going one direction, then reverse and go further in the other direction, the selection will drop - is still present.

    Yep, I'm just getting around to looking at that one today.

  • volts
    volts
    Community Member

    Hahah, maybe I did. Maybe I was looking on a different machine? Math is hard, time is nonlinear, nothing is real.

    I appreciate it. Maybe it's not "core" functionality, but making this sort of thing smooth really improves how the app feels.

  • rob
    rob
    edited March 2022

    I've got a fix ready for this. Will post back when it's been released in a nightly.

    ref: dev/core/core#13924

  • Fixed in 80700030. Thanks again!

  • volts
    volts
    Community Member

    Looks great. TYVM!

  • 😎👍

This discussion has been closed.