Enhancement: Setting Reject status will now require text before rejec…

…ting. (#1218)

* Force user input before reject

* remove semi colon

* formatting and redundant code

client/src/js/collectionReview.js

@@ -1655,29 +1655,40 @@ async function addCollectionReview ( params ) {
 				const textArea = new Ext.form.TextArea({
 					emptyText: 'Provide feedback explaining this rejection.',
 					maxLength: 255,
+					enableKeyEvents: true,
 					listeners: {
-						valid: () => {
-							submitBtn.enable()
-						},
-						invalid: () => {
-							submitBtn.disable()
+						keyup: (field) => {
+							if (field.isValid() && field.getValue().trim().length > 0) {
+								submitBtn.enable()
+							} 
+							else {
+								submitBtn.disable()
+							}
 						}
 					}
 				})
 				const submitBtn = new Ext.Button({
 					text: 'Reject with this feedback',
 					action: 'reject',
 					iconCls: 'sm-rejected-icon',
+					disabled: true,
+					handler
+				})
+				const cancelBtn = new Ext.Button(	{
+					text: 'Cancel',
+					action: 'cancel',
 					handler
 				})
 				function handler (btn) {
-					if (btn.action === 'reject') {
-						const value = textArea.getValue()
+					const value = textArea.getValue()
+					if (btn.action === 'reject'){
 						fpwindow.close()
 						resolve(value)
 					}
-					fpwindow.close()
-					reject()
+					else{
+						fpwindow.close()
+						reject()
+					}
 				}
 				const fpwindow = new Ext.Window({
 					title: `Reject Reviews`,
@@ -1690,22 +1701,11 @@ async function addCollectionReview ( params ) {
 					plain: true,
 					bodyStyle: 'padding:5px;',
 					buttonAlign: 'right',
-					items: [
-						textArea
-					],
-					buttons: [
-						{
-							text: 'Cancel',
-							action: 'cancel',
-							handler
-						},
-						submitBtn
-					]
+					items: [textArea],
+					buttons: [cancelBtn,submitBtn]
 				})
 				fpwindow.show()
-
 			})
-
 		}
 
 		async function handleStatusChange (grid, sm, status) {