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

T233 DateTime Picker - Added Datetimepicker in seedset form. closes #223 #226

Merged
merged 1 commit into from
May 6, 2016

Conversation

rajatvij
Copy link
Contributor

@lwrubel I have added Datetime picker on Seedset form screens. We can configure it a bit more by adding attr in forms.py
Eg.,
'start_date': DateTimeWidget(usel10n=True, bootstrap_version=3),

can be changed to
'start_date': DateTimeWidget(attrs={'id':"yourdatetimeid"}, usel10n=True, bootstrap_version=3),

id values:
'0' for the hour view
'1' for the day view
'2' for month view (the default)
'3' for the 12-month overview
'4' for the 10-year overview. Useful for date-of-birth datetimepickers.

Please refer https://github.com/asaglimbeni/django-datetime-widget for more.

Right now we are using default month view

Let me know if you require any change in this.

@rajatvij rajatvij added this to the 0.6 milestone Apr 29, 2016
@lwrubel
Copy link
Collaborator

lwrubel commented May 4, 2016

I think month default view is good to start.

To preserve ability to manually edit dates/times in the field, as described here, request you add to the widget:
attrs={'data-readonly': 'false'}

And beyond that, looks ready to squash and merge.

@rajatvij rajatvij force-pushed the t223-datetime_widget branch from 20e3c24 to 1b5af56 Compare May 6, 2016 17:58
@rajatvij
Copy link
Contributor Author

rajatvij commented May 6, 2016

@lwrubel Corrected date field as per your comment above. Please check and let me know if you find any other issues with it.

@lwrubel lwrubel merged commit 60c099e into master May 6, 2016
@lwrubel lwrubel deleted the t223-datetime_widget branch May 6, 2016 18:45
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