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

STSMACOM-848 Expose startLabel and endLabel props for DateRangeFilter #1499

Merged
merged 5 commits into from
Aug 13, 2024

Conversation

JohnC-80
Copy link
Contributor

@JohnC-80 JohnC-80 commented Aug 12, 2024

STSMACOM-848

Problem

<DateRangeFilter> fields are currently labeled as only as "From" and "To" by default... but for better context, they should be labeled more specifically to the context, for example the filters within Data Export have separate date range filters for "Started running" and "Ended running" - these should be labeled to include their "Started running", "Ended running" contexts so that AT users can easily disambiguate the fields.

Approach

Expose startLabel and endLabel props for the DateRangeFilter so that specific aria-labels can be passed to the fields. For sighted users, the fields are already collected within their separate accordions.

Copy link

github-actions bot commented Aug 12, 2024

Jest Unit Test Statistics

0 tests  ±0   0 ✔️ ±0   0s ⏱️ ±0s
0 suites ±0   0 💤 ±0 
0 files   ±0   0 ±0 

Results for commit ebef007. ± Comparison against base commit d5c0f17.

♻️ This comment has been updated with latest results.

Copy link

github-actions bot commented Aug 12, 2024

BigTest Unit Test Statistics

    1 files  ±0      1 suites  ±0   15s ⏱️ -1s
498 tests +2  461 ✔️ +3  37 💤 ±0  0  - 1 
500 runs  +2  463 ✔️ +3  37 💤 ±0  0  - 1 

Results for commit ebef007. ± Comparison against base commit d5c0f17.

♻️ This comment has been updated with latest results.

@JohnC-80 JohnC-80 requested a review from a team August 12, 2024 15:16
@artem-blazhko artem-blazhko requested a review from a team August 13, 2024 06:19
Copy link

@JohnC-80 JohnC-80 merged commit ba72ce0 into master Aug 13, 2024
6 checks passed
@JohnC-80 JohnC-80 deleted the STSMACOM-848 branch August 13, 2024 18:35
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.

3 participants