Skip to content

fix(DateTimePicker): remove incorrect combobox role and aria attribute from date range picker inputs#1133

Merged
ychhabra-eightfold merged 7 commits into
EightfoldAI:mainfrom
vrajput-eightfold:main
Jun 8, 2026
Merged

fix(DateTimePicker): remove incorrect combobox role and aria attribute from date range picker inputs#1133
ychhabra-eightfold merged 7 commits into
EightfoldAI:mainfrom
vrajput-eightfold:main

Conversation

@vrajput-eightfold

@vrajput-eightfold vrajput-eightfold commented Jun 4, 2026

Copy link
Copy Markdown
Contributor

SUMMARY:

Removing role="combobox" and aria-haspopup attribute from DateRange picker inputs.

GITHUB ISSUE (Open Source Contributors)

JIRA TASK (Eightfold Employees Only):

https://eightfoldai.atlassian.net/browse/ENG-167620

CHANGE TYPE:

  • Bugfix Pull Request
  • Feature Pull Request

TEST COVERAGE:

  • Tests for this change already exist
  • I have added unittests for this change

TEST PLAN:

Goto /campaigns page on stage and check in filters that the date range picker does not have role="combobox"

@codesandbox-ci

codesandbox-ci Bot commented Jun 4, 2026

Copy link
Copy Markdown

This pull request is automatically built and testable in CodeSandbox.

To see build info of the built libraries, click here or the icon next to each commit SHA.

@vrajput-eightfold vrajput-eightfold changed the title fix(DateTimePicker): remove incorrect combobox role from picker inputs fix(DateTimePicker): remove incorrect combobox role and aria attribute from date range picker inputs Jun 5, 2026
Comment thread src/components/DateTimePicker/Internal/OcRangePicker.tsx
@vrajput-eightfold

Copy link
Copy Markdown
Contributor Author

shipit

@ychhabra-eightfold ychhabra-eightfold merged commit 47ac84c into EightfoldAI:main Jun 8, 2026
3 checks passed
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