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

This is a PR with all the features and bugfixes added so far by me and others #340

Closed
wants to merge 235 commits into from

Conversation

tonix-tuft
Copy link

Hello guys,

These are all the PRs we have already merged into my two forks (https://github.com/tonix-tuft/tempusdominus and https://github.com/tonix-tuft/tempusdominus-bootstrap)
which I am sending to the main repo of Eonasdan directly on master.

I wanna thank @HaroldPutman, @pablo-lp, @zondatw for their contribution.

@Eonasdan
Copy link
Contributor

hi @tonix-tuft thanks for doing this. Let's get this bad boy in. This repo will be the base for my revitalization efforts

@tonix-tuft
Copy link
Author

tonix-tuft commented Nov 13, 2020

@Eonasdan Great! So, we have changed several things (updated packages, added patches, bugfixes and features), here is a brief summary of what we did:

  • Updated composer.json
  • Changed composer.json to point to the forked repo; Changed CONTRIBUTING.md for contribution notes
  • Updated package.json's description
  • Added setMultiDate function
  • Added Feathers icons support (fallback to Font Awesome icons in case the Feathers icons are not used)
  • Patch for locale when using useCurrent option set to false (requires tempusdominus update)
  • Bugfix for parseInputDate option
  • Bugfix for view update and onUpdate's change property of event payload
  • Bugfix for picker when useCurrent option is set to false
  • Added setting of isDateUpdateThroughDateOptionFromClientCode flag when modifying the value of date option from the client code
  • Fixed styles for picker with calendarWeeks option set to true
  • Bugfix for allowInputToggle option when used with input[data-toggle="datetimepicker"]
  • Fixed some styles in time picker
  • Bugfix for incrementing/decrementing hours/minutes/seconds in multidate mode
  • Added CSS minification
  • Added watch build command for development
  • Downgraded moment version because of issue #5472 Can't resolve './locale' moment/moment#5472
  • Fixed blinking cursor (UX) on click of a day cell on the calendar in a cross-browser manner
  • Changed build scripts
  • Updated build commands
  • Added multidate mode fixes
  • Added bugfixes for .isAfter() and .clone() when picker is in multidate mode
  • Added fork section in README
  • Fix for issue Cannot read property 'isSame' of undefined when removing all the dates when in multidate mode #325 - respecting ESLint one-var rule
  • Added .prettierignore to ignore the whole codebase (for now)
  • Updated NPM packages
  • Added .prettierignore
  • Fix for issue Cannot read property 'isSame' of undefined when removing all the dates when in multidate mode #325

All these things now are on my forked tonix-tuft/tempusdominus-bootstrap repo. How do you want to proceed? Let me know. Thank you!

@Eonasdan
Copy link
Contributor

@tonix-tuft I'm working on resolving the merge conflicts. There's a couple of other things I'm trying to get resolved right now, but I do plan to get this in one way or another. I think I'll put in a PR for the conflicts resolved and then give you a chance to look over it and see if you think anything is missing.

I also want to remove or at least make less embedded the feather icon stuff once it goes in. Dayjs has a plugin model that I think might be worth copying so if FA isn't your thing and uses a different implementation that could be a plugin. I think that might let us do RTL and calendars outside of Georgian.

@tonix-tuft
Copy link
Author

tonix-tuft commented Nov 13, 2020 via email

@Eonasdan Eonasdan mentioned this pull request Nov 14, 2020
@Eonasdan
Copy link
Contributor

@tonix-tuft pr #360 is in. I had to change node-sass for dart sass as node-sass didn't seem to work on windows 10. I took most of your changes, except I put back some of the branding. I also updated most of the npm packages. I see the momentjs might still be an issue even with the latest. Can you check your specific issue?

Once 360 is merged, I'm going to start in on bringing core and bs 4 into the old repo. Did you make changes to the core project?

Also, does this PR close a bunch of the existing PRs? I don't want to take those changes if it does.

Thanks for your help :)

@tonix-tuft
Copy link
Author

tonix-tuft commented Nov 14, 2020

@Eonasdan

I see the momentjs might still be an issue even with the latest. Can you check your specific issue?

I downgraded moment.js in my fork to stick to version ~2.24.0 (2.24.*) because at the time I couldn't compile the project when using moment >=2.25.0. You said you tried to upgrade it, which issue did you get?

Once 360 is merged, I'm going to start in on bringing core and bs 4 into the old repo. Did you make changes to the core project?

Yes I have forked and modified your core repo too (here is the fork: https://github.com/tonix-tuft/tempusdominus).
My PR for core is here: tempusdominus/core#26

Also, does this PR close a bunch of the existing PRs? I don't want to take those changes if it does.

This PR includes some of the PRs you received from the other guys too, yes. Here are the ones they resent to my fork and that have already been merged in this PR:

On the core repo, I have received and merged the following PRs:

@Eonasdan
Copy link
Contributor

Eonasdan commented Sep 1, 2021

@tonix-tuft can you please send me a Twitter DM or email at me at eonasdan.com

@tonix-tuft
Copy link
Author

Hi @Eonasdan! I don't use Twitter, I wrote you an email.

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.

7 participants