Skip to content

FORMS-28701 - #2022

Open
armaang1729 wants to merge 6 commits into
devfrom
FORMS-28701
Open

armaang1729 wants to merge 6 commits into
devfrom
FORMS-28701

Conversation

@armaang1729

Copy link
Copy Markdown
Collaborator

Description

Related Issue

Motivation and Context

How Has This Been Tested?

Screenshots (if appropriate):

Types of changes

  • Bug fix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)
  • Breaking change (fix or feature that would cause existing functionality to change)

Checklist:

  • I have signed the Adobe Open Source CLA.
  • My change requires a change to the documentation.
  • I have updated the documentation accordingly.
  • I have read the CONTRIBUTING document.
  • I have added tests to cover my changes and the overall coverage did not decrease.
  • All unit tests pass on CircleCi.
  • I ran all tests locally and they pass.

Armaan Gupta and others added 4 commits November 4, 2026 16:19
Add the Slider interface and SliderImpl exposing min, max and step, and
export type "number" explicitly since af-core has no default type for
"range". Register the resource type and reserve the step property so it
is not exported twice. Bump the models.form package version to 5.13.0.

Co-Authored-By: Claude <noreply@anthropic.com>
Add the slider component with a native range input and live value
output, authoring dialog with min/max/step/default and validation
messages, editor clientlib validating the configuration, site clientlib
and styles. Embed the runtime clientlib in the runtime aggregate and
register the component in the package filter.

Co-Authored-By: Claude <noreply@anthropic.com>
…MS-28701)

Add the examples proxy component and library page, plus IT sample forms
covering range, step, read-only, disabled, tooltip and a calculation
using the slider value. Embed the slider runtime clientlib in the IT
aggregate.

Co-Authored-By: Claude <noreply@anthropic.com>
Cover runtime behaviour (model export, live value, clamping, step,
read-only, disabled, calculation, aria-invalid) and authoring dialog
validation.

Co-Authored-By: Claude <noreply@anthropic.com>
@armaang1729 armaang1729 changed the title Forms 28701 FORMS-28701 Nov 5, 2026
Armaan Gupta and others added 2 commits November 10, 2026 22:55
…t (FORMS-28701)

Authoring specs matched two elements per ./minimum, ./maximum and ./step name
selector; use the component BEM classes instead. The runtime disabled test
picked the slider by array index; look it up by name and assert the model
state before the disabled attribute. Add a disabled slider unit test and
exporter fixture.

Co-Authored-By: Claude <noreply@anthropic.com>
…ecs (FORMS-28701)

Revert the models.form package-info bump to 5.12.3: with it, the IT instance
failed to resolve com.adobe.aem.forms.af.rest (uses constraint violation
between two af-core bundles), 404-ing the forms REST endpoints used by many
unrelated specs. Authoring specs now target the inner input and assert the
validator messages.

Co-Authored-By: Claude <noreply@anthropic.com>
@adobe-bot

Copy link
Copy Markdown

Accessibility Violations Found

Id Impact
aria-required-attr critical
empty-heading minor
label-title-only serious
target-size serious

This branch has not been deployed

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

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants