-
Notifications
You must be signed in to change notification settings - Fork 123
Commit
This commit does not belong to any branch on this repository, and may belong to a fork outside of the repository.
Add simplebar which was missing to get sidebar styling correct
- Loading branch information
1 parent
d520e88
commit 8e96219
Showing
2 changed files
with
5 additions
and
0 deletions.
There are no files selected for viewing
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Original file line number | Diff line number | Diff line change |
---|---|---|
|
@@ -12,3 +12,7 @@ | |
|
||
pin "chartkick", to: "chartkick.js" | ||
pin "Chart.bundle", to: "Chart.bundle.js" | ||
pin "simplebar", to: "https://ga.jspm.io/npm:[email protected]/dist/index.mjs" | ||
pin "can-use-dom", to: "https://ga.jspm.io/npm:[email protected]/index.js" | ||
pin "lodash-es", to: "https://ga.jspm.io/npm:[email protected]/lodash.js" | ||
pin "simplebar-core", to: "https://ga.jspm.io/npm:[email protected]/dist/index.mjs" |
8e96219
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Did this hide the navbar options? They are in the DOM but not visible in the UI. I think they are behind the nav element or something...looking at it now.
There's
display: flex
css being added somewhere. Removing that on the topnav
element fixes the issue. Not sure where it's coming from in the erb file yet.