Block mover: disabled buttons should be disabled

[afercia]

Currently, when a block is the first or last one, the "block mover" shows just one button to move the block down or up.

However, the "disabled" button is still there: it gets revealed on hover and it's still focusable and actionable. It just takes a CSS class that applies a close-to-white color, so it can't be distinguished from the white background:

It should probably use display: none or visibility: hidden if keeping the space that it takes is necessary for layout purposes. I'd consider to make it really disabled too.

[nylen]

It should probably use display: none or visibility: hidden if keeping the space that it takes is necessary for layout purposes. I'd consider to make it really disabled too.

I think it's nice to leave the buttons visible to indicate that they would be active if not for some condition being true.

[afercia]

Yep that also could work 🙂 visually disabled and with a disabled attribute.