새로운 내비게이션 레이아웃은 마우스를 누를 때 cursor-grab을 사용함

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.

5개의 좋아요

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.

5개의 좋아요

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.

4개의 좋아요

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

6개의 좋아요

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

6개의 좋아요

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

https://github.com/discourse/discourse/pull/20447

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

5개의 좋아요

This topic was automatically closed after 21 hours. New replies are no longer allowed.