From c49b99e0bcbbfa8d7e6c201ced34f3d3e6b4f966 Mon Sep 17 00:00:00 2001
From: Jeremy Herve <jeremy@jeremy.hu>
Date: Wed, 25 Sep 2019 14:55:59 +0200
Subject: [PATCH] Admin Page: change default settings tab depending on your
 role

Editors shouldn't see the Security tab when they load wp-admin/admin.php?page=jetpack#/settings,
since they can't see anything on that tab. They should see the Writing tab instead, where they can use things like Post By Email.
---
 _inc/client/settings/index.jsx | 10 ++++++++--
 1 file changed, 8 insertions(+), 2 deletions(-)

diff --git a/_inc/client/settings/index.jsx b/_inc/client/settings/index.jsx
index 057c47341d478..b45449a2dbe57 100644
--- a/_inc/client/settings/index.jsx
+++ b/_inc/client/settings/index.jsx
@@ -43,7 +43,10 @@ export default class extends React.Component {
 				<Security
 					siteAdminUrl={ this.props.siteAdminUrl }
 					siteRawUrl={ this.props.siteRawUrl }
-					active={ '/security' === this.props.route.path || '/settings' === this.props.route.path }
+					active={
+						'/security' === this.props.route.path ||
+						( '/settings' === this.props.route.path && this.props.userCanManageModules )
+					}
 					{ ...commonProps }
 				/>
 				<Discussion
@@ -60,7 +63,10 @@ export default class extends React.Component {
 				/>
 				<Writing
 					siteAdminUrl={ this.props.siteAdminUrl }
-					active={ '/writing' === this.props.route.path }
+					active={
+						'/writing' === this.props.route.path ||
+						( '/settings' === this.props.route.path && ! this.props.userCanManageModules )
+					}
 					{ ...commonProps }
 				/>
 				<Sharing