tiya-9975 commented on issue #37674: URL: https://github.com/apache/superset/issues/37674#issuecomment-3875362620
Thanks for the suggestion @rusackas, @JustineAssoulant — that makes sense, and I agree it would be a better outcome UX-wise. I initially went the documentation route because the alignment change appeared to be an unintentional side-effect of the Emotion migration rather than a deliberate chart behavior change, and I wasn’t sure if adding a new control was in scope for this issue. That said, adding an explicit alignment control (defaulting to “centered” for backward compatibility, with left/right options) sounds like a cleaner solution than documenting a workaround. I’m happy to explore implementing this if that direction makes sense — my next step would be to understand where alignment is currently being enforced in the Big Number chart component and whether restoring the previous centered default is feasible without unintended side effects. Let me know if you’d prefer: • converting this PR into a feature change, or • keeping this one doc-only and opening a follow-up issue/PR for the control Either way works for me. -- 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]
