mistercrunch commented on PR #29686:
URL: https://github.com/apache/superset/pull/29686#issuecomment-2261338177
Sorry for the confusion, I thought some of the css you're changing here was
around to support the fixed width on the "dynamically labeled" `Run` button,
but looking at the code in more details now, it clearly is not the case.
Digging in - and that's not related to this PR directly, but I'm not sure
why so much of this styling logic is in this file as opposed to simply being
more vanilla antd, or simply living in the component library, but that's out of
scope for this PR.
One question is whether the `&:first-of-type { width: auto; }` is necessary.
Isn't that the default `antd` behavior?
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
