Consider removing Inline style of MenuPositioner #3353
Replies: 1 comment 1 reply
-
The goal of the positioner is to dynamically position the menu relative to the trigger. If you want to take full control of the positoning logic, replace the The library is headless and doesn't include any styles by default so you can style them to match design system. If this doesn't meet your need, consider a fully styled library like Park UI, Chakra UI, ShadCN, and so on. |
Beta Was this translation helpful? Give feedback.
1 reply
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
-
Currently, the
<Menu.Positioner/>
component always generartes these inline style:Imo, this limits the creativity for dev. On the other hand, the other component didn't come with pre-style, why would this one needs one? Even worse that inline style can't be overwritten easily which making DX go bad.
My suggestion is to add an option to disable inline style. Doing so won't cause BC and still fit well with philosophy of the library. What do you guy think?
Beta Was this translation helpful? Give feedback.
All reactions