Skip to content

Commit

Permalink
Merge pull request #9396 from rhamilto/console-2838
Browse files Browse the repository at this point in the history
CONSOLE-2838:  masthead changes to better align with ACM
  • Loading branch information
openshift-merge-robot committed Jul 23, 2021
2 parents dfa7e1b + af0ba16 commit f603b9f
Show file tree
Hide file tree
Showing 3 changed files with 16 additions and 6 deletions.
15 changes: 15 additions & 0 deletions frontend/public/components/masthead-toolbar.jsx
Expand Up @@ -24,6 +24,7 @@ import { FLAGS, YellowExclamationTriangleIcon, ACM_LINK_ID } from '@console/shar
import { formatNamespacedRouteForResource } from '@console/shared/src/utils';
import CloudShellMastheadButton from '@console/app/src/components/cloud-shell/CloudShellMastheadButton';
import CloudShellMastheadAction from '@console/app/src/components/cloud-shell/CloudShellMastheadAction';
import isMultiClusterEnabled from '@console/app/src/utils/isMultiClusterEnabled';
import * as UIActions from '../actions/ui';
import { connectToFlags } from '../reducers/connectToFlags';
import { flagPending, featureReducerName } from '../reducers/features';
Expand Down Expand Up @@ -327,6 +328,20 @@ class MastheadToolbarContents_ extends React.Component {
fireTelemetryEvent('Documentation Clicked');
},
},
...(isMultiClusterEnabled()
? [
{
label: t('public~ACM Documentation'),
externalLink: true,
// TODO: add version number to end of URL
href:
'https://access.redhat.com/documentation/en-us/red_hat_advanced_cluster_management_for_kubernetes',
callback: () => {
fireTelemetryEvent('ACM Documentation Clicked');
},
},
]
: []),
...(flags[FLAGS.CONSOLE_CLI_DOWNLOAD]
? [
{
Expand Down
1 change: 1 addition & 0 deletions frontend/public/locales/en/public.json
Expand Up @@ -643,6 +643,7 @@
"Red Hat applications": "Red Hat applications",
"Quick Starts": "Quick Starts",
"Documentation": "Documentation",
"ACM Documentation": "ACM Documentation",
"Command line tools": "Command line tools",
"About": "About",
"Language preference": "Language preference",
Expand Down
6 changes: 0 additions & 6 deletions frontend/public/style/_overrides.scss
Expand Up @@ -83,12 +83,6 @@ h6 {
font-family: $font-family-base;
}

// Temp fix to adjust user menu dropdown toggle padding until it can be converted back to a standard dropdown
.co-user-menu .pf-c-app-launcher__toggle {
padding-left: 0;
padding-right: 0;
}

.pf-c-breadcrumb {
padding-bottom: 12px;
padding-top: 25px;
Expand Down

0 comments on commit f603b9f

Please sign in to comment.