Skip to content

Commit 1c12c7b

Browse files
authored
Merge pull request #40 from coreui/fix-rtl-sidebar-ps-issue
fix(SidebarNav): dirty fix for rtl ps scrollingX issue
2 parents 1bf5f2a + 862c67b commit 1c12c7b

File tree

2 files changed

+6
-3
lines changed

2 files changed

+6
-3
lines changed

Diff for: package.json

+2-2
Original file line numberDiff line numberDiff line change
@@ -1,6 +1,6 @@
11
{
22
"name": "@coreui/react",
3-
"version": "2.0.6",
3+
"version": "2.0.7",
44
"description": "CoreUI React Bootstrap 4 components",
55
"license": "MIT",
66
"author": {
@@ -46,7 +46,7 @@
4646
"reactstrap": "^6.4.0"
4747
},
4848
"peerDependencies": {
49-
"@coreui/coreui": "^2.0.12",
49+
"@coreui/coreui": "^2.0.14",
5050
"react": "16.x"
5151
},
5252
"devDependencies": {

Diff for: src/SidebarNav.js

+4-1
Original file line numberDiff line numberDiff line change
@@ -173,9 +173,12 @@ class AppSidebarNav extends Component {
173173

174174
const navClasses = classNames(className, 'sidebar-nav');
175175

176+
// ToDo: find better rtl fix
177+
const isRtl = getComputedStyle(document.querySelector('html')).direction === 'rtl'
178+
176179
// sidebar-nav root
177180
return (
178-
<PerfectScrollbar className={navClasses} {...attributes} option={{ suppressScrollX: true }} >
181+
<PerfectScrollbar className={navClasses} {...attributes} option={{ suppressScrollX: !isRtl }} >
179182
<Nav>
180183
{children || this.navList(navConfig.items)}
181184
</Nav>

0 commit comments

Comments
 (0)