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

[Experimental] AppHeader.vue: use a range date picker #88

Closed
wants to merge 2 commits into from

Conversation

Tofee
Copy link
Contributor

@Tofee Tofee commented Sep 12, 2022

This is a quick and dirty implementation of a range picker, to illustrate my comment #43 (comment) about date picker.

For my own use it seems to work, but it's probably not what you'd want to merge right now...

Signed-off-by: Christophe Chapuis <[email protected]>
Copy link
Member

@linusg linusg left a comment

Choose a reason for hiding this comment

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

If you want to have this merged at the very least remove the vue-ctk-date-time-picker dependency :)

@Tofee Tofee requested a review from linusg May 24, 2023 17:14
@Tofee
Copy link
Contributor Author

Tofee commented Aug 18, 2023

If you want to have this merged at the very least remove the vue-ctk-date-time-picker dependency :)

The change has been done, do I need to do something else to mark the requested change as "handled" ?

@Tofee
Copy link
Contributor Author

Tofee commented Jun 9, 2024

Superseeded by #116

@Tofee Tofee closed this Jun 9, 2024
@jpmens
Copy link
Member

jpmens commented Jun 9, 2024

@Tofee thank you for your contribution, even if it has been superseded; much appreciated!

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.

3 participants