KAAV-1473 Added proptypes and cleaned scss comments

src/components/ProjectTimeline/AddGroupModal.js

@@ -4,6 +4,7 @@ import { useDispatch } from 'react-redux';
 import { EDIT_PROJECT_TIMETABLE_FORM } from '../../constants'
 import { IconPlus,Button } from 'hds-react'
 import { useTranslation } from 'react-i18next'
+import PropTypes from 'prop-types'
 
 const AddGroupModal = ({toggleOpenAddDialog,addDialogStyle,addDialogData,closeAddDialog, allowedToEdit, isAdmin}) => {
   const {t} = useTranslation()
@@ -31,4 +32,13 @@ const AddGroupModal = ({toggleOpenAddDialog,addDialogStyle,addDialogData,closeAd
   );
 }
 
+AddGroupModal.propTypes = {
+  toggleOpenAddDialog: PropTypes.func,
+  addDialogStyle: PropTypes.object,
+  addDialogData: PropTypes.object,
+  closeAddDialog: PropTypes.func,
+  allowedToEdit: PropTypes.bool,
+  isAdmin: PropTypes.bool
+};
+
 export default AddGroupModal;

src/components/ProjectTimeline/TimelineModal.js

@@ -5,6 +5,7 @@ import { EDIT_PROJECT_TIMETABLE_FORM } from '../../constants'
 import FormField from '../input/FormField'
 import { isArray } from 'lodash'
 import { showField } from '../../utils/projectVisibilityUtils'
+import PropTypes from 'prop-types'
 import './VisTimeline.css'
 
 const TimelineModal = ({ open,group,content,deadlinegroup,deadlines,openDialog,visValues,deadlineSections,formSubmitErrors,projectPhaseIndex,archived,allowedToEdit }) => {
@@ -150,5 +151,20 @@ const TimelineModal = ({ open,group,content,deadlinegroup,deadlines,openDialog,v
       </Modal>
     )
   }
+
+  TimelineModal.propTypes = {
+    open: PropTypes.bool,
+    group: PropTypes.object,
+    content: PropTypes.string,
+    deadlinegroup: PropTypes.object,
+    deadlines: PropTypes.array,
+    openDialog: PropTypes.func,
+    visValues: PropTypes.object,
+    deadlineSections: PropTypes.array,
+    formSubmitErrors: PropTypes.object,
+    projectPhaseIndex: PropTypes.number,
+    archived: PropTypes.bool,
+    allowedToEdit: PropTypes.bool
+  };
 
   export default TimelineModal

src/components/ProjectTimeline/VisTimeline.scss

@@ -8,13 +8,11 @@
         width: 300px;
     }
     .vis-drag-left{
-        //Disabled for now because we don't want to allow dragging the timeline at this point
-        //pointer-events: auto;
+        /*TODO: v1.2 Disabled for now because we don't want to allow dragging the timeline at this point set to auto*/
         pointer-events:none;
     }
     .vis-drag-right{
-        //Disabled for now because we don't want to allow dragging the timeline at this point
-        //pointer-events: auto;
+        /*TODO: v1.2 Disabled for now because we don't want to allow dragging the timeline at this point set to auto*/
         pointer-events:none;
     }
     .vis-time-axis.vis-foreground{

src/components/ProjectTimeline/VisTimelineGroup.js

@@ -8,6 +8,7 @@ import 'vis-timeline/dist/vis-timeline-graph2d.min.css'
 import TimelineModal from './TimelineModal'
 import VisTimelineMenu from './VisTimelineMenu'
 import AddGroupModal from './AddGroupModal';
+import PropTypes from 'prop-types';
 import './VisTimeline.css'
 Moment().locale('fi');
 
@@ -671,4 +672,16 @@ const VisTimelineGroup = forwardRef(({ groups, items, deadlines, visValues, dead
     )
 });
 VisTimelineGroup.displayName = 'VisTimelineGroup';
+VisTimelineGroup.propTypes = {
+  groups: PropTypes.array,
+  items: PropTypes.array,
+  deadlines: PropTypes.array,
+  visValues: PropTypes.object,
+  deadlineSections: PropTypes.array,
+  formSubmitErrors: PropTypes.object,
+  projectPhaseIndex: PropTypes.number,
+  archived: PropTypes.bool,
+  allowedToEdit: PropTypes.bool,
+  isAdmin: PropTypes.bool
+};
 export default VisTimelineGroup

src/components/ProjectTimeline/VisTimelineMenu.js

@@ -1,5 +1,6 @@
 import React, { useState } from 'react'
 import {Button,IconAngleLeft,IconAngleRight} from 'hds-react'
+import PropTypes from 'prop-types';
 
 function VisTimelineMenu({goToToday, moveLeft, moveRight,showYers,showMonths}) {
   const [selectedButton, setSelectedButton] = useState(null);
@@ -38,4 +39,12 @@ function VisTimelineMenu({goToToday, moveLeft, moveRight,showYers,showMonths}) {
   )
 }
 
+VisTimelineMenu.propTypes = {
+  goToToday: PropTypes.func,
+  moveLeft: PropTypes.func,
+  moveRight: PropTypes.func,
+  showYers: PropTypes.func,
+  showMonths: PropTypes.func
+};
+
 export default VisTimelineMenu

src/components/common/ConfirmModal.js

@@ -1,8 +1,9 @@
 import React from 'react';
+import PropTypes from 'prop-types';
 import { Modal } from 'semantic-ui-react';
 import { Button,IconInfoCircle } from 'hds-react';
 
-function ConfirmCancelModal({ headerText, contentText, button1Text, button2Text, onContinue, onCancel }) {
+function ConfirmModal({ headerText, contentText, button1Text, button2Text, onContinue, onCancel }) {
   return (
     <Modal open={true} className="timetable-confirm-modal">
       <Modal.Header><IconInfoCircle className='header-icon' size="s" aria-hidden="true"/>{headerText}</Modal.Header>
@@ -17,4 +18,13 @@ function ConfirmCancelModal({ headerText, contentText, button1Text, button2Text,
   );
 }
 
-export default ConfirmCancelModal;
+ConfirmModal.propTypes = {
+  headerText: PropTypes.string,
+  contentText: PropTypes.string,
+  button1Text: PropTypes.string,
+  button2Text: PropTypes.string,
+  onContinue: PropTypes.func,
+  onCancel: PropTypes.func
+};
+
+export default ConfirmModal;

src/components/input/CustomCheckbox.js

@@ -1,4 +1,5 @@
 import React, { useState, useEffect } from 'react'
+import PropTypes from 'prop-types';
 import { getFormValues } from 'redux-form'
 import { EDIT_PROJECT_TIMETABLE_FORM } from '../../constants'
 import { getFieldAutofillValue } from '../../utils/projectAutofillUtils'
@@ -108,4 +109,23 @@ const CustomCheckbox = ({
   }
 }
 
+CustomCheckbox.propTypes = {
+  input: PropTypes.shape({
+    name: PropTypes.string,
+    value: PropTypes.oneOfType([PropTypes.string, PropTypes.number, PropTypes.bool]),
+    onChange: PropTypes.func,
+  }),
+  meta: PropTypes.shape({
+    error: PropTypes.string,
+  }),
+  autofillRule: PropTypes.string,
+  label: PropTypes.string,
+  className: PropTypes.string,
+  disabled: PropTypes.bool,
+  updated: PropTypes.bool,
+  formName: PropTypes.string,
+  display: PropTypes.string,
+  isProjectTimetableEdit: PropTypes.bool
+};
+
 export default CustomCheckbox

src/components/input/DeadlineInfoText.js

@@ -88,6 +88,11 @@ const DeadlineInfoText = props => {
 DeadlineInfoText.propTypes = {
   fieldData:PropTypes.object,
   meta: PropTypes.object,
+  input: PropTypes.oneOfType([
+    PropTypes.string,
+    PropTypes.number,
+    PropTypes.bool,
+  ])
 }
 
 export default DeadlineInfoText

src/components/input/DeadlineInput.js

@@ -182,6 +182,15 @@ const DeadLineInput = ({
 
 DeadLineInput.propTypes = {
   input: PropTypes.object.isRequired,
+  error: PropTypes.string,
+  attributeData: PropTypes.object,
+  currentDeadline: PropTypes.object,
+  editable: PropTypes.bool,
+  type: PropTypes.string,
+  disabled: PropTypes.bool,
+  placeholder: PropTypes.string,
+  className: PropTypes.string,
+  autofillRule: PropTypes.string,
   timeTableDisabled: PropTypes.bool
 }
 

src/components/project/EditProjectTimetableModal/index.js

@@ -14,7 +14,7 @@ import { Button,IconInfoCircle } from 'hds-react'
 import { isEqual } from 'lodash'
 import VisTimelineGroup from '../../ProjectTimeline/VisTimelineGroup'
 import * as visdata from 'vis-data'
-import ConfirmCancelModal from '../../common/ConfirmModal';
+import ConfirmModal from '../../common/ConfirmModal';
 
 class EditProjectTimeTableModal extends Component {
   constructor(props) {
@@ -579,7 +579,7 @@ class EditProjectTimeTableModal extends Component {
               toggleTimelineModal={this.state.toggleTimelineModal}
             />
             {this.state.showModal && 
-            <ConfirmCancelModal 
+            <ConfirmModal 
               headerText={"Haluatko peruuttaa tekemäsi muutokset?"} 
               contentText={"Olet muuttanut aikataulun tietoja. Mikäli jatkat, tekemäsi muutokset peruutetaan. Haluatko jatkaa?"} 
               button1Text={"Jatka"} 
@@ -629,6 +629,10 @@ EditProjectTimeTableModal.propTypes = {
   allowedToEdit: PropTypes.bool,
   attributeData: PropTypes.object,
   isAdmin: PropTypes.bool,
+  formSubmitErrors: PropTypes.object,
+  deadlineSections: PropTypes.array,
+  formValues: PropTypes.object,
+  deadlines: PropTypes.array
 }
 
 const mapStateToProps = state => ({

src/components/projectEdit/ProjectEdit.scss

@@ -533,7 +533,6 @@
     margin-top: 20px;
     margin-left: 0px !important;
     margin-right: 20px !important;
-    //max-height: 700px !important;
     overflow: auto !important;
     .date-content{
       display: flex;