other(editor): Moved the identifier & aliases editin to be part of th…

…e man editing flow

src/client/entity-editor/alias-editor/alias-editor.js

@@ -16,14 +16,14 @@
  * 51 Franklin Street, Fifth Floor, Boston, MA 02110-1301 USA.
  */
 
-import {Button, Modal, OverlayTrigger, Tooltip} from 'react-bootstrap';
-import {hideAliasEditor, removeEmptyAliases} from './actions';
+import { Button, Modal, OverlayTrigger, Tooltip } from 'react-bootstrap';
+import { hideAliasEditor, removeEmptyAliases } from './actions';
 import AliasModalBody from './alias-modal-body';
-import {FontAwesomeIcon} from '@fortawesome/react-fontawesome';
+import { FontAwesomeIcon } from '@fortawesome/react-fontawesome';
 import PropTypes from 'prop-types';
 import React from 'react';
-import {connect} from 'react-redux';
-import {faQuestionCircle} from '@fortawesome/free-solid-svg-icons';
+import { connect } from 'react-redux';
+import { faQuestionCircle } from '@fortawesome/free-solid-svg-icons';
 
 
 /**
@@ -37,15 +37,13 @@ import {faQuestionCircle} from '@fortawesome/free-solid-svg-icons';
  * @param {Array} props.languageOptions - The list of possible languages for an
  *        alias.
  * @param {Function} props.onClose - A function to be called when the button to
- *        close the editor is clicked.
- * @param {boolean} props.show - Whether or not the editor modal should be
- *        visible.
+ *        add the alias is clicked.
  * @returns {ReactElement} React element containing the rendered AliasEditor.
  */
 const AliasEditor = ({
 	languageOptions,
- 	onClose,
-	// show 
+	onClose,
+
 }) => {
 	const helpText = `Variant names for an entity such as alternate spelling, different script, stylistic representation, acronyms, etc.
 		Refer to the help page for more details and examples.`;
@@ -65,16 +63,16 @@ const AliasEditor = ({
 	return (
 		<div>
 			<div>
-			<div style={{display:'flex'}}>
-					<h2 style={{marginRight:"5px"}}>Add a Alias</h2> 
-					<div style={{marginTop:"15px"}}>
+				<div style={{ display: 'flex' }}>
+					<h2 style={{ marginRight: "5px" }}>Add a Alias</h2>
+					<div style={{ marginTop: "15px" }}>
 						{helpIconElement}
-						</div>
+					</div>
 				</div>
 			</div>
 
 			<div>
-				<AliasModalBody languageOptions={languageOptions}/>
+				<AliasModalBody languageOptions={languageOptions} />
 			</div>
 
 			<div>
@@ -87,16 +85,14 @@ AliasEditor.displayName = 'AliasEditor';
 AliasEditor.propTypes = {
 	languageOptions: PropTypes.array.isRequired,
 	onClose: PropTypes.func.isRequired,
-	//show: PropTypes.bool
+
 };
-// AliasEditor.defaultProps = {
-// 	show: false
-// };
+
 
 function mapDispatchToProps(dispatch) {
 	return {
 		onClose: () => {
-			//dispatch(hideAliasEditor());
+
 			dispatch(removeEmptyAliases());
 		}
 	};

src/client/entity-editor/identifier-editor/identifier-editor.js

@@ -16,14 +16,14 @@
  * 51 Franklin Street, Fifth Floor, Boston, MA 02110-1301 USA.
  */
 
-import {Button, Modal, OverlayTrigger, Tooltip} from 'react-bootstrap';
-import {hideIdentifierEditor, removeEmptyIdentifiers} from './actions';
-import {FontAwesomeIcon} from '@fortawesome/react-fontawesome';
+import { Button, Modal, OverlayTrigger, Tooltip } from 'react-bootstrap';
+import { hideIdentifierEditor, removeEmptyIdentifiers } from './actions';
+import { FontAwesomeIcon } from '@fortawesome/react-fontawesome';
 import IdentifierModalBody from './identifier-modal-body';
 import PropTypes from 'prop-types';
 import React from 'react';
-import {connect} from 'react-redux';
-import {faQuestionCircle} from '@fortawesome/free-solid-svg-icons';
+import { connect } from 'react-redux';
+import { faQuestionCircle } from '@fortawesome/free-solid-svg-icons';
 
 
 /**
@@ -39,16 +39,13 @@ import {faQuestionCircle} from '@fortawesome/free-solid-svg-icons';
  * @param {Function} props.onAddIdentifier - A function to be called when the
  *        button to add an identifier is clicked.
  * @param {Function} props.onClose - A function to be called when the button to
- *        close the editor is clicked.
- * @param {boolean} props.show - Whether or not the editor modal should be
- *        visible.
+ *        Add the new identifier is clicked.
  * @returns {ReactElement} React element containing the rendered
  *          IdentifierEditor.
  */
 const IdentifierEditor = ({
 	identifierTypes,
-	 onClose,
-	// show
+	onClose,
 }) => {
 	const helpText = `identity of the entity in other databases and services, such as ISBN, barcode, MusicBrainz ID, WikiData ID, OpenLibrary ID, etc.
 	You can enter either the identifier only (Q2517049) or a full link (https://www.wikidata.org/wiki/Q2517049).`;
@@ -69,39 +66,36 @@ const IdentifierEditor = ({
 	return (
 		<div>
 			<div>
-				<div style={{display:'flex'}}>
-					<h2 style={{marginRight:"5px"}}>Add a Identifier</h2> 
-					<div style={{marginTop:"15px"}}>
+				<div style={{ display: 'flex' }}>
+					<h2 style={{ marginRight: "5px" }}>Add a Identifier</h2>
+					<div style={{ marginTop: "15px" }}>
 						{helpIconElement}
 					</div>
 				</div>
 			</div>
 
 			<div>
-				<IdentifierModalBody identifierTypes={identifierTypes}/>
+				<IdentifierModalBody identifierTypes={identifierTypes} />
 			</div>
 			<div>
-			<Button variant="primary" onClick={onClose}>Done</Button>
+				<Button variant="primary" onClick={onClose}>Done</Button>
 			</div>
 
-			
+
 		</div>
 	);
 };
 IdentifierEditor.displayName = 'IdentifierEditor';
 IdentifierEditor.propTypes = {
 	identifierTypes: PropTypes.array.isRequired,
 	onClose: PropTypes.func.isRequired,
-	//show: PropTypes.bool
 };
-// IdentifierEditor.defaultProps = {
-// 	show: false
-// };
+
 
 function mapDispatchToProps(dispatch) {
 	return {
 		onClose: () => {
-			//dispatch(hideIdentifierEditor());
+
 			dispatch(removeEmptyIdentifiers());
 		}
 	};