refactor: simplify and fix button rendering logic in network settings

This commit is contained in:
Adam Shiervani 2025-10-08 12:42:44 +02:00
parent 8449911343
commit 638949c289
1 changed files with 22 additions and 28 deletions

View File

@ -222,19 +222,16 @@ export default function SettingsNetworkRoute() {
description="Configure the network settings for the device"
action={
<>
{(formState.isDirty || formState.isSubmitting) && (
<div>
<Button
size="SM"
theme="primary"
disabled={formState.isSubmitting}
disabled={!(formState.isDirty || formState.isSubmitting)}
loading={formState.isSubmitting}
type="submit"
text={formState.isSubmitting ? "Saving..." : "Save Settings"}
/>
</div>
)}
</>
}
/>
@ -420,21 +417,18 @@ export default function SettingsNetworkRoute() {
)}
</AutoHeight>
</div>
{(formState.isDirty || formState.isSubmitting) && (
<>
<div className="h-px w-full bg-slate-800/10 dark:bg-slate-300/20" />
<div className="animate-fadeInStill opacity-0 animation-duration-300">
<div className="animate-fadeInStill animation-duration-300">
<Button
size="SM"
theme="primary"
disabled={formState.isSubmitting}
disabled={!(formState.isDirty || formState.isSubmitting)}
loading={formState.isSubmitting}
type="submit"
text={formState.isSubmitting ? "Saving..." : "Save Settings"}
/>
</div>
</>
)}
</div>
</form>
</FormProvider>