Skip to content

Commit

Permalink
Merge pull request #958 from Hyperkid123/fix-navigation-text-align
Browse files Browse the repository at this point in the history
Fix navigation text alignment.
  • Loading branch information
Hyperkid123 authored Sep 11, 2020
2 parents 79c79fe + 7587d9f commit 88f7a34
Show file tree
Hide file tree
Showing 3 changed files with 10 additions and 2 deletions.
1 change: 1 addition & 0 deletions src/js/App/Sidenav/Navigation.js
Original file line number Diff line number Diff line change
Expand Up @@ -100,6 +100,7 @@ export class Navigation extends Component {
if (!(item.disabled_on_stable && window.location.pathname.indexOf('/beta') === -1)) {
if (item.subItems) {
return <NavExpandable
className="ins-m-navigation-align"
title={item.title}
ouia-nav-group={item.id}
itemID={item.id}
Expand Down
4 changes: 2 additions & 2 deletions src/js/App/Sidenav/__snapshots__/Navigation.test.js.snap
Original file line number Diff line number Diff line change
Expand Up @@ -244,7 +244,7 @@ exports[`Navigation should render corectly 1`] = `
ariaRightScroll="Scroll right"
>
<NavExpandable
className=""
className="ins-m-navigation-align"
groupId={null}
id=""
isActive={true}
Expand Down Expand Up @@ -351,7 +351,7 @@ exports[`Navigation should render correctly 2 1`] = `
ariaRightScroll="Scroll right"
>
<NavExpandable
className=""
className="ins-m-navigation-align"
groupId={null}
id=""
isActive={true}
Expand Down
7 changes: 7 additions & 0 deletions src/sass/chrome.scss
Original file line number Diff line number Diff line change
Expand Up @@ -178,3 +178,10 @@ section.ins-c-app-switcher--loading {
.ins-c-tagfilter__option-value { max-width: 360px; }
}
}

// PF navigation text alignment fix
.ins-m-navigation-align {
.pf-c-nav__link {
text-align: inherit;
}
}

0 comments on commit 88f7a34

Please sign in to comment.