Skip to content

fix(Hint): support modal props#4031

Draft
wanxiankai wants to merge 1 commit into
wix:masterfrom
wanxiankai:fix/hint-modal-props
Draft

fix(Hint): support modal props#4031
wanxiankai wants to merge 1 commit into
wix:masterfrom
wanxiankai:fix/hint-modal-props

Conversation

@wanxiankai

Copy link
Copy Markdown

Description

Adds a modalProps API to Hint and forwards it to the internal Modal when Hint uses modal mode. This allows React Native Modal props such as supportedOrientations to be set for landscape-only apps.

Closes #3938

Changelog

Hint: added modalProps to pass additional props to the internal Modal.

Additional info

Validation:

  • yarn workspace react-native-ui-lib test packages/react-native-ui-lib/src/components/hint/__tests__/index.spec.tsx --runInBand --watchman=false
  • yarn build:dev
  • yarn eslint packages/react-native-ui-lib/src/components/hint/types.ts packages/react-native-ui-lib/src/components/hint/index.tsx packages/react-native-ui-lib/src/components/hint/Hint.driver.new.ts packages/react-native-ui-lib/src/components/hint/__tests__/index.spec.tsx -c .eslintrc.js

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.

Modal props missing from Hint component

1 participant