-
Notifications
You must be signed in to change notification settings - Fork 4
Description
Event Calendar is a phenomenal component! Love it!
I noticed there is functionality to delete an event, but it doesn't seem to be implemented anywhere in the component. I wasn't sure if this was deliberate and you're wanting the developer to add that implementation, most likely with a "delete" button on the event-dialog.component.ts, or if that is a future enhancement you just haven't gotten to yet.
I have no problem implementing it myself, I just want to make sure I'm following best practices for your design.
Also, just a suggestion, you might want to put the event-dialog in a scrollable container. If you open up the browser dev tools, you'll see what I'm talking about. The component spills out past the dialog container.
Additionally, moving an event by dragging/dropping on the Month display doesn't seem to be interpreting the dates properly (probably timezone related), so when you drop the event on a date, it moves to the previous day. You can see that behavior in your demo.
One more...not sure if this is a bug or as intended. You cannot click to edit or drag/drop to move on "All Day" event when on the Week and Day views.
And now I'm being picky LOL...the title of the dialog always says "Create new event". That should probably be dynamic and change to "Edit event" or something when editing an event.
Thanks!