Skip to content

Commit 3a8b500

Browse files
authored
fix(Slider): correct tab order when input is above thumb (#8190)
1 parent 9d771b7 commit 3a8b500

File tree

2 files changed

+14
-14
lines changed

2 files changed

+14
-14
lines changed

packages/react-core/src/components/Slider/Slider.tsx

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -416,16 +416,16 @@ export const Slider: React.FunctionComponent<SliderProps> = ({
416416
{buildSteps()}
417417
</div>
418418
)}
419+
{isInputVisible && inputPosition === 'aboveThumb' && (
420+
<div className={css(styles.sliderValue, styles.modifiers.floating)}>{displayInput()}</div>
421+
)}
419422
{hasTooltipOverThumb ? (
420423
<Tooltip entryDelay={0} content={findAriaTextValue()}>
421424
{thumbComponent}
422425
</Tooltip>
423426
) : (
424427
thumbComponent
425428
)}
426-
{isInputVisible && inputPosition === 'aboveThumb' && (
427-
<div className={css(styles.sliderValue, styles.modifiers.floating)}>{displayInput()}</div>
428-
)}
429429
</div>
430430
{isInputVisible && inputPosition === 'right' && <div className={css(styles.sliderValue)}>{displayInput()}</div>}
431431
{rightActions && <div className={css(styles.sliderActions)}>{rightActions}</div>}

packages/react-core/src/components/Slider/_tests_/__snapshots__/Slider.test.tsx.snap

Lines changed: 11 additions & 11 deletions
Original file line numberDiff line numberDiff line change
@@ -475,17 +475,6 @@ exports[`slider renders slider with input above thumb 1`] = `
475475
</div>
476476
</div>
477477
</div>
478-
<div
479-
aria-disabled="false"
480-
aria-label="Value"
481-
aria-valuemax="100"
482-
aria-valuemin="0"
483-
aria-valuenow="50"
484-
aria-valuetext="50"
485-
class="pf-c-slider__thumb"
486-
role="slider"
487-
tabindex="0"
488-
/>
489478
<div
490479
class="pf-c-slider__value pf-m-floating"
491480
>
@@ -509,6 +498,17 @@ exports[`slider renders slider with input above thumb 1`] = `
509498
</span>
510499
</div>
511500
</div>
501+
<div
502+
aria-disabled="false"
503+
aria-label="Value"
504+
aria-valuemax="100"
505+
aria-valuemin="0"
506+
aria-valuenow="50"
507+
aria-valuetext="50"
508+
class="pf-c-slider__thumb"
509+
role="slider"
510+
tabindex="0"
511+
/>
512512
</div>
513513
</div>
514514
</DocumentFragment>

0 commit comments

Comments
 (0)