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

fix: add tabIndex support #779

Merged
merged 2 commits into from
Oct 16, 2024

Conversation

wang-yikai
Copy link
Contributor

@wang-yikai wang-yikai commented Mar 25, 2024

I've created this pull request to fix ant-design/ant-design#48059

Copy link

vercel bot commented Mar 25, 2024

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Comments Updated (UTC)
picker ✅ Ready (Inspect) Visit Preview 💬 Add feedback Aug 6, 2024 7:43am

@afc163
Copy link
Member

afc163 commented Apr 29, 2024

Could you add a test case?

@wang-yikai
Copy link
Contributor Author

@afc163 I've added some simple tests for tabIndex. Can you help merge the pull request?

@afc163
Copy link
Member

afc163 commented Apr 30, 2024

Contained conflicting files

@wang-yikai
Copy link
Contributor Author

@afc163 I fixed the conflicts. Let me know if you can merge this pull request, thanks!

@Chevindu
Copy link

Chevindu commented Aug 5, 2024

Hi all, I'd like to know if there is more to be done regarding this PR. Happy to contribute in any way I can to get this merged.

@wang-yikai
Copy link
Contributor Author

@Chevindu Can you resolve the conflicts (https://github.com/react-component/picker/pull/779/conflicts) and merge my pull request? The conflict is just the import order which should've been resolved by my latest push (https://github.com/react-component/picker/compare/477c6bf8899a308c1d5a1f464f465defc5dcfccb..cb7e3ed5383efe7a1f707d7e705b4982e298fe27).

@Chevindu
Copy link

Chevindu commented Aug 6, 2024

@wang-yikai I don't have that access to either the main repo or your fork. Will you please be able to rebase/merge from the upstream master and add another commit? I think that should resolve all conflicts.

@wang-yikai
Copy link
Contributor Author

@Chevindu I've just made a push in my fork. The conflicts should be resolved now. Let me know if you're able to merge, thank you!

@Chevindu
Copy link

Chevindu commented Aug 6, 2024

@wang-yikai Thank you. But I do not have that privilege.

@afc163 Will you kindly be able look into this?

@afc163 afc163 merged commit a6db0d6 into react-component:master Oct 16, 2024
7 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.

DatePicker doesn't support tabIndex
3 participants