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

Added autocenter to timeline widget #8

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

Matt0550
Copy link

@Matt0550 Matt0550 commented Aug 26, 2023

Added autocenter day in the timeline widget. Consider testing on multiple devices
This is close #7

@Matt0550
Copy link
Author

I noticed that in the init state it is not set to the center. Trying to put it anyway doesn't put it central

@FadyFayezYounan
Copy link
Owner

Great job! I will perform tests on the code, and I will approve it.
one small enhancement is to replace this MediaQuery.of(context).size.width with MediaQuery.sizeOf(context).width

@Matt0550
Copy link
Author

great, reassurances also to set the day centered when initializing the widget 👍👋

@saadzarif
Copy link

@Matt0550 Just saw your comment in Issues and now found you here. Thanks man ❤️❤️

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.

Auto center the selected day in the listview
3 participants