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

[WIP] ng15 support #97

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

[WIP] ng15 support #97

wants to merge 2 commits into from

Conversation

IliaIdakiev
Copy link
Owner

@IliaIdakiev IliaIdakiev commented Jan 21, 2023

Status:

  • Code building
  • UI looking good
  • ValueAccessor working properly (with the new Angular Material and Angular there are certain changes that break validation and forms functionality)

@@ -378,7 +379,7 @@ export class MatTimepickerDirective
}
}

@HostListener('keydown', ['$event']) keydownHandler(event: any) {
@HostListener('keydown', ['$event']) keydownHandler({ event }: { event: any; }): true | undefined {
Copy link

Choose a reason for hiding this comment

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

Suggested change
@HostListener('keydown', ['$event']) keydownHandler({ event }: { event: any; }): true | undefined {
@HostListener('keydown', ['$event']) keydownHandler(event : KeyboardEvent): true | undefined {

Angular does not wrap then $event in an object.

@angelaki
Copy link

Oh nice, ng15 support ahead! 🥳 Any estimations, yet?

@Timkoeln
Copy link

Timkoeln commented Mar 8, 2023

Nice ng15🎉 is there an estimate of when it will be merged?

@angelaki
Copy link

@IliaIdakiev no pressure, but any estimations on this? My Angular15 migration is just waiting for this component. Since you already announced this in January, I hoped it would arrive pretty soon. Sure this is only community work, so really no pressure! But I start getting afraid this won't happen anytime soon / at all.

@angelaki
Copy link

angelaki commented May 8, 2023

@IliaIdakiev sorry for bumping, but any news on this? Its even time for ng16 now. Really don't mind if you just don't find the time but since I'd really need it, some news would be cool. If you're done with this lib, I could continue it.

@angelaki
Copy link

angelaki commented Jun 14, 2023

That push gave me a litte hope at least 😉 Any noticable plans on this project?

@IliaIdakiev
Copy link
Owner Author

Hey guys, the project is open source for a reason, there are a few small things not working on this PR so if you are in a hurry feel free to contribute. You can always run the PR see what's not working and resolve the issues. Cheers.

@angelaki
Copy link

Yeah, sure. The fact that you opened this PR just made me hope that you'll get this lib compatible with ng15 (or 16 now) yourself. This sure is way easier for an lib's author. If you could tell if you're at least planning to, it would be helpful in deciding wether to fix it myself, wait for an update or switch the lib. Open source or not, it would be nice to still moderate it (or deprecate if no longer needed).

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.

None yet

4 participants