New nav layout uses cursor-grab when mouse is pushed

[darkpixlz]

Hello,

I like the new navigation layout, but I found an issue with it.

To reproduce, just hold down:

This is mainly misleading, I thought I would be able to drag it around, but instead, it’s just showing the cursor:drag cursor while serving no purpose. I think it’s best if it stays at pointer or default.

[keegan (Keegan George)]

On smaller device widths the menu can actually be grabbed and dragged across to reach hidden items:

Though, perhaps we could make the cursor default only when the nav can’t scroll.

[darkpixlz]

Yeah, I noticed that just trying it.

I noticed a bit of an inconsistency though. The labels are hidden at the top, but not the bottom.

[sam (Sam Saffron)]

@keegan

Though, perhaps we could make the cursor default only when the nav can’t scroll.

I am not sure this is doable in CSS @awesomerobot ?

[awesomerobot (Kris)]

We check if it’s scrollable on resize so we can add the left/right controls… so I think this should be doable!

[awesomerobot (Kris)]

This will make it so it’s only grabby when it’s scrollable

UX: only allow scroll grab if nav is scrollable

@darkpixlz

I noticed a bit of an inconsistency though. The labels are hidden at the top, but not the bottom.

Yeah that’s intentional, the bottom list has more items and I don’t think they’re as easy to figure out without text.