Skip to content
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

Closed
thamara opened this issue Oct 12, 2020 · 8 comments · Fixed by #455
Closed

Punch button should overflow to the right #441

thamara opened this issue Oct 12, 2020 · 8 comments · Fixed by #455
Assignees
Labels
enhancement New feature or request good first issue Good for newcomers hacktoberfest Hacktoberfest

Comments

@thamara
Copy link
Collaborator

thamara commented Oct 12, 2020

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.

image

@thamara thamara added enhancement New feature or request good first issue Good for newcomers hacktoberfest Hacktoberfest labels Oct 12, 2020
@ccsCoder
Copy link
Contributor

Do you think we should make the Punch Time button a little less prominent in terms of Width / and centre it ?
The reason I'm suggesting this is that it may lead to a lot of accidental taps / touches. Especially if it has a scrollbar on top of it .

While UX wise it is good to have large interact-able targets, this one may result in a lot of accidents.
Btw, is this action "Undoable" ?

@thamara
Copy link
Collaborator Author

thamara commented Oct 12, 2020

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.

@ccsCoder
Copy link
Contributor

Makes sense. Please feel free to assign this to me 👍

@thamara
Copy link
Collaborator Author

thamara commented Oct 12, 2020

Thanks!

@ccsCoder
Copy link
Contributor

I investigated this.
The reason for the gap is that the Scrollbar is Actually appearing on the body and it automatically reserves 10px for the scrollbar.
Ideally, the scrollbar should appear on the container holding the table.

@ccsCoder
Copy link
Contributor

This one is tricky.
Increasing the width is Not a solution because the browser Always leaves room for the scrollbar.
Trying to style scrollbars differently isn't flexible enough to target the right portion of the button.

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 ?

Screen Shot 2020-10-12 at 5 23 21 PM

@thamara
Copy link
Collaborator Author

thamara commented Oct 12, 2020

I really thought that was one of those one-line changes hahahahha
Thank you for investigating this.
I think this slightly narrow button looks good. It doesn't affect the overall feel but solves the little issue of the scrollbar meeting the button edge.
Unless this becomes extremely weird in one of the other themes (which I doubt), it's a go from me.

@ccsCoder
Copy link
Contributor

I've also made the scrollbar a little thinner. i.e width: 5px, height: 5px to leave some room around the right side of the button.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request good first issue Good for newcomers hacktoberfest Hacktoberfest
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants