manojshetty2004 commented on PR #36229: URL: https://github.com/apache/superset/pull/36229#issuecomment-3607344716
Thanks for the clarification — that makes sense. The DOM-mounted embedding setup I'm describing is definitely outside the official iframe/SDK approach. A few internal apps still use this method, which is why we noticed the spacing issue. The intention here wasn’t to extend Superset’s functionality, just to restore the default layout when it gets unintentionally overridden in this non-standard setup. If this isn’t something Superset wants to support in core, no problem at all — I can handle this on the application side instead. Just let me know what you prefer, and I’m happy to follow the project direction. -- 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]
