Merge pull request #15810 from prometheus/ui-merge-alerts-page-settings

Merge two accidental "Alerts page settings" sections into one
This commit is contained in:
Julius Volz 2025-01-13 18:23:05 +01:00 committed by GitHub
commit 1e7e5b63cf
No known key found for this signature in database
GPG key ID: B5690EEEBB952194

View file

@ -107,36 +107,36 @@ const SettingsMenu: FC = () => {
<Stack>
<Fieldset p="md" legend="Alerts page settings">
<Checkbox
checked={showAnnotations}
label="Show expanded annotations"
onChange={(event) =>
dispatch(
updateSettings({
showAnnotations: event.currentTarget.checked,
})
)
}
/>
</Fieldset>
<Fieldset p="md" legend="Alerts page settings">
<NumberInput
min={1}
allowDecimal={false}
label="Alert groups per page"
value={alertGroupsPerPage}
onChange={(value) => {
if (typeof value !== "number") {
return;
<Stack>
<Checkbox
checked={showAnnotations}
label="Show expanded annotations"
onChange={(event) =>
dispatch(
updateSettings({
showAnnotations: event.currentTarget.checked,
})
)
}
/>
<NumberInput
min={1}
allowDecimal={false}
label="Alert groups per page"
value={alertGroupsPerPage}
onChange={(value) => {
if (typeof value !== "number") {
return;
}
dispatch(
updateSettings({
alertGroupsPerPage: value,
})
);
}}
/>
dispatch(
updateSettings({
alertGroupsPerPage: value,
})
);
}}
/>
</Stack>
</Fieldset>
<Fieldset p="md" legend="Rules page settings">
<NumberInput