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

Add disabledIndexes support to ActionSheetIOS for parity w/ ActionSheetCustom + clean-up #24

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

jzaleski
Copy link

@jzaleski jzaleski commented Mar 25, 2023

Exposes and adds type-definition to support disabledIndexes which is supported by the underlying RN API (as disabledButtonIndices).

@jzaleski jzaleski force-pushed the master branch 2 times, most recently from 4000da5 to cde2527 Compare March 25, 2023 19:23
@jzaleski jzaleski changed the title Allow setting the RN disabledButtonIndices prop from an ActionSheet prop Add disableIndexes support to ActionSheetIOS for parity w/ ActionSheetCustom + clean-up Mar 25, 2023
@jzaleski
Copy link
Author

jzaleski commented Mar 28, 2023

@alessiocancian added a decent quality of life (I feel) change with this PR, also addressed a bug w/ the disabledIndexes behavior on Android. What do you think?

@jzaleski jzaleski changed the title Add disableIndexes support to ActionSheetIOS for parity w/ ActionSheetCustom + clean-up Add disabledIndexes support to ActionSheetIOS for parity w/ ActionSheetCustom + clean-up Mar 29, 2023
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.

1 participant