feat: create Slider component#8485

Merged

vursen merged 56 commits intomainfrom

slider-component

Feb 4, 2026

Contributor

vursen commented Jan 14, 2026 •

edited

Loading

Description

The PR introduces a Flow counterpart for the <vaadin-slider> web component, with the following API:

Constructors:

Slider()
Slider(valueChangeListener)
Slider(min, max, value)
Slider(min, max, value, valueChangeListener)
Slider(min, max, step, value)
Slider(min, max, step, value, valueChangeListener)
Slider(label)
Slider(label, valueChangeListener)
Slider(label, min, max, value)
Slider(label, min, max, value, valueChangeListener)
Slider(label, min, max, step, value)
Slider(label, min, max, step, value, valueChangeListener)

Methods:

setValue(value)
setValue(value, min, max)
setValue(value, min, max, step)
setMin(min)
getMin()
setMax(max)
getMax()
setStep(step)
getStep()

Fixes #8480

Type of change

Feature

vursen force-pushed the slider-component branch from db3a5dd to 2282aaf Compare

January 14, 2026 12:47

vursen changed the title ~~feat: add vaadin-slider-flow-parent module~~ feat: create Slider component

sissbruecker reviewed

View reviewed changes

vaadin-slider-flow-parent/vaadin-slider-flow-integration-tests/pom.xml

vaadin-slider-flow-parent/pom.xml

vursen force-pushed the slider-component branch 2 times, most recently from ce3a04e to 35b5b9d Compare

January 16, 2026 08:42

knoobie reviewed

View reviewed changes

...er-flow-parent/vaadin-slider-flow/src/main/java/com/vaadin/flow/component/slider/Slider.java

+              @Tag("vaadin-slider")
+              // @NpmPackage(value = "@vaadin/slider", version = "25.1.0-alpha1")
+              // @JsModule("@vaadin/slider/src/vaadin-slider.js")
+              public class Slider extends SliderBase<Slider, Double> {

Contributor

knoobie Jan 16, 2026 •

edited

Loading

Thinking out load: I'm wondering if it's better for java developer to either use

Slider<T extends Number> or IntSlider / DoubleSlider.

Having to deal with double all the time when I just have a simple 0-5 slider feels awkward. (Same could be said for min/max and step.. might be good to add overloaded methods?)

Contributor Author

vursen Jan 19, 2026

Valid point, thank you. We could add an IntegerSlider later

vursen linked an issue

that may be closed by this pull request

[Slider] Add feature flag to activate flow counterparts #8482

Closed

sissbruecker reviewed

View reviewed changes

vaadin-slider-flow-parent/vaadin-slider-flow/pom.xml

vursen mentioned this pull request

feat: create RangeSlider component #8503

Merged

1 task

vursen and others added 21 commits

February 2, 2026 18:30


          feat: add vaadin-slider-flow-parent module

603f030

Add Slider Flow component wrapping <vaadin-slider> web component.
The Slider extends AbstractSinglePropertyField and provides min, max,
step, and value properties.

Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>


          fix: address PR review comments

4e545b6

- Remove explicit flow dependency versions from integration-tests pom.xml
- Add README.md and LICENSE files
- Add vite.config.ts for integration tests

Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>


          test: add unit tests for Slider component

1e988d6

Add tests for constructors, min/max/step properties, and validation
that throws IllegalArgumentException for invalid values.

Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>


          test: add unit test for setStep IllegalArgumentException

65e4456

Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>


          add test for serializability

e70d99c


          feat: add experimental feature flag and basic integration test for Sl…

772310c

…ider

- Add ExperimentalFeatureException thrown when feature flag is disabled
- Add feature flag check in Slider.onAttach
- Add getMin, getMax, getStep methods to SliderElement
- Add basic integration test verifying slider properties

Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>


          run formatter

70d35d3


          implement Focusable and KeyNotifier interfaces

ee3ea09


          refactor: extract SliderBase abstract class with min, max, step prope…

f515c69

…rties

- Create SliderBase<TComponent, TValue> abstract class extending AbstractSinglePropertyField
- Move min, max, step getters/setters with validation to SliderBase
- Update Slider to extend SliderBase<Slider, Double>

Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>


          docs: improve JavaDoc descriptions for Slider constructors

4a2deeb

Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>


          test: add unit tests for Slider constructors and basic validation

5321f51

Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>


          test: add SliderBase tests and refactor Slider tests

62b7af4

Move min, max, step tests to SliderBaseTest. Add tests for SliderBase
constructors and value change listener constructors.

Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>


          docs: add JavaDoc for Slider constructors and unit tests

58f556c

Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>


          move interfaces to base class, format fixes

ad9ed94


          refactor SliderBase to support IntegerSlider in the future

7e535db


          feat: add scheduleBeforeClientResponse helper to SliderBase

12d9359

Add a utility method to schedule actions that run before the client
response, with deduplication by key. This allows deferring state
validity checks until all property changes are complete.

Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>


          feat: add state validity warnings for Slider

5b36df8

Log warnings when setMin, setMax, or setStep is called and the current
value becomes inconsistent with the new constraints. The warnings are
deferred using beforeClientResponse to check the final state after all
property changes in the current request are complete.

Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>


          test: add unit tests for Slider state validity warnings

859d4be

Add tests verifying that warnings are logged when value doesn't align
with min, max, or step constraints after property changes.

Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>


          docs: update JavaDoc for Slider constructors to include step parameter

548bf73

Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>


          refactor: change SliderBase constructor visibility and update tests

40403b1

Change SliderBase constructor from protected to package-private.
Update SliderTest to use the 4-parameter constructor with step value.

Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>


          minor corrections

6497cc0

vursen marked this pull request as ready for review

February 3, 2026 12:06

vursen requested review from sissbruecker and web-padawan

February 3, 2026 12:09

sissbruecker reviewed

View reviewed changes

...low-parent/vaadin-slider-flow/src/main/java/com/vaadin/flow/component/slider/SliderBase.java Outdated

...er-flow-parent/vaadin-slider-flow/src/main/java/com/vaadin/flow/component/slider/Slider.java

...er-flow-parent/vaadin-slider-flow/src/main/java/com/vaadin/flow/component/slider/Slider.java Outdated

...rent/vaadin-slider-flow/src/test/java/com/vaadin/flow/component/slider/tests/SliderTest.java

...rent/vaadin-slider-flow/src/test/java/com/vaadin/flow/component/slider/tests/SliderTest.java Outdated

...low-parent/vaadin-slider-flow/src/main/java/com/vaadin/flow/component/slider/SliderBase.java

vursen and others added 9 commits

February 3, 2026 18:25


          change values in step test

e2e5f57


          refactor: make setValue overloads public in SliderBase

2939ce3

Remove redundant override methods in Slider that just called super.

Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>


          refactor: reorder setValue parameters to put value first

591bb8f

Change setValue(min, max, value) to setValue(value, min, max) and
setValue(min, max, step, value) to setValue(value, min, max, step).

Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>


          docs: clarify that step is relative to min value in getStep JavaDoc

Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>


          test: add test for setValue(null)

552a98d

Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>


          refactor: implement InputField instead of individual interfaces

9b0562f

SliderBase now implements InputField which consolidates HasLabel,
HasHelper, HasSize, HasStyle, HasTooltip, HasEnabled, and HasValue.

Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>


          refactor: simplify clear() to call setValue(getMin()) directly

21b5662

Remove getEmptyValue() override and have clear() directly set value
to minimum. Add test for clear() behavior.

Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>


          fix: ignore invalid values from client side instead of throwing

5caad99

Invalid values (out of range or not aligned with step) sent from the
client are now silently ignored, returning the current value instead
of throwing an exception.

Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>


          Merge branch 'main' into slider-component

c7ec6ab

sissbruecker reviewed

View reviewed changes

...slider-testbench/src/main/java/com/vaadin/flow/component/slider/testbench/SliderElement.java Outdated

...er-flow-parent/vaadin-slider-flow/src/main/java/com/vaadin/flow/component/slider/Slider.java

vursen and others added 3 commits

February 4, 2026 11:06


          format code

fe5ca7e


          feat: sync slider value on change event and improve testbench element

767e096

Synchronize slider value with the server on the change event instead
of relying on deferred updates. This also adds TestBench utility
interfaces and a setValue method for emulating user input in tests.

Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>


          test: add integration test for SliderElement.setValue

d6031da

Verify that setValue synchronizes the value to the server by checking
the value change event is fired with the correct value.

Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>

sissbruecker approved these changes

View reviewed changes

web-padawan reviewed

View reviewed changes

...er-flow-parent/vaadin-slider-flow/src/main/java/com/vaadin/flow/component/slider/Slider.java Outdated

...er-flow-parent/vaadin-slider-flow/src/main/java/com/vaadin/flow/component/slider/Slider.java Outdated

...er-flow-parent/vaadin-slider-flow/src/main/java/com/vaadin/flow/component/slider/Slider.java Outdated

vursen and others added 3 commits

February 4, 2026 12:57


          chore: update @vaadin/slider to 25.1.0-alpha6

e22a6a1

Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>


          refactor: replace "range" with "min, max" in Javadoc

3815fb9

Avoid using the word "range" in Slider Javadoc to prevent confusion
with RangeSlider. Also rename requireValidRange to requireValidMinMax.

Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>


          Merge branch 'main' into slider-component

5929c29

web-padawan approved these changes

View reviewed changes


          format code

bd2464a

sonarqubecloud Bot commented Feb 4, 2026

Quality Gate passed

Issues
22 New issues
0 Accepted issues

Measures
0 Security Hotspots
0.0% Coverage on New Code
0.0% Duplication on New Code

See analysis details on SonarQube Cloud

vursen enabled auto-merge (squash)

February 4, 2026 09:29

vursen merged commit d5164db into main

5 checks passed

vursen deleted the slider-component branch

February 4, 2026 10:09

vaadin-bot added the Released with Vaadin 25.1.0-alpha6 label

Collaborator

vaadin-bot commented Mar 10, 2026

This ticket/PR has been released with Vaadin 25.1.0-alpha6.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Released with Vaadin 25.1.0-alpha6