Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Range slider component #793

Closed
wants to merge 25 commits into from
Closed
Changes from 1 commit
Commits
File filter...
Filter file types
Jump to…
Jump to file or symbol
Failed to load files and symbols.

Always

Just for now

zindex comment

  • Loading branch information...
mikejolley committed Aug 8, 2019
commit 5cc89074b8e77e236b58064bfa9e66c59edfee0d
@@ -168,6 +168,10 @@ class PriceSlider extends Component {
const minXDiff = Math.abs( x - minX );
const maxXDiff = Math.abs( x - maxX );

/**
* The default z-index in the stylesheet as 20. 20 vs 21 is just for determining which range
* slider should be at the front and has no meaning beyond this.
*/
if ( minXDiff > maxXDiff ) {
this.minRange.current.style.zIndex = 20;
this.maxRange.current.style.zIndex = 21;
ProTip! Use n and p to navigate between commits in a pull request.
You can’t perform that action at this time.