Tools need better feedback to prevent double-clicking

[jenschr]

Windows users tend to double-click, so they checkin and out within less than a second. Adding red color to the button of checked out tools will make it very visible if a tool is checked out.

[jonnor]

If people double clicking is the problem, why would a visual warning fix this? They won't have time to see it?

Or are they supposed to notice the flash after doing it wrong one time, then not do it wrong the next time?

[jenschr]

Well... As they can see other tools be checked out as red color, they'll hopefully notice that theirs didn't go red? I'm very open to other alternatives. I had a go at disabling the button briefly, but the problem is that Angular is using two physical buttons (toggling between these), so disabling the clicked button won't have any effect.

[trygvis]

Hm, I'm pretty sure Angular only uses a single button and that disabling the button until the request comes back should work.

[jenschr]

Angular uses a single button, but if you look through this code - we are using multiple buttons to implement conditional logic.

[jenschr]

Anyway - I think PR #62 solves this without complex button interaction. Would be great if we could just try that one out. This issue/PR was created to solve one of the most common questions we get from users. Just highlighting the buttons for tools that are checked out could be enough, so great if you guys can just approve the PR after checking that I did nothing wrong.