Disable select when only one domain exiists (#29873)

Automattic · Jan 3, 2019 · f890d85 · f890d85
1 parent 12c38a5
commit f890d85
Show file tree

Hide file tree

Showing 3 changed files with 10 additions and 3 deletions.
diff --git a/client/my-sites/domains/domain-management/add-google-apps/domains-select.jsx b/client/my-sites/domains/domain-management/add-google-apps/domains-select.jsx
@@ -28,13 +28,14 @@ class DomainsSelect extends React.Component {
 	}
 
 	render() {
-		const { isRequestingSiteDomains, onChange, onFocus, value } = this.props;
+		const { domains, isRequestingSiteDomains, onChange, onFocus, value } = this.props;
+
 		return (
 			<select
 				value={ value }
 				onChange={ onChange }
 				onFocus={ onFocus }
-				disabled={ isRequestingSiteDomains }
+				disabled={ isRequestingSiteDomains || 1 === domains.length }
 			>
 				{ isRequestingSiteDomains && this.renderLoadingState() }
 				{ ! isRequestingSiteDomains && this.renderDomainSelect() }

diff --git a/...sites/domains/domain-management/add-google-apps/test/__snapshots__/domains-select.js.snap b/...sites/domains/domain-management/add-google-apps/test/__snapshots__/domains-select.js.snap
@@ -16,7 +16,7 @@ exports[`DomainSelect it renders DomainsSelect loading state correctly 1`] = `
 
 exports[`DomainSelect it renders DomainsSelect with one domain correctly 1`] = `
 <select
-  disabled={false}
+  disabled={true}
   onChange={[Function]}
   onFocus={[Function]}
   value="foo"

diff --git a/client/my-sites/domains/domain-management/style.scss b/client/my-sites/domains/domain-management/style.scss
@@ -712,6 +712,12 @@ ul.email-forwarding__list {
 			margin-left: -1px;
 		}
 	}
+
+	select:disabled {
+		background: none;
+		cursor: default;
+		padding-right: 14px;
+	}
 }
 
 .name-servers__dns {