Copilot commented on code in PR #37528:
URL: https://github.com/apache/superset/pull/37528#discussion_r2737964175
##########
superset-frontend/src/dashboard/components/DashboardBuilder/DashboardBuilder.tsx:
##########
@@ -314,14 +313,17 @@ const StyledDashboardContent = styled.div<{
}
.dashboard-component-chart-holder {
+ /* Explicit box-sizing ensures consistent behavior in embedded mode,
+ where the global .ant-layout wrapper providing border-box is absent */
+ box-sizing: border-box;
Review Comment:
The PR description incorrectly claims to include a database migration. All
checkboxes under "ADDITIONAL INFORMATION" related to database migrations are
checked (including "Includes DB Migration", "Migration is atomic", "Confirm DB
migration upgrade and downgrade tested", and "Runtime estimates and downtime
expectations provided"), but this PR only contains CSS/styling changes to a
TypeScript React component. No database migration files are included or
modified in this PR.
Please uncheck all the database migration related checkboxes in the PR
description as they do not apply to this change.
--
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]
