search

Keleo / CustomCSSBundle

Kimai bundle to add custom CSS rules to your installation.
https://www.kimai.org/store/keleo-css-custom-bundle.html
MIT License
15 stars 10 forks source link

Hide unneccessary scrollbar in timesheet responsive.css #6

Closed ssendev closed 5 years ago

ssendev commented 5 years ago

When the screen is wide enough a scrollbar is always visible. This fixes it.

kevinpapst commented 5 years ago

Hi @ssendev !

Thanks for your PR 👍

I am not able to reproduce the constant scrollbar, its not visible for me, unless I swipe to the right. Probably thats due to me using a Mac without mouse... but I saw these 15px one the right side as well. With your changes I can't swipe to the side anymore (scrollbar gone!), but it looks funny, as there is a white area at the right side next to the table. What do you think about:

#datatable_timesheet_admin .dataTables_wrapper > .row,
#datatable_timesheet .dataTables_wrapper > .row,
#datatable_timesheet_admin .dataTables_wrapper > .row table,
#datatable_timesheet .dataTables_wrapper > .row table {
   max-width: calc(100% + 15px);
}

For me thats even better, but I can currently only test on Desktop Safari and Chrome...

ssendev commented 5 years ago

Now that you mention it i had 15px on the right as well but your changes didn't make a difference. I pushed some changes that make it work on my machine. But I think they should work on mac too.