Selection history navigation with Alt+[ and Alt+] keys should filter out empty selections

[0SlowPoke0]

filters empty selections made by select tool before adding in the selection history

[0HyperCube]

Thanks for this fix; it appears to work as expected (although perhaps @Keavon will want to check as well).

[Keavon]

@0SlowPoke0 @0HyperCube can you both please confirm that my simplification commit c362e26 is indeed equivalent? I don't understand why it was so complicated before, and I'm pretty sure the changes I made are correct, but I'm asking to make sure I didn't miss any subtleties that explained why the previous code was so complicated.

Edit: Oh, my bad, it was to satisfy the borrow checker. I didn't actually confirm my simplifications compiled, whoops.

[Keavon]

!build

[github-actions]

📦 Build Complete for dd88eba
https://de6a16bf.graphite.pages.dev

[Keavon]

This PR breaks the editor (unable to draw more shapes, unable to select layers, etc.) after you've drawn the first shape. This was the case since 5427956, the most recent commit by @0SlowPoke0 (rather than commits from Hypercube and me).

[0SlowPoke0]

@Keavon , can you please make a build for testing, thanks !

[Keavon]

!build

[github-actions]

📦 Build Complete for 9dafb91
https://7eb6e77d.graphite.pages.dev

[0SlowPoke0]

@Keavon I think this solves the issue can you please review the build, thanks.

[Keavon]

Superseded by #2138