Merge "Add deployment configuration IDs"

This commit is contained in:
Jenkins 2017-07-17 12:57:36 +00:00 committed by Gerrit Code Review
commit 21b6f09045
4 changed files with 20 additions and 7 deletions

View File

@ -33,7 +33,10 @@ const ConfigurePlanStep = props => {
<div>
<DeploymentConfigurationSummary {...props} />
&nbsp;
<Link to={`/plans/${props.planName}/configuration`}>
<Link
id="ConfigurePlanStep__EditDeploymentLink"
to={`/plans/${props.planName}/configuration`}
>
<FormattedMessage {...messages.editConfigurationLink} />
</Link>
</div>

View File

@ -45,7 +45,7 @@ class DeploymentConfiguration extends React.Component {
render() {
const { location, match } = this.props;
return (
<Modal dialogClasses="modal-xl">
<Modal id="DeploymentConfiguration__ModalDialog" dialogClasses="modal-xl">
<div className="modal-header">
<Link
to={`/plans/${match.params.planName}`}
@ -60,10 +60,16 @@ class DeploymentConfiguration extends React.Component {
</div>
<ul className="nav nav-tabs">
<NavTab to={`${match.url}/environment`}>
<NavTab
id="DeploymentConfiguration__OverallSettingsTab"
to={`${match.url}/environment`}
>
<FormattedMessage {...messages.overallSettings} />
</NavTab>
<NavTab to={`${match.url}/parameters`}>
<NavTab
id="DeploymentConfiguration__ParametersTab"
to={`${match.url}/parameters`}
>
<FormattedMessage {...messages.parameters} />
</NavTab>
</ul>

View File

@ -186,7 +186,10 @@ class EnvironmentConfiguration extends React.Component {
<div className="container-fluid">
<div className="row row-eq-height">
<div className="col-sm-4 sidebar-pf sidebar-pf-left">
<ul className="nav nav-pills nav-stacked nav-arrows">
<ul
id="DeploymentConfiguration__CategoriesList"
className="nav nav-pills nav-stacked nav-arrows"
>
{topicTabs}
</ul>
</div>

View File

@ -18,14 +18,14 @@ import PropTypes from 'prop-types';
import React from 'react';
import { Link, Route } from 'react-router-dom';
const NavTab = ({ activeClassName, children, to, exact, location }) => {
const NavTab = ({ activeClassName, children, to, exact, location, id }) => {
return (
<Route
location={location}
path={typeof to === 'object' ? to.pathname : to}
exact={exact}
children={({ match, location }) => (
<li className={match ? activeClassName : ''}>
<li className={match ? activeClassName : ''} id={id}>
<Link to={to}>{children}</Link>
</li>
)}
@ -36,6 +36,7 @@ NavTab.propTypes = {
activeClassName: PropTypes.string.isRequired,
children: PropTypes.node,
exact: PropTypes.bool.isRequired,
id: PropTypes.string,
location: PropTypes.object,
to: PropTypes.oneOfType([PropTypes.string, PropTypes.object]).isRequired
};