Fix #128 : Bottom overflowed error at dashboard - Githubissues

hamaluik / timecop

A time tracking app that respects your privacy and the gets the job done without being fancy.

https://timecop.app/

Apache License 2.0

832 stars 155 forks source link

Fix #128 : Bottom overflowed error at dashboard #129

Closed vasusomani closed 12 months ago

vasusomani commented 1 year ago

Description

Fixes #128: Bottom overflowed error at dashboard when multiple timers are running.

Changes Made

I made the following changes to address this issue:

Added Scrollable ListView over RunningTimerRow
Added box constraint to ListView so that it doesnt take much space
Implemented a freeze header feature for the running timer so that users can see the total timing even while scrolling.

Visual Changes: ezgif com-video-to-gif

Testing

I tested this fix by following these steps:

Opened the dashboard with multiple timers running.
Scrolled through the running timers to ensure they stay within the view.
Verified that the header of the running timer remains visible during scrolling.
Checked for any new visual issues or errors.

Everything worked as expected without any errors.

Checklist

[x] Fixes #128.
[x] All tests pass.
[x] Code follows project coding standards.

12people commented 12 months ago

Thanks @vasusomani ! Sorry it took so long to merge. I'm going to make one extra edit to always show the scrollbar in the area, as there are cases when it's not clear there are other timers running, and I'll also add your name to the CONTRIBUTORS file.