Skip to content

Commit

Permalink
Fix #6 : Merge pull request #16 from mandy6720/fix-slider-on-small-sc…
Browse files Browse the repository at this point in the history
…reen

Makes fixed stuff on top always have higher z-index
  • Loading branch information
emibcn authored Oct 8, 2020
2 parents 91be957 + 1aa3e55 commit 0254361
Show file tree
Hide file tree
Showing 3 changed files with 7 additions and 2 deletions.
5 changes: 5 additions & 0 deletions app/README.md
Original file line number Diff line number Diff line change
@@ -1,5 +1,10 @@
This project was bootstrapped with [Create React App](https://github.com/facebook/create-react-app).

## Installation

From the root of the repo, run `cd app && npm install`.
Once installation is finished, you can run the following commands to run your project locally. Note: if you don't have yarn installed globally, you can run `npm i -g yarn` to do a global install or add `npx` at the start of your commands, e.g. `npx yarn start`

## Available Scripts

In the project directory, you can run:
Expand Down
2 changes: 1 addition & 1 deletion app/src/Slider.jsx
Original file line number Diff line number Diff line change
Expand Up @@ -13,7 +13,7 @@ import './Slider.scss';
// Use special non-intrusive zIndex for the Slider tooltip
const useStyles = makeStyles((theme) => ({
popper: {
zIndex: 1,
zIndex: 1200
},
tooltip: {
fontSize: '1rem',
Expand Down
2 changes: 1 addition & 1 deletion app/src/Widget/List.jsx
Original file line number Diff line number Diff line change
Expand Up @@ -41,7 +41,7 @@ const useStyles = (theme) => ({
fixed: {
position: 'sticky',
top: 0,
zIndex: 1,
zIndex: 1200,
padding: 0,
// Not using it, adds a scrolling slider to the container on some screens
paddingBottom: 1,
Expand Down

0 comments on commit 0254361

Please sign in to comment.