-
Notifications
You must be signed in to change notification settings - Fork 277
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Punch button should overflow to the right #441
Comments
Do you think we should make the Punch Time button a little less prominent in terms of Width / and centre it ? While UX wise it is good to have large interact-able targets, this one may result in a lot of accidents. |
That's a very valid point, but I have a feeling that having it smaller would be a little strange. I think usually people don't grab the scrollbar right on the bottom or use the drag feature of the scrollbar as much, preferring usually the mouse/trackpad movement (totally based on my experience, not sure how much this is true). The action is Undoable, but this means that the user would need to go over the last punched entry and erase it manually. |
Makes sense. Please feel free to assign this to me 👍 |
Thanks! |
I investigated this. |
This one is tricky. The proper solution would be to change the layout around, but that's not trivial at this point. Can we settle for something like this where the button is not 100% wide, but has some left / right margin ? |
I really thought that was one of those one-line changes hahahahha |
I've also made the scrollbar a little thinner. i.e |
Punch button should overflow to the right, filling up the gap between the button and the scroll bar. Still, the scrollbar needs to be on top of the button when in the bottom of the page.
The text was updated successfully, but these errors were encountered: