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

fix: 🐛live time indicator position on timeline is not correct when st… #349

Conversation

apurva010
Copy link
Collaborator

@apurva010 apurva010 commented Apr 30, 2024

…artHour set in day and week view

  • Removed/minus startHour minutes from totalMinutes of day to set dy offset of current timeline painter which set live time line indicator properly on timeline

Description

Checklist

  • The title of my PR starts with a Conventional Commit prefix (fix:, feat:, docs: etc).
  • I have followed the Contributor Guide when preparing my PR.
  • I have updated/added tests for ALL new/updated/fixed functionality.
  • I have updated/added relevant documentation in docs and added dartdoc comments with ///.
  • I have updated/added relevant examples in examples or docs.

Breaking Change?

  • Yes, this PR is a breaking change.
  • No, this PR is not a breaking change.

Related Issues

Closes #346

Before : live indicator in day view when startHour is set

before_day_view

After : live indicator in day view when startHour is set

after_day_view_live_indicator

Before : live indicator in week view when startHour is set

before_week_view

After : live indicator in week view when startHour is set

after_week_view_live_indicator

@apurva010 apurva010 requested a review from PRBaraiya April 30, 2024 11:58
@apurva010 apurva010 force-pushed the fix/issue_346_live_time_indicator_display_position branch from 4eadd8d to 2b8b656 Compare April 30, 2024 11:59
@apurva010 apurva010 requested review from vatsaltanna-simformsolutions and removed request for PRBaraiya April 30, 2024 13:18
…artHour set in day and week view #346

- Removed/minus startHour minutes from totalMinutes of day to set dy offset of current timeline painter which set live time line indicator properly on timeline
@apurva010 apurva010 force-pushed the fix/issue_346_live_time_indicator_display_position branch from 2b8b656 to 8add974 Compare April 30, 2024 13:20
@vatsaltanna-simformsolutions vatsaltanna-simformsolutions merged commit fadd7f5 into master May 1, 2024
1 check passed
@vatsaltanna-simformsolutions vatsaltanna-simformsolutions deleted the fix/issue_346_live_time_indicator_display_position branch May 1, 2024 05:22
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Live Time Indicator is wrong when startHour is set in WeekView
2 participants