release/next (#6)

release/next

package.json

@@ -56,7 +56,8 @@
     "uuid": "^3.3.2"
   },
   "peerDependencies": {
-    "react": "^16.0.0"
+    "react": "^16.8.6",
+    "react-dom": "^16.8.6"
   },
   "devDependencies": {
     "@babel/preset-react": "^7.0.0",
@@ -83,8 +84,8 @@
     "jest-cli": "^23.6.0",
     "nwb": "0.23.0",
     "prettier": "^1.15.3",
-    "react": "^16.6.3",
-    "react-dom": "^16.6.3",
+    "react": "^16.8.6",
+    "react-dom": "^16.8.6",
     "semantic-release": "^15.12.3",
     "travis-deploy-once": "^5.0.9"
   },

src/components/Calendar/Calendar.js

@@ -62,6 +62,10 @@ const DAY_MAP = {
 const DEFAULT_SLOT_HEIGHT = 20;
 const ONE_WEEK = 7;
 
+function hasTimeChanged(start1, end1, start2, end2) {
+  return `${start1}` !== `${start2}` || `${end1}` !== `${end2}`;
+}
+
 class Calendar extends Component {
   static MONDAY = MONDAY;
   static TUESDAY = TUESDAY;
@@ -293,7 +297,10 @@ class Calendar extends Component {
     } else if (isFunction(onEventClick) && !isDirty) {
       // It is the same start & end, so we probably wanted a click...
       onEventClick(event);
-    } else if (canUpdateEvent({ event, start, end })) {
+    } else if (
+      hasTimeChanged(event.start, event.end, start, end) &&
+      canUpdateEvent({ event, start, end })
+    ) {
       // We do have an existing event
       // Let's update it
       onEventUpdate({
@@ -371,7 +378,6 @@ class Calendar extends Component {
       renderGutterItem,
       renderEvent,
       renderPlaceholder,
-      onEventClick,
       daysToRender,
       minTime,
       locale,
@@ -567,7 +573,6 @@ class Calendar extends Component {
                               <Event
                                 key={event.id || eventIndex} // assuming an id exist on the event, if not, use the index
                                 event={event}
-                                onEventClick={onEventClick}
                                 interactionInfo={
                                   event === pseudoEvent &&
                                   event.isPlaceholderEvent

src/components/Calendar/Event.js

@@ -4,15 +4,14 @@ import { isFunction } from '../../utils/isFunction';
 
 import { DragHandle } from './DragHandle';
 
-function EventWrapper({ children, positionTop, height, onEventClick }) {
+function EventWrapper({ children, positionTop, height }) {
   return (
     <div
       className="SkedifyCalendar__EventWrapper"
       style={{
         transform: `translateY(${positionTop}px)`,
         height,
       }}
-      onClick={onEventClick}
     >
       {children}
     </div>
@@ -25,29 +24,20 @@ function EventComponent({
   render,
   positionTop,
   height,
-  onEventClick,
   interactionInfo,
 }) {
   const shouldBeInteractable = isFunction(onActivate);
 
   if (!shouldBeInteractable) {
     return (
-      <EventWrapper
-        onEventClick={() => onEventClick(event)}
-        positionTop={positionTop}
-        height={height}
-      >
+      <EventWrapper positionTop={positionTop} height={height}>
         {render(event, interactionInfo)}
       </EventWrapper>
     );
   }
 
   return (
-    <EventWrapper
-      onEventClick={() => onEventClick(event)}
-      positionTop={positionTop}
-      height={height}
-    >
+    <EventWrapper positionTop={positionTop} height={height}>
       <DragHandle
         placement={DragHandle.PLACEMENT_TOP}
         onActivate={onActivate(event, DragHandle.PLACEMENT_TOP)}