| [09:34:31] | <paul121[m]> | Asking about this in Drupal gin Slack, someone said there is a core container-inline class. I tested and it works, basically sets display: inline-block on all children, but isn't perfect. I think we would need some styles on top of it. One thing it does is put all the field labels + descriptions inline as well which I don't think we want. Also doesn't add horizontal spacing between fields. The flex solution feels more elegant :-) |
| [09:34:32] | <paul121[m]> | https://drupal.slack.com/archives/C012J63GVB8/p1679058492478809 |
| [09:50:29] | <mstenta[m]> | Oh interesting |
| [09:50:36] | <mstenta[m]> | Could we just override the styles in farm_ui_theme? |
| [09:51:38] | <paul121[m]> | Probably... but I would worry what that could mess up :-) |
| [09:52:29] | <paul121[m]> | The label/description issue demonstrates why it's hard to make a generic inline class I think |
| [09:52:39] | <paul121[m]> | So perhaps this should be quite specific to quick forms |
| [09:53:21] | <mstenta[m]> | mm yea |
| [09:53:46] | <mstenta[m]> | well i'm glad we at least explored the bigger context before shrinking our target back down to quick forms specifically :-) |