[refactor] Doubled "Container" component #1136
mateuszmazurek
started this conversation in
General
Replies: 0 comments
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
-
I created issue #1135 but it has been "automatically closed because it does not follow the issue template.". There's no adequate issue template for my case so I will paste it here.
I found one confusing thing while working on some feature (it will allow even more customization) for this lib.
We have
style
prop which can be used to apply additional styling "at the sheet container" (as docs says). InsideBottomSheet
component you use internalBottomSheetContainer
component which can be styled using undocumented (but typed)containerStyle
prop.Both props are used to style container but:
containerStyle
prop so as not to mislead this library users ("two containers? wtf?")I need to name this "public" container (this styled using
style
prop, rawAnimated.View
, child of currentBottomSheetContainer
) for the purpose of feature I work on but I'm not convinced by any of my ideas. I'm thinking of renamingBottomSheetContainer
to(edited)BottomSheetRoot
BottomSheetRootContainer
(or something similar) and useBottomSheetContainer
name for the "container" mentioned in docs (this "public" one). Is it good direction? What do you think @gorhom?Beta Was this translation helpful? Give feedback.
All reactions