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

Datepicker for advanced movie options blocks out dates that it shouldn't #498

Closed
dgarciabriseno opened this issue Oct 26, 2023 · 8 comments

Comments

@dgarciabriseno
Copy link
Contributor

image

This is using flatpickr, I think it's set to black out dates beyond the end date, but that makes it feel clunky.

Just need to remove the maxDate option on the flatpickr instance on #movie-start-date in MovieManagerUI.js.

@aayushtmG
Copy link

Hey, l will like to be assigned on this issue

@dgarciabriseno
Copy link
Contributor Author

@aayushtmG all yours.
I'll send some info in a minute about running helioviewer on your pc. Still working on the docs.

@dgarciabriseno
Copy link
Contributor Author

@aayushtmG Try following the instructions in the readme here for running hv locally for testing.

@raptor-07
Copy link
Contributor

raptor-07 commented Dec 5, 2023

Is this resolved? can I PR?

@dgarciabriseno
Copy link
Contributor Author

dgarciabriseno commented Dec 5, 2023

Not resolved yet, you can PR

@raptor-07
Copy link
Contributor

raptor-07 commented Dec 6, 2023

image

Not able to reproduce this!

EDIT:

image

Reproduced it!

raptor-07 added a commit to raptor-07/helioviewer.org that referenced this issue Dec 6, 2023
Datepicker for advanced movie options blocks out dates that it shouldn't.

'maxDate' option defaults to null.
reference: https://flatpickr.js.org/options/

The option has been removed from flatpickr instance. This allows user to select future dates.
@raptor-07
Copy link
Contributor

#512

Review PR please!

raptor-07 added a commit to raptor-07/helioviewer.org that referenced this issue Dec 6, 2023
Helioviewer-Project#512 PR review suggested further changes.

doOpen handler is entirely removed
dgarciabriseno pushed a commit that referenced this issue Dec 14, 2023
* Issue #498 - Update MovieManagerUI.js

Datepicker for advanced movie options blocks out dates that it shouldn't.

'maxDate' option defaults to null.
reference: https://flatpickr.js.org/options/

The option has been removed from flatpickr instance. This allows user to select future dates.

* #498 - Update MovieManagerUI.js

#512 PR review suggested further changes.

doOpen handler is entirely removed
@dgarciabriseno
Copy link
Contributor Author

This was completed with PR #512

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

3 participants