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

bug fixed #101

Open
wants to merge 5 commits into
base: master
Choose a base branch
from

Conversation

DHRUVA-N-L-CHOUDHARY
Copy link

WhatsApp Image 2022-10-06 at 2 04 26 PM

The following required space between month and date picker is arranged

Copy link
Owner

@MariaMelnik MariaMelnik left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Hi!
Thank for you impact and PR.
I made few comments. Please, check it.

.fvm/flutter_sdk Outdated
@@ -0,0 +1 @@
C:/Users/dhruv/fvm/versions/2.8.0
Copy link
Owner

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

this file should not be added to repo. It is your local settings.

@@ -0,0 +1,4 @@
{
Copy link
Owner

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

this file should not be added to repo. It is your local settings.

@@ -0,0 +1,13 @@
#!/bin/sh
Copy link
Owner

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

this file should not be added to repo. It is your local settings.

@@ -42,7 +42,7 @@ packages:
name: collection
url: "https://pub.dartlang.org"
source: hosted
version: "1.16.0"
version: "1.15.0"
Copy link
Owner

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Why it is necessary to decrease package version?
Seems like you committed your local saved versions.

@@ -179,6 +179,7 @@ class _DayBasedChangeablePickerState<T>
padding: widget.datePickerLayoutSettings.contentPadding,
child: _buildMonthNavigationRow()),
),
SizedBox( height: 40, width: double.infinity,),
Copy link
Owner

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Why height is 40?
I assume it should not be hardcoded.
Also if we made new version with such changes all previous users will get this padding. Which is not good.
Seems like solution should not involve breaking changes.

pubspec.lock Outdated
@@ -7,7 +7,7 @@ packages:
name: async
url: "https://pub.dartlang.org"
source: hosted
version: "2.8.1"
version: "2.8.2"
Copy link
Owner

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Why it is necessary to change package versions here?
Seems like you committed your local saved versions.

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.

2 participants