fix: Spacing in alert modal (#22709)

This commit is contained in:
Lyndsi Kay Williams 2023-01-12 21:51:22 -06:00 committed by GitHub
parent 2de19f1d66
commit 2ccdb72830
No known key found for this signature in database
GPG Key ID: 4AEE18F83AFDEB23
1 changed files with 20 additions and 12 deletions

View File

@ -171,6 +171,10 @@ const StyledSectionContainer = styled.div`
display: flex;
flex-direction: column;
.control-label {
margin-top: ${({ theme }) => theme.gridUnit}px;
}
.header-section {
display: flex;
flex: 0 0 auto;
@ -339,6 +343,7 @@ const StyledRadioGroup = styled(Radio.Group)`
const StyledCheckbox = styled(AntdCheckbox)`
margin-left: ${({ theme }) => theme.gridUnit * 5.5}px;
margin-top: ${({ theme }) => theme.gridUnit}px;
`;
// Notification Method components
@ -356,6 +361,12 @@ const StyledNotificationAddButton = styled.div`
}
`;
const StyledNotificationMethodWrapper = styled.div`
.inline-container .input-container {
margin-left: 0;
}
`;
const timezoneHeaderStyle = (theme: SupersetTheme) => css`
margin: ${theme.gridUnit * 3}px 0;
`;
@ -1445,18 +1456,15 @@ const AlertReportModal: FunctionComponent<AlertReportModalProps> = ({
<span className="required">*</span>
</StyledSectionTitle>
{notificationSettings.map((notificationSetting, i) => (
<NotificationMethod
setting={notificationSetting}
index={i}
key={`NotificationMethod-${i}`}
onUpdate={updateNotificationSetting}
onRemove={removeNotificationSetting}
css={css`
.input-container {
margin-left: 0;
}
`}
/>
<StyledNotificationMethodWrapper>
<NotificationMethod
setting={notificationSetting}
index={i}
key={`NotificationMethod-${i}`}
onUpdate={updateNotificationSetting}
onRemove={removeNotificationSetting}
/>
</StyledNotificationMethodWrapper>
))}
<NotificationMethodAdd
data-test="notification-add"